OpenROAD-flow-scripts icon indicating copy to clipboard operation
OpenROAD-flow-scripts copied to clipboard

makefile: fix empty bash command when user doesn't have klayout

Open kamilrakoczy opened this issue 9 months ago • 0 comments

This PR fixes checking for KLAYOUT_VERSION when KLAYOUT_CMD is missing.

Currently Makefile always tries to check for KLAYOUT_VERSION but when KLAYOUT_CMD variable is empty it tries to execute invalid command and produces following error:

bash: - : invalid option
Usage:    bash [GNU long option] [option] ...
  bash [GNU long option] [option] script-file ...
GNU long options:
 --debug
 --debugger
 --dump-po-strings
 --dump-strings
 --help
 --init-file
 --login
 --noediting
 --noprofile
 --norc
 --posix
 --pretty-print
 --rcfile
 --restricted
 --verbose
 --version
Shell options:
 -ilrsD or -c command or -O shopt_option         (invocation only)
 -abefhkmnptuvxBCHP or -o option

kamilrakoczy avatar May 15 '24 09:05 kamilrakoczy