Skip to content

Commit 155a67d

Browse files
jiangxb1987dongjoon-hyun
authored andcommitted
[SPARK-29666][BUILD] Fix the publish release failure under dry-run mode
### What changes were proposed in this pull request? `release-build.sh` fail to publish release under dry run mode with the following error message: ``` /opt/spark-rm/release-build.sh: line 429: pushd: spark-repo-g4MBm/org/apache/spark: No such file or directory ``` We need to at least run the `mvn clean install` command once to create the `$tmp_repo` path, but now those steps are all skipped under dry-run mode. This PR fixes the issue. ### How was this patch tested? Tested locally. Closes #26329 from jiangxb1987/dryrun. Authored-by: Xingbo Jiang <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent fd6cfb1 commit 155a67d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

dev/create-release/release-build.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -414,13 +414,13 @@ if [[ "$1" == "publish-release" ]]; then
414414

415415
# TODO: revisit for Scala 2.13 support
416416

417-
if ! is_dry_run && [[ $PUBLISH_SCALA_2_11 = 1 ]]; then
417+
if [[ $PUBLISH_SCALA_2_11 = 1 ]]; then
418418
./dev/change-scala-version.sh 2.11
419419
$MVN -DzincPort=$ZINC_PORT -Dmaven.repo.local=$tmp_repo -DskipTests \
420420
$SCALA_2_11_PROFILES $PUBLISH_PROFILES clean install
421421
fi
422422

423-
if ! is_dry_run && [[ $PUBLISH_SCALA_2_12 = 1 ]]; then
423+
if [[ $PUBLISH_SCALA_2_12 = 1 ]]; then
424424
./dev/change-scala-version.sh 2.12
425425
$MVN -DzincPort=$((ZINC_PORT + 2)) -Dmaven.repo.local=$tmp_repo -DskipTests \
426426
$SCALA_2_11_PROFILES $PUBLISH_PROFILES clean install

0 commit comments

Comments
 (0)