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

Duplicate config files in configdir causes merge problems -> set ? = NULL #510

Open
olofhagsand opened this issue Apr 10, 2024 · 0 comments
Open

Duplicate config files in configdir causes merge problems -> set ? = NULL #510

olofhagsand opened this issue Apr 10, 2024 · 0 comments
Labels
bug fixed please verify

Comments

@olofhagsand
Copy link
Member

If CLICON_CONFIGDIR is set, multiple config files are read, not just the main. But if the same content are in two files the merge algorithm is broken.
In particular two autocli.xml files make no clispec generated.
Example, suppose CLICON_CONFIGDIR=/usr/local/etc/clixon/example/ and the dir contains:

  autocli.xml
  autocli.xml~

Then both files will be read. But the merge algorithm will make the autocli structure (in-mem) to be broken, and the cli generating code will genererate no code. Ultimately, there will be no sub.menu to set for example
@olofhagsand olofhagsand changed the title Dublicate config files in configdir causes merge problems -> set ? = NULL Duplicate config files in configdir causes merge problems -> set ? = NULL Apr 11, 2024
olofhagsand added a commit that referenced this issue Apr 11, 2024
@olofhagsand
Fixed: [Duplicate config files in configdir causes merge problems -> …
344786a 
…set ? = NULL](#510)

Made file-dir match more exact, eg .cli not ?cli
Added -1 as any arg to xml_rm_children()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug fixed please verify
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@olofhagsand