search

clicon / clixon

YANG-based toolchain including NETCONF and RESTCONF interfaces and an interactive CLI
http://www.clicon.org/
Other
208 stars 70 forks source link

Avoid additional 'eval echo ...' sequences #460

Closed pprindeville closed 10 months ago

pprindeville commented 10 months ago

Simplifying the previous "fix --with-cligen build option" so that we don't have to keep using eval echo $var.

pprindeville commented 10 months ago

cc: @rcmcdonald91

olofhagsand commented 10 months ago

This seems to be a part of https://github.com/clicon/clixon/pull/459 as well. Why is it broken out /why is it still in #459?

pprindeville commented 10 months ago

This seems to be a part of #459 as well. Why is it broken out /why is it still in #459?

Because #459 will rebase onto #460 and it will go away, but #459 requires it to get a reliable a reliable run.

olofhagsand commented 10 months ago

This part of configure is there primarily on request for downstream build for netgate. Please ensures it works in that context and its OK.

pprindeville commented 10 months ago

This part of configure is there primarily on request for downstream build for netgate. Please ensures it works in that context and its OK.

Yes, tested it and it's working fine. But Netgate is still on 6.3.0, aren't they?