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
