YosysHQ / yosys

Yosys Open SYnthesis Suite
https://yosyshq.net/yosys/
ISC License
3.32k stars 859 forks source link

driver: Fix crashes on missing cli arguments #4206

Closed povik closed 5 months ago

povik commented 5 months ago

In #3973 we switched to using an internal getopt implementation unconditionally, but there turns out to be an issue with missing option arguments which lead to crashes. Address that.

Now we handle it gracefully:

$ ./yosys -p
./yosys: option '-p' expects an argument
Run './yosys -h' for help.