osmith submitted this change.

View Change

Approvals: pespin: Looks good to me, but someone else must approve dexter: Looks good to me, approved Jenkins Builder: Verified
gradlew: fix "Unrecognized option: --add-opens"

Comment out the code that causes the following errors during build:

+ ./gradlew test
Unrecognized option: --add-opens
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

The build did not abort but this is distracting.

Change-Id: I788e44cfe20ba6a331e8247677ecb5a526c433f7
---
M gradlew
1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/gradlew b/gradlew
index 17a9170..f03a59b 100755
--- a/gradlew
+++ b/gradlew
@@ -162,9 +162,10 @@
APP_ARGS=$(save "$@")

# Collect all arguments for the java command, following the shell quoting and substitution rules
-if $JAVACMD --add-opens java.base/java.lang=ALL-UNNAMED -version ; then
- DEFAULT_JVM_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED $DEFAULT_JVM_OPTS"
-fi
+# NOTE: commented out as this doesn't work with openjdk-8
+# if $JAVACMD --add-opens java.base/java.lang=ALL-UNNAMED -version ; then
+# DEFAULT_JVM_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED $DEFAULT_JVM_OPTS"
+# fi

eval set -- $DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS "\"-Dorg.gradle.appname=$APP_BASE_NAME\"" -classpath "\"$CLASSPATH\"" org.gradle.wrapper.GradleWrapperMain "$APP_ARGS"


To view, visit change 39583. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: aram-applet
Gerrit-Branch: master
Gerrit-Change-Id: I788e44cfe20ba6a331e8247677ecb5a526c433f7
Gerrit-Change-Number: 39583
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>