Fix shellcheck warnings

Signed-off-by: Kemal Akkoyun <[email protected]>
parca-dev · Nov 8, 2023 · dd8ca52 · dd8ca52
1 parent cd89b65
commit dd8ca52
Showing 1 changed file with 19 additions and 19 deletions.
diff --git a/mojo/build_mojo_image.sh b/mojo/build_mojo_image.sh
@@ -29,21 +29,21 @@ container_engine=${container_engine:=docker}
 extra_cap=${extra_cap:=}
 while [ $# -gt 0 ]; do
     case "$1" in
-        --auth-key)
-            user_key="$2"
-            shift
-            ;;
-        --use-podman)
-            container_engine=podman
-            extra_cap="--cap-add SYS_PTRACE"
-            ;;
-        --mojo-version)
-            mojo_ver="$2"
-            shift
-            ;;
-        --*)
-            echo "Unrecognized option $1"
-            ;;
+    --auth-key)
+        user_key="$2"
+        shift
+        ;;
+    --use-podman)
+        container_engine=podman
+        extra_cap="--cap-add SYS_PTRACE"
+        ;;
+    --mojo-version)
+        mojo_ver="$2"
+        shift
+        ;;
+    --*)
+        echo "Unrecognized option $1"
+        ;;
     esac
     shift $(($# > 0 ? 1 : 0))
 done
@@ -58,11 +58,11 @@ check_options() {
 build_image() {
     check_options
     echo "# Building image with ${container_engine}..."
-    ${container_engine} build --no-cache ${extra_cap} \
-        --build-arg AUTH_KEY=${user_key} \
-        --pull -t modular/mojo-v${mojo_ver} \
+    ${container_engine} build --no-cache "${extra_cap}" \
+        --build-arg AUTH_KEY="${user_key}" \
+        --pull -t modular/mojo-v"${mojo_ver}" \
         --file Dockerfile.mojo .
-    ${container_engine} tag modular/mojo-v${mojo_ver} modular/mojo:latest
+    ${container_engine} tag modular/mojo-v"${mojo_ver}" modular/mojo:latest
 }
 
 # Wrap the build in a function