uci.sh: Fix uci
UCI doesn't support dash inside of section name. So when fetching LEDs settings, replace dash with underscore.
cc @jschlehofer
Merge request reports
Activity
changed milestone to %Turris OS 6.0
requested review from @mvasilek
assigned to @mvasilek
mentioned in merge request luci-rainbow!3 (merged)
mentioned in merge request turris/os/packages!975 (merged)
It is important to know that UCI identifiers and config file names may contain only the characters a-z, 0-9 and _. E.g. no hyphens (-) are allowed. Option values may contain any character (as long they are properly quoted).
Last paragraph in https://openwrt.org/docs/guide-user/base-system/uci#file_syntax
It's acoording to the specification, not a bug ;-)
Thanks! Indeed, it is a good finding. I did not check it before and was not aware of it, but I think sooner or later OpenWrt will experience the same issue, which we are having and it will be time to reconsider their specification and release a new one. Since we are using Linux kernel LED subsystem, which has naming with hyphens.
I will ask them on GitHub to let them know about it.
added 2 commits
mentioned in merge request turris/os/packages!1052 (merged)