Add workaround for illegal access warnings on Java 9

airlift · Mar 19, 2018 · c870da4 · c870da4
1 parent 7e00283
commit c870da4
Showing 1 changed file with 26 additions and 0 deletions.
diff --git a/src/main/java/io/airlift/maven/sphinx/SphinxMojo.java b/src/main/java/io/airlift/maven/sphinx/SphinxMojo.java
@@ -349,19 +349,33 @@ private void runForkedSphinx()
             throws MavenReportException
     {
         String jvmBinary = jvm;
+        boolean sameJvmBinary = false;
 
         if ((jvmBinary == null) || "".equals(jvmBinary)) {
             jvmBinary = System.getProperty("java.home") + File.separator + "bin" + File.separator + "java";
+            sameJvmBinary = true;
         }
         getLog().debug("Using JVM: " + jvmBinary);
 
         Commandline cmdLine = new Commandline();
 
         cmdLine.setWorkingDirectory(basedir);
         cmdLine.setExecutable(jvmBinary);
+
+        // work around for http://bugs.jython.org/issue2582
+        if (sameJvmBinary && isAtLeastJava9()) {
+            cmdLine.createArg().setLine("--add-opens java.base/java.lang=ALL-UNNAMED");
+            cmdLine.createArg().setLine("--add-opens java.base/java.io=ALL-UNNAMED");
+            cmdLine.createArg().setLine("--add-opens java.base/java.nio=ALL-UNNAMED");
+            cmdLine.createArg().setLine("--add-opens java.base/sun.nio.ch=ALL-UNNAMED");
+            cmdLine.createArg().setLine("--add-opens java.xml/com.sun.org.apache.xerces.internal.jaxp=ALL-UNNAMED");
+            cmdLine.createArg().setLine("--add-opens java.xml/com.sun.org.apache.xerces.internal.parsers=ALL-UNNAMED");
+        }
+
         if (argLine != null) {
             cmdLine.createArg().setLine(argLine.replace("\n", " ").replace("\r", " "));
         }
+
         try {
             cmdLine.addEnvironment("CLASSPATH", getPluginClasspath());
         }
@@ -443,4 +457,16 @@ private String getPluginClasspath()
         }
         return classpath.toString();
     }
+
+    @SuppressWarnings("JavaReflectionMemberAccess")
+    private static boolean isAtLeastJava9()
+    {
+        try {
+            Runtime.class.getMethod("version");
+            return true;
+        }
+        catch (NoSuchMethodException e) {
+            return false;
+        }
+    }
 }