scripts: Fix quoting to prevent unintended word splitting

scripts/build-static-libfesvr.sh

@@ -13,5 +13,5 @@ set -e
 
 objs=$(head -n 1 <(make -f <( echo -e 'include Makefile\n$(info $(value fesvr_objs))') -n))
 ar rcs -o libfesvr.a $objs
-cp -f libfesvr.a $RISCV/lib
+cp -f libfesvr.a "${RISCV}/lib"
 

scripts/build-toolchains.sh

@@ -26,10 +26,10 @@ error() {
 
 #taken from riscv-tools to check for open-ocd autoconf versions
 check_version() {
-    $1 --version | awk "NR==1 {if (\$NF>$2) {exit 0} exit 1}" || (
-        echo $3 requires at least version $2 of $1. Aborting.
+    "$1" --version | awk "NR==1 {if (\$NF>$2) {exit 0} exit 1}" || {
+        error "${3} requires at least ${1} version ${2}"
         exit 1
-    )
+    }
 }
 
 TOOLCHAIN="riscv-tools"
@@ -60,22 +60,22 @@ shift $((OPTIND - 1))
 
 if [ "$1" = ec2fast ] ; then
     EC2FASTINSTALL=true
-elif [ -z "$1" ] ; then
+elif [ -n "$1" ] ; then
     TOOLCHAIN="$1"
 fi
 
 
 if [ "$EC2FASTINSTALL" = "true" ]; then
     if [ "$TOOLCHAIN" = "riscv-tools" ]; then
-      cd $RDIR
+      cd "$RDIR"
       git clone https://github.com/firesim/firesim-riscv-tools-prebuilt.git
       cd firesim-riscv-tools-prebuilt
-      git checkout $PRECOMPILED_REPO_HASH
+      git checkout "$PRECOMPILED_REPO_HASH"
       PREBUILTHASH="$(cat HASH)"
-      git -C $CHIPYARD_DIR submodule update --init  toolchains/$TOOLCHAIN
+      git -C "${CHIPYARD_DIR}" submodule update --init "toolchains/${TOOLCHAIN}"
       cd "$CHIPYARD_DIR/toolchains/$TOOLCHAIN"
       GITHASH="$(git rev-parse HEAD)"
-      cd $RDIR
+      cd "$RDIR"
       echo "prebuilt hash: $PREBUILTHASH"
       echo "git      hash: $GITHASH"
       if [[ $PREBUILTHASH == $GITHASH && "$EC2FASTINSTALL" == "true" ]]; then
@@ -104,28 +104,28 @@ if [ "$FASTINSTALL" = true ]; then
     mv distrib "$RISCV"
     # copy HASH in case user wants it later
     cp HASH "$RISCV"
-    cd $RDIR
+    cd "$RDIR"
     rm -rf firesim-riscv-tools-prebuilt
 else
     mkdir -p "$RISCV"
-    git -C $CHIPYARD_DIR submodule update --init --recursive toolchains/$TOOLCHAIN #--jobs 8
+    git -C "${CHIPYARD_DIR}" submodule update --init --recursive "toolchains/${TOOLCHAIN}" #--jobs 8
     cd "$CHIPYARD_DIR/toolchains/$TOOLCHAIN"
     export MAKEFLAGS="-j16"
     #build the actual toolchain
     #./build.sh
     source build.common
     echo "Starting RISC-V Toolchain build process"
-    build_project riscv-fesvr --prefix=$RISCV
-    build_project riscv-isa-sim --prefix=$RISCV --with-fesvr=$RISCV
-    build_project riscv-gnu-toolchain --prefix=$RISCV
-    CC= CXX= build_project riscv-pk --prefix=$RISCV --host=riscv64-unknown-elf
-    build_project riscv-tests --prefix=$RISCV/riscv64-unknown-elf
+    build_project riscv-fesvr --prefix="${RISCV}"
+    build_project riscv-isa-sim --prefix="${RISCV}" --with-fesvr="${RISCV}"
+    build_project riscv-gnu-toolchain --prefix="${RISCV}"
+    CC= CXX= build_project riscv-pk --prefix="${RISCV}" --host=riscv64-unknown-elf
+    build_project riscv-tests --prefix="${RISCV}/riscv64-unknown-elf"
     echo -e "\\nRISC-V Toolchain installation completed!"
 
     # build static libfesvr library for linking into firesim driver (or others)
     cd riscv-fesvr/build
-    $CHIPYARD_DIR/scripts/build-static-libfesvr.sh
-    cd $RDIR
+    "${CHIPYARD_DIR}/scripts/build-static-libfesvr.sh"
+    cd "$RDIR"
     # build linux toolchain
     cd "$CHIPYARD_DIR/toolchains/$TOOLCHAIN/riscv-gnu-toolchain/build"
     make -j16 linux
@@ -133,7 +133,7 @@ else
 
 fi
 
-cd $RDIR
+cd "$RDIR"
 
 echo "export CHIPYARD_TOOLCHAIN_SOURCED=1" > env.sh
 echo "export RISCV=$RISCV" >> env.sh
@@ -153,8 +153,8 @@ if [ "$FASTINSTALL" = "false" ]; then
         cd "$CHIPYARD_DIR/toolchains/$TOOLCHAIN"
         check_version automake 1.14 "OpenOCD build"
         check_version autoconf 2.64 "OpenOCD build"
-        build_project riscv-openocd --prefix=$RISCV --enable-remote-bitbang --enable-jtag_vpi --disable-werror
+        build_project riscv-openocd --prefix="${RISCV}" --enable-remote-bitbang --enable-jtag_vpi --disable-werror
         echo -e "\\nRISC-V OpenOCD installation completed!"
-        cd $RDIR
+        cd "$RDIR"
     fi
 fi

scripts/check-tracegen.sh

@@ -13,11 +13,11 @@ AXE_SHRINK=${AXE_DIR}/src/axe-shrink.py
 PATH=$PATH:${AXE_DIR}/src
 
 grep '.*:.*#.*@' $1 > /tmp/clean-trace.txt
-$TO_AXE /tmp/clean-trace.txt > /tmp/trace.axe
-result=$($AXE check wmo /tmp/trace.axe)
+"$TO_AXE" /tmp/clean-trace.txt > /tmp/trace.axe
+result=$("$AXE" check wmo /tmp/trace.axe)
 
-if [ $result != "OK" ]; then
-    $AXE_SHRINK wmo /tmp/trace.axe
+if [ "$result" != OK ]; then
+    "$AXE_SHRINK" wmo /tmp/trace.axe
 else
-    echo "OK"
+    echo OK
 fi

scripts/firesim-setup.sh

@@ -8,11 +8,11 @@ set -o pipefail
 RDIR=$(pwd)
 scripts_dir="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )"
 
-cd $scripts_dir/..
+cd "${scripts_dir}/.."
 
 # Reenable the FireSim submodule
 git config --unset submodule.sims/firesim.update || true
 git submodule update --init sims/firesim
 cd sims/firesim
-./build-setup.sh $@ --library
-cd $RDIR
+./build-setup.sh "$@" --library
+cd "$RDIR"

scripts/init-submodules-no-riscv-tools.sh

@@ -28,9 +28,9 @@ git config --unset submodule.vlsi/hammer-cad-plugins.update
 # Renable firesim and init only the required submodules to provide
 # all required scala deps, without doing a full build-setup
 git config --unset submodule.sims/firesim.update
-cd $scripts_dir/../sims/
+cd "${scripts_dir}/../sims"
 git submodule update --init firesim
 cd firesim/sim
 git submodule update --init midas
-cd $RDIR
+cd "$RDIR"
 git config submodule.sims/firesim.update none

scripts/init-vlsi.sh

@@ -3,11 +3,9 @@
 set -e
 set -o pipefail
 
-
-
 # Initialize HAMMER and CAD-plugins
 git submodule update --init --recursive vlsi/hammer
 git submodule update --init --recursive vlsi/hammer-cad-plugins
 
 # Initialize HAMMER tech plugin
-git submodule update --init --recursive vlsi/hammer-$1-plugin
+git submodule update --init --recursive vlsi/hammer-"$1"-plugin