search

status-im / nim-confutils

Simplified handling of command line options and config files
Apache License 2.0
64 stars 16 forks source link

Dynlib fix for status-go integration #97

Closed vitvly closed 8 months ago

zah commented 8 months ago

The global variable is unnecessary (it would also infect the library with side effects). The simplest solution is to just declare a local version of commandLineParams (as a regular proc) if the one from the standard library is not declared.

vitvly commented 8 months ago

Done, also updated in https://github.com/status-im/nimbus-eth1/pull/1971.