On 19/12/2016 22:41, Mandy Chung wrote:

tools/launcher/modules/patch/systemmodules/PatchSystemModules.java needs to be 
updated since ModuleBootstrap now depends on this new method:

diff --git 
a/test/tools/launcher/modules/patch/systemmodules/src1/java.base/jdk/internal/modules/SystemModules.java
 
b/test/tools/launcher/modules/patch/systemmodules/src1/java.base/jdk/internal/modules/SystemModules.java
--- 
a/test/tools/launcher/modules/patch/systemmodules/src1/java.base/jdk/internal/modules/SystemModules.java
+++ 
b/test/tools/launcher/modules/patch/systemmodules/src1/java.base/jdk/internal/modules/SystemModules.java
@@ -29,4 +29,8 @@
   */
  public final class SystemModules {
      public static final String[] MODULE_NAMES = new String[0];
+
+    public static boolean hasSplitPackages() {
+        return true;
  }
+}

Since this fix has been pushed, I will fix this with a separate issue.

This looks okay, I'm surprised this test wasn't fail for Claes.

-Alan

Reply via email to