Skip to content

Commit

Permalink
Merge pull request #16300 from chamathns/pr-builder-patch-2
Browse files Browse the repository at this point in the history
Set system property jdk.util.zip.disableZip64ExtraFieldValidation to true in MAVEN_OPTS
  • Loading branch information
chamathns authored Jul 21, 2023
2 parents 5568b38 + ce8e472 commit c060f69
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 5 additions & 5 deletions .github/scripts/pr-builder.sh
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ if [ "$REPO" = "product-is" ]; then

cat pom.xml
export JAVA_HOME=$JAVA_11_HOME
mvn clean install -Djdk.util.zip.disableZip64ExtraFieldValidation=true --batch-mode | tee mvn-build.log
mvn clean install --batch-mode | tee mvn-build.log

PR_BUILD_STATUS=$(cat mvn-build.log | grep "\[INFO\] BUILD" | grep -oE '[^ ]+$')
PR_TEST_RESULT=$(sed -n -e '/\[INFO\] Results:/,/\[INFO\] Tests run:/ p' mvn-build.log)
Expand Down Expand Up @@ -152,9 +152,9 @@ else
fi

if [ "$REPO" = "carbon-kernel" ]; then
mvn clean install -Dmaven.test.skip=true -Djdk.util.zip.disableZip64ExtraFieldValidation=true --batch-mode | tee mvn-build.log
mvn clean install -Dmaven.test.skip=true --batch-mode | tee mvn-build.log
else
mvn clean install -Djdk.util.zip.disableZip64ExtraFieldValidation=true --batch-mode | tee mvn-build.log
mvn clean install --batch-mode | tee mvn-build.log
fi

echo ""
Expand Down Expand Up @@ -227,7 +227,7 @@ else


export JAVA_HOME=$JAVA_11_HOME
mvn clean install -Dmaven.test.skip=true -Djdk.util.zip.disableZip64ExtraFieldValidation=true --batch-mode | tee mvn-build.log
mvn clean install -Dmaven.test.skip=true --batch-mode | tee mvn-build.log

echo "Repo $OUTBOUND_AUTH_OIDC_REPO build complete."
SUB_REPO_BUILD_STATUS=$(cat mvn-build.log | grep "\[INFO\] BUILD" | grep -oE '[^ ]+$')
Expand Down Expand Up @@ -327,7 +327,7 @@ else

export JAVA_HOME=$JAVA_11_HOME
cat pom.xml
mvn clean install -Djdk.util.zip.disableZip64ExtraFieldValidation=true --batch-mode | tee mvn-build.log
mvn clean install --batch-mode | tee mvn-build.log

PR_BUILD_STATUS=$(cat mvn-build.log | grep "\[INFO\] BUILD" | grep -oE '[^ ]+$')
PR_TEST_RESULT=$(sed -n -e '/\[INFO\] Results:/,/\[INFO\] Tests run:/ p' mvn-build.log)
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pr-builder-test-JDK11.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ on:
required: true

env:
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3
MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3 -Djdk.util.zip.disableZip64ExtraFieldValidation=true


jobs:
Expand Down

0 comments on commit c060f69

Please sign in to comment.