experiments/clixon
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed: [Schema Ambiguity Error with openconfig-system re: NTP](https://github.com/clicon/clixon/issues/334)

Browse source
This commit is contained in:
Olof hagsand 2022-07-16 18:11:54 +02:00
parent 1d78241115
commit a846989569
3 changed files with 14 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

4
.github/workflows/ci.yml vendored
View file

@ -2,9 +2,9 @@ name: Clixon CI
on: on:
push: push:
branches: [ master, snmp ] branches: [ master, callhome ]
pull_request: pull_request:
branches: [ master, snmp ] branches: [ master, callhome ]
jobs: jobs:
build: build:

11
CHANGELOG.md
View file

@ -35,7 +35,10 @@
* [3.3.2](#332) Aug 27 2017 * [3.3.2](#332) Aug 27 2017
* [3.3.1](#331) June 7 2017 * [3.3.1](#331) June 7 2017
## SNMP branch ## 5.8.0
Planned: July 2022
### New features
* Clixon SNMP frontend * Clixon SNMP frontend
* Support of SNMP for retreiving and setting values via net-snmp using a MIB-YANG mapping defined in RFC6643. * Support of SNMP for retreiving and setting values via net-snmp using a MIB-YANG mapping defined in RFC6643.
@ -49,11 +52,6 @@
* `--with-mib-generated-yang-dir=DIR` * `--with-mib-generated-yang-dir=DIR`
* Thanks to Siklu Communications LTD for sponsoring this work * Thanks to Siklu Communications LTD for sponsoring this work
## 5.8.0
Planned: July 2022
### New features
* YANG Action (RFC 7950 Section 7.15) * YANG Action (RFC 7950 Section 7.15)
* Register action callback with `action_callback_register()`, see main example * Register action callback with `action_callback_register()`, see main example
* Remains: check list keys, validate output * Remains: check list keys, validate output
@ -124,6 +122,7 @@ Developers may need to change their code
### Corrected Bugs ### Corrected Bugs
* Fixed: [Schema Ambiguity Error with openconfig-system re: NTP](https://github.com/clicon/clixon/issues/334)
* Fixed: [YANG mandatory statements within case nodes do not work](https://github.com/clicon/clixon/issues/344) * Fixed: [YANG mandatory statements within case nodes do not work](https://github.com/clicon/clixon/issues/344)
* Fixed: [Nested YANG choice does not work](https://github.com/clicon/clixon/issues/342) * Fixed: [Nested YANG choice does not work](https://github.com/clicon/clixon/issues/342)
* Fixed: [YANG if-feature does not support nested boolean expression](https://github.com/clicon/clixon/issues/341) * Fixed: [YANG if-feature does not support nested boolean expression](https://github.com/clicon/clixon/issues/341)

13
apps/cli/cli_main.c
View file

@ -781,13 +781,14 @@ main(int argc,
/* Join rest of argv to a single command */ /* Join rest of argv to a single command */
restarg = clicon_strjoin(argc, argv, " "); restarg = clicon_strjoin(argc, argv, " ");
/* If several cligen object variables match same preference, select first /* Clixon hardcodes variable tie-breaks to non-terminals (2)
* There is some unclarities if this should be set to 0 or 1. * There are cases in the autocli such as:
* By default in CLIgen, it is 0 * (<string regexp:"r1" | <string regexp:"r2"){ ... }
* It used to be 1 in Clixon. But see eg https://github.com/clicon/clixon/issues/330 * where r1 and r2 are regexps that overlap.
* There may be cases where there will be "ambiguous". * Alterntaive is to add "preference" keyword in the CLIgen syntax that overrides this.
* Note there may be terminal tiebreaks liuke this which would motivate a setting to "3"?
*/ */
cligen_preference_mode_set(cli_cligen(h), 0); cligen_preference_mode_set(cli_cligen(h), 2);
/* Call start function in all plugins before we go interactive /* Call start function in all plugins before we go interactive
*/ */