Merge pull request vespa-engine#32630 from vespa-engine/aressem/skip-…

…quoting-for-cmdline Skip quoting for multi argument vars.
a-dubaj · Oct 22, 2024 · 453e1c3 · 453e1c3
2 parents 694dc2a + 02df65b
commit 453e1c3
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/.buildkite/release-java-artifacts.sh b/.buildkite/release-java-artifacts.sh
@@ -90,17 +90,18 @@ sign_module() {
         EXTRA_FILES_OPTS="-Dfiles=$(echo "${AFILES[@]}" | sed 's/\ /,/g') -Dtypes=$(echo "${ATYPES[@]}" | sed 's/\ /,/g') -Dclassifiers=$(echo "${ACLASSIFIERS[@]}" | sed 's/\ /,/g')"
     fi
 
-    "$ECHO" "$MVN" --settings="$SOURCE_DIR/.buildkite/settings-publish.xml" \
-          "$MVN_OPTS" gpg:sign-and-deploy-file \
+    # shellcheck disable=2086
+    $ECHO $MVN --settings="$SOURCE_DIR/.buildkite/settings-publish.xml" \
+          $MVN_OPTS gpg:sign-and-deploy-file \
           -Durl="file://$TMP_STAGING" \
           -DrepositoryId=maven-central \
           -DgroupId="$G" \
           -DartifactId="$A" \
           -Dversion="$V" \
           -DpomFile="$POM" \
           -DgeneratePom=false \
-          "$FILE_OPTS" \
-          "$EXTRA_FILES_OPTS"
+          $FILE_OPTS \
+          $EXTRA_FILES_OPTS
 
 }
 export -f sign_module