fixed Tcl lint

Signed-off-by: Cho Moon <[email protected]>

Author: precisionmoon

dev_env.sh

@@ -3,7 +3,7 @@
 # Set developer paths and environment variables here,
 # user settings go in ./env.sh
 function __setpaths() {
-    local DIR=$(readlink -f "$(dirname "${BASH_SOURCE[0]}")")
+    local DIR=$(readlink -f "$(dirname "${BASH_SOURCE[0]:-${(%):-%x}}")")
     [ "$(find $DIR/dependencies -type f -user root)" ] && echo "WARNING! Files set up by sudo found in $DIR"
     export PATH="$DIR/dependencies/bin:$PATH"
     export CMAKE_INSTALL_RPATH=$DIR/dependencies/lib:$DIR/dependencies/lib64

env.sh

@@ -1,9 +1,9 @@
 #!/usr/bin/env bash
 function __setpaths() {
   if [[ "$OSTYPE" == "darwin"* ]]; then
-    DIR="$(dirname $(perl -e 'use Cwd "abs_path";print abs_path(shift)' "${BASH_SOURCE[0]}"))"
+    DIR="$(dirname $(perl -e 'use Cwd "abs_path";print abs_path(shift)' "${BASH_SOURCE[0]:-${(%):-%x}}"))"
   else
-    DIR="$(dirname $(readlink -f "${BASH_SOURCE[0]}"))"
+    DIR="$(dirname $(readlink -f "${BASH_SOURCE[0]:-${(%):-%x}}"))"
   fi
 
   export OPENROAD=${DIR}/tools/OpenROAD

flow/scripts/load.tcl

@@ -18,15 +18,19 @@ proc load_design { design_file sdc_file } {
       }
     }
     read_verilog $::env(RESULTS_DIR)/$design_file
-    if { [env_var_exists_and_non_empty SYNTH_WRAPPED_OPERATORS] ||
-         [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {
+    if {
+      [env_var_exists_and_non_empty SYNTH_WRAPPED_OPERATORS] ||
+      [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS]
+    } {
       link_design -hier $::env(DESIGN_NAME)
     } else {
       link_design $::env(DESIGN_NAME)
     }
   } elseif { $ext == ".odb" } {
-    if { [env_var_exists_and_non_empty SYNTH_WRAPPED_OPERATORS] ||
-         [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {
+    if {
+      [env_var_exists_and_non_empty SYNTH_WRAPPED_OPERATORS] ||
+      [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS]
+    } {
       read_db -hier $::env(RESULTS_DIR)/$design_file
     } else {
       read_db $::env(RESULTS_DIR)/$design_file

flow/scripts/synth.tcl

@@ -56,8 +56,10 @@ json -o $::env(RESULTS_DIR)/mem.json
 exec -- $::env(PYTHON_EXE) $::env(SCRIPTS_DIR)/mem_dump.py \
   --max-bits $::env(SYNTH_MEMORY_MAX_BITS) $::env(RESULTS_DIR)/mem.json
 
-if { [env_var_exists_and_non_empty SYNTH_WRAPPED_OPERATORS] ||
-     [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } {
+if {
+  [env_var_exists_and_non_empty SYNTH_WRAPPED_OPERATORS] ||
+  [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS]
+} {
   source $::env(SCRIPTS_DIR)/synth_wrap_operators.tcl
 } else {
   synth -top $::env(DESIGN_NAME) -run fine: {*}$synth_full_args

flow/scripts/variables.mk

@@ -215,7 +215,15 @@ vars:
 .PHONY: print-%
 # Print any variable, for instance: make print-DIE_AREA
 print-%:
-	$(file >$(OBJECTS_DIR)/print_tmp_$$,$($*))
+  # HERE BE DRAGONS!
+  #
+  # We have to use /tmp. $(OBJECTS_DIR) may not exist
+  # at $(file) expansion time, which is before commands are run
+  # here, so we can't mkdir -p $(OBJECTS_DIR) either
+  #
+  # We have to use $(file ...) because we want to be able
+  # to print variables that contain newlines.
+	$(file >/tmp/print_tmp$$,$($*))
 	@echo -n "$* = "
-	@cat $(OBJECTS_DIR)/print_tmp_$$
-	@rm $(OBJECTS_DIR)/print_tmp_$$
+	@cat /tmp/print_tmp$$
+	@rm /tmp/print_tmp$$