Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

YANG if-feature does not support nested boolean expression #341

Closed
olofhagsand opened this issue Jun 24, 2022 · 1 comment
Closed

YANG if-feature does not support nested boolean expression #341

olofhagsand opened this issue Jun 24, 2022 · 1 comment
Labels
bug

Comments

@olofhagsand
Copy link
Member

Clixon does not support nested boolean expresssions in if-feature argument as described in RFC 7950 Section 7.20.2.
This is noted in https://clixon-docs.readthedocs.io/en/latest/standards.html#yang
Clixon terminates when encountering such a statement.
This was noted when loading the new RFC 9243 ietf-dhcpv6-client YANG module: https://github.com/YangModels/yang/blob/main/standard/ietf/RFC/ietf-dhcpv6-client%402022-06-20.yang
Please fix:

  1. Do not exit when encountering nested if-feature-expr
  2. Implement the boolean logic of if-feature-expr
olofhagsand added a commit that referenced this issue Jun 24, 2022
@olofhagsand
Workaround for [YANG if-feature does not support nested boolean expre…
5bb3767 
…ssion](#341)
olofhagsand added a commit that referenced this issue Jun 26, 2022
@olofhagsand
* Full RFC 7950 if-feature-expr support (Section 7.20.2)
a00e36c 
  * Previous implementation did not handle nested if-feature expressions
  * As part of fixing: [YANG if-feature does not support nested boolean expression](#341)
  * Added new yacc/lex parser for if-feature-expr string
@olofhagsand
Copy link
Member Author

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@olofhagsand