kconfig*.sh: fix shellcheck errors

used double-quotes and export tested with: "shellcheck kconfig-*.sh" Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
thesofproject · Apr 29, 2024 · 8fee06f · 8fee06f
1 parent bf11017
commit 8fee06f
Show file tree

Hide file tree

Showing 9 changed files with 92 additions and 90 deletions.
diff --git a/kconfig-distro-sof-dev-update.sh b/kconfig-distro-sof-dev-update.sh
@@ -2,22 +2,22 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 echo "-sof" > localversion
 make olddefconfig
 make localmodconfig
 
 $COMMAND .config \
-	 $KCONFIG_DIR/sof-defconfig  \
-	 $KCONFIG_DIR/sof-dev-defconfig  \
-	 $KCONFIG_DIR/amd-defconfig \
-	 $KCONFIG_DIR/mach-driver-defconfig \
-	 $KCONFIG_DIR/hdaudio-codecs-defconfig \
-	 $KCONFIG_DIR/lock-stall-defconfig \
-	 $KCONFIG_DIR/soundwire-defconfig \
-	 $KCONFIG_DIR/soundwire-codecs-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/sof-defconfig  \
+	 "$KCONFIG_DIR"/sof-dev-defconfig  \
+	 "$KCONFIG_DIR"/amd-defconfig \
+	 "$KCONFIG_DIR"/mach-driver-defconfig \
+	 "$KCONFIG_DIR"/hdaudio-codecs-defconfig \
+	 "$KCONFIG_DIR"/lock-stall-defconfig \
+	 "$KCONFIG_DIR"/soundwire-defconfig \
+	 "$KCONFIG_DIR"/soundwire-codecs-defconfig \
+	 "$@"
diff --git a/kconfig-distro-sof-update.sh b/kconfig-distro-sof-update.sh
@@ -2,20 +2,20 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 echo "-sof" > localversion
 make olddefconfig
 make localmodconfig
 
 $COMMAND .config \
-	 $KCONFIG_DIR/sof-defconfig  \
-	 $KCONFIG_DIR/amd-defconfig \
-	 $KCONFIG_DIR/mach-driver-defconfig \
-	 $KCONFIG_DIR/hdaudio-codecs-defconfig \
-	 $KCONFIG_DIR/lock-stall-defconfig \
-	 $KCONFIG_DIR/soundwire-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/sof-defconfig  \
+	 "$KCONFIG_DIR"/amd-defconfig \
+	 "$KCONFIG_DIR"/mach-driver-defconfig \
+	 "$KCONFIG_DIR"/hdaudio-codecs-defconfig \
+	 "$KCONFIG_DIR"/lock-stall-defconfig \
+	 "$KCONFIG_DIR"/soundwire-defconfig \
+	 "$@"
diff --git a/kconfig-hda.sh b/kconfig-hda.sh
@@ -2,14 +2,14 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 make defconfig
 $COMMAND .config \
-	 $KCONFIG_DIR/base-defconfig \
-	 $KCONFIG_DIR/devices-defconfig \
-	 $KCONFIG_DIR/hdaudio-codecs-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/base-defconfig \
+	 "$KCONFIG_DIR"/devices-defconfig \
+	 "$KCONFIG_DIR"/hdaudio-codecs-defconfig \
+	 "$@"
diff --git a/kconfig-lib.sh b/kconfig-lib.sh
@@ -1,4 +1,6 @@
-
+#!/bin/bash
+# shellcheck disable=SC2034
+# ^-----^ SC2034 (warning): COMMAND appears unused.
 BUILD_DIR=$(pwd)
 
 # find merge_config in code directory

diff --git a/kconfig-minimize-distro-add-sof-defaults.sh b/kconfig-minimize-distro-add-sof-defaults.sh
@@ -2,18 +2,18 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 $COMMAND .config \
-	 $KCONFIG_DIR/minimize-defconfig \
-	 $KCONFIG_DIR/sof-defconfig  \
-	 $KCONFIG_DIR/sof-dev-defconfig  \
-	 $KCONFIG_DIR/mach-driver-defconfig \
-	 $KCONFIG_DIR/hdaudio-codecs-defconfig \
-	 $KCONFIG_DIR/telemetry-debugfs-defconfig \
-	 $KCONFIG_DIR/lock-stall-defconfig \
-	 $KCONFIG_DIR/soundwire-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/minimize-defconfig \
+	 "$KCONFIG_DIR"/sof-defconfig  \
+	 "$KCONFIG_DIR"/sof-dev-defconfig  \
+	 "$KCONFIG_DIR"/mach-driver-defconfig \
+	 "$KCONFIG_DIR"/hdaudio-codecs-defconfig \
+	 "$KCONFIG_DIR"/telemetry-debugfs-defconfig \
+	 "$KCONFIG_DIR"/lock-stall-defconfig \
+	 "$KCONFIG_DIR"/soundwire-defconfig \
+	 "$@"
diff --git a/kconfig-sof-arm64.sh b/kconfig-sof-arm64.sh
@@ -2,15 +2,15 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 make defconfig ARCH=arm64
 $COMMAND -m .config \
-	 $KCONFIG_DIR/arm64-nobloat-defconfig \
-	 $KCONFIG_DIR/sof-defconfig  \
-	 $KCONFIG_DIR/sof-dev-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/arm64-nobloat-defconfig \
+	 "$KCONFIG_DIR"/sof-defconfig  \
+	 "$KCONFIG_DIR"/sof-dev-defconfig \
+	 "$@"
 make olddefconfig ARCH=arm64
diff --git a/kconfig-sof-default-nodev.sh b/kconfig-sof-default-nodev.sh
@@ -2,18 +2,18 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 make defconfig
 $COMMAND .config \
-	 $KCONFIG_DIR/base-defconfig \
-	 $KCONFIG_DIR/devices-defconfig \
-	 $KCONFIG_DIR/sof-defconfig  \
-	 $KCONFIG_DIR/amd-defconfig \
-	 $KCONFIG_DIR/mach-driver-defconfig \
-	 $KCONFIG_DIR/hdaudio-codecs-defconfig \
-	 $KCONFIG_DIR/soundwire-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/base-defconfig \
+	 "$KCONFIG_DIR"/devices-defconfig \
+	 "$KCONFIG_DIR"/sof-defconfig  \
+	 "$KCONFIG_DIR"/amd-defconfig \
+	 "$KCONFIG_DIR"/mach-driver-defconfig \
+	 "$KCONFIG_DIR"/hdaudio-codecs-defconfig \
+	 "$KCONFIG_DIR"/soundwire-defconfig \
+	 "$@"
diff --git a/kconfig-sof-default.sh b/kconfig-sof-default.sh
@@ -2,24 +2,24 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 make defconfig
 $COMMAND .config \
-	 $KCONFIG_DIR/base-defconfig \
-	 $KCONFIG_DIR/devices-defconfig \
-	 $KCONFIG_DIR/sof-defconfig  \
-	 $KCONFIG_DIR/sof-dev-defconfig  \
-	 $KCONFIG_DIR/amd-defconfig \
-	 $KCONFIG_DIR/avs-defconfig \
-	 $KCONFIG_DIR/mach-driver-defconfig \
-	 $KCONFIG_DIR/hdaudio-codecs-defconfig \
-	 $KCONFIG_DIR/telemetry-debugfs-defconfig \
-	 $KCONFIG_DIR/lock-stall-defconfig \
-	 $KCONFIG_DIR/soundwire-defconfig \
-	 $KCONFIG_DIR/soundwire-codecs-defconfig \
-	 $KCONFIG_DIR/bpf-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/base-defconfig \
+	 "$KCONFIG_DIR"/devices-defconfig \
+	 "$KCONFIG_DIR"/sof-defconfig  \
+	 "$KCONFIG_DIR"/sof-dev-defconfig  \
+	 "$KCONFIG_DIR"/amd-defconfig \
+	 "$KCONFIG_DIR"/avs-defconfig \
+	 "$KCONFIG_DIR"/mach-driver-defconfig \
+	 "$KCONFIG_DIR"/hdaudio-codecs-defconfig \
+	 "$KCONFIG_DIR"/telemetry-debugfs-defconfig \
+	 "$KCONFIG_DIR"/lock-stall-defconfig \
+	 "$KCONFIG_DIR"/soundwire-defconfig \
+	 "$KCONFIG_DIR"/soundwire-codecs-defconfig \
+	 "$KCONFIG_DIR"/bpf-defconfig \
+	 "$@"
diff --git a/kconfig-sof-nocodec.sh b/kconfig-sof-nocodec.sh
@@ -2,20 +2,20 @@
 
 set -e
 
-KCONFIG_DIR=$(dirname ${BASH_SOURCE[0]})
-echo $KCONFIG_DIR
+KCONFIG_DIR=$(dirname "${BASH_SOURCE[0]}")
+echo "$KCONFIG_DIR"
 
-. $KCONFIG_DIR/kconfig-lib.sh
+. "$KCONFIG_DIR"/kconfig-lib.sh
 
 make defconfig
 $COMMAND .config \
-	 $KCONFIG_DIR/base-defconfig    \
-	 $KCONFIG_DIR/devices-defconfig \
-	 $KCONFIG_DIR/sof-defconfig     \
-	 $KCONFIG_DIR/sof-dev-defconfig \
-	 $KCONFIG_DIR/amd-defconfig \
-	 $KCONFIG_DIR/nocodec-defconfig \
-	 $KCONFIG_DIR/telemetry-debugfs-defconfig \
-	 $KCONFIG_DIR/lock-stall-defconfig \
-	 $KCONFIG_DIR/bpf-defconfig \
-	 $@
+	 "$KCONFIG_DIR"/base-defconfig    \
+	 "$KCONFIG_DIR"/devices-defconfig \
+	 "$KCONFIG_DIR"/sof-defconfig     \
+	 "$KCONFIG_DIR"/sof-dev-defconfig \
+	 "$KCONFIG_DIR"/amd-defconfig \
+	 "$KCONFIG_DIR"/nocodec-defconfig \
+	 "$KCONFIG_DIR"/telemetry-debugfs-defconfig \
+	 "$KCONFIG_DIR"/lock-stall-defconfig \
+	 "$KCONFIG_DIR"/bpf-defconfig \
+	 "$@"