Merge pull request #1159 from abelsromero/issue-1155-remove-open-access-warnings-from-cli-for-v2.5.x

(v2.5.x) Remove open access warnings from cli

Author: robertpanzer

CHANGELOG.adoc

@@ -307,6 +307,7 @@ Improvements::
   * Upgrade to Asciidoctor 2.0.8
   * Upgrade to asciidoctor-pdf 1.5.0-alpha.17 (#809)
   * Add Rouge source highlighter to asciidoctor.jar (#806)
+  * Add required `--add-opens` to cli launch script to remove Jdk warnings when running on Java8 (#1155) (@abelsromero)
 
 Bug Fixes::
 

asciidoctorj-distribution/src/main/resources/customUnixStartScript.txt

@@ -29,6 +29,10 @@ APP_BASE_NAME=`basename "\$0"`
 
 # Add default JVM options here. You can also use JAVA_OPTS and ${optsEnvironmentVar} to pass JVM options to this script.
 DEFAULT_JVM_OPTS=${defaultJvmOpts}
+readonly java_version=\$(java -version 2>&1 | sed -n ';s/.* version "\\(.*\\)\\.\\(.*\\)\\..*".*/\\1\\.\\2/p;')
+if [ "\$java_version" != "1.8" ]; then
+  DEFAULT_JVM_OPTS="--add-opens java.base/sun.nio.ch=ALL-UNNAMED --add-opens java.base/java.io=ALL-UNNAMED \$DEFAULT_JVM_OPTS"
+fi
 
 # Use the maximum available, or set MAX_FD != -1 to use that value.
 MAX_FD="maximum"
@@ -164,4 +168,4 @@ APP_ARGS=\$(save "\$@")
 # Collect all arguments for the java command, following the shell quoting and substitution rules
 eval set -- \$DEFAULT_JVM_OPTS \$JAVA_OPTS \$${optsEnvironmentVar} <% if ( appNameSystemProperty ) { %>"\"-D${appNameSystemProperty}=\$APP_BASE_NAME\"" <% } %>-classpath "\"\$CLASSPATH\"" ${mainClassName} "\$APP_ARGS"
 
-exec "\$JAVACMD" "\$@"
+exec "\$JAVACMD" "\$@"