scripts: module-check: replace eval with indirection

freifunk-gluon · Jul 1, 2024 · 12b9308 · 12b9308
1 parent cf0d2ce
commit 12b9308
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/scripts/module_check.sh b/scripts/module_check.sh
@@ -1,5 +1,4 @@
 #!/bin/bash
-# shellcheck disable=SC2154
 
 set -e
 
@@ -15,8 +14,10 @@ fi
 need_sync=false
 
 for module in $GLUON_MODULES; do
-	var=$(echo "$module" | tr '[:lower:]/' '[:upper:]_')
-	eval 'commit_expected=${'"${var}"'_COMMIT}'
+	echo "Checking module '$module'"
+	var=${module//\//_}
+	_remove_commit=${var^^}_COMMIT
+	commit_expected=${!_remove_commit}
 
 	prefix=invalid
 	cd "$GLUONDIR/$module" 2>/dev/null && prefix="$(git rev-parse --show-prefix 2>/dev/null)"