Skip to content

Commit bc5a4d5

Browse files
committed
Remove double quoting of bazel_config
The bazel_config variable can either contain a java-specific config or the empty string. When the bazel_config is empty, it should not pass anything to the bazel command line, otherwise it would be interpreted as an additional empty parameter and failing the build. The builds for Gerrit v3.12 were failing because of this extra empty bazel_config, therefore needed manual adjustments before this change. Change-Id: I24970921f72965500093179812eda0510c148f22
1 parent 4e3080c commit bc5a4d5

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

jenkins-docker/agent-release/gerrit-release.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ git tag -f -s -m "v$version" "v$version"
6262
git submodule foreach 'if [ "$path" != "modules/jgit" ]; then git tag -f -s -m "v$version" "v$version"; fi'
6363

6464
bazelisk build $bazel_config release Documentation:searchfree
65-
./tools/maven/api.sh install "$bazel_config"
65+
./tools/maven/api.sh install $bazel_config
6666

6767
echo -n "Checking Gerrit version ... "
6868

@@ -87,8 +87,8 @@ fi
8787

8888
echo "Publishing Gerrit WAR and APIs to Maven Central ..."
8989
export VERBOSE=1
90-
./tools/maven/api.sh war_deploy "$bazel_config"
91-
./tools/maven/api.sh deploy "$bazel_config"
90+
./tools/maven/api.sh war_deploy $bazel_config
91+
./tools/maven/api.sh deploy $bazel_config
9292

9393
echo "Download the artifacts from SonaType staging repository at https://oss.sonatype.org"
9494
echo "logging in using your credentials"

0 commit comments

Comments
 (0)