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