hg: jigsaw/jake/langtools: Add classfile Module_attribute.OPensEntry

Changeset: 51f0bf95d616 Author:jjg Date: 2016-11-22 18:42 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/51f0bf95d616 Add classfile Module_attribute.OPensEntry ! src/jdk.jdeps/share/classes/com/sun/tools/classfile/ClassWriter.java ! src/jdk.jdeps/share/classes/com/

hg: jigsaw/jake/langtools: remove support for old-style Module attribute

Changeset: ea87091ceefc Author:jjg Date: 2016-11-22 18:18 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ea87091ceefc remove support for old-style Module attribute ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassWriter.java ! src/jdk.jdeps/share/class

hg: jigsaw/jake/langtools: fix 2 failing tests after Module attribute changes

Changeset: ecda27ca1011 Author:jjg Date: 2016-11-22 17:49 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ecda27ca1011 fix 2 failing tests after Module attribute changes ! test/tools/javac/classfiles/attributes/Module/ModuleTestBase.java ! test/tools/javac/modules/O

hg: jigsaw/jake/jdk: Fix FieldSetAccessibleTest test to work with setAccessible

Changeset: 6a19edab7f47 Author:mchung Date: 2016-11-22 16:38 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/6a19edab7f47 Fix FieldSetAccessibleTest test to work with setAccessible ! test/ProblemList.txt ! test/java/lang/Class/getDeclaredField/FieldSetAccessibleTest.java

hg: jigsaw/jake/langtools: 2 new changesets

Changeset: 0ade799cfe9d Author:mchung Date: 2016-11-22 15:33 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/0ade799cfe9d Check ACC_MODULE before getting this_class name ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/ClassFileReader.java ! src/jdk.jdeps/share/cla

hg: jigsaw/jake/jdk: 2 new changesets

Changeset: 58f07216d2e7 Author:mchung Date: 2016-11-22 15:32 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/58f07216d2e7 Fix test to check ACC_MODULE before getting this_class name ! test/jdk/internal/reflect/CallerSensitive/CallerSensitiveFinder.java Changeset: 8aecfe7

Re: Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

+1 > On Nov 22, 2016, at 5:07 PM, Mandy Chung wrote: > > This patch moves src.zip and jrt-fs.jar from the top-level into > the `lib` directory in the run-time image as we proposed [1]. > > Webrev: > http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/ > > thanks > Mandy > [1] >

hg: jigsaw/jake/jdk: Minor typo in com.sun.jdi.ModuleReference class spec

Changeset: ea07cfe18be3 Author:mchung Date: 2016-11-22 13:23 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/ea07cfe18be3 Minor typo in com.sun.jdi.ModuleReference class spec ! src/jdk.jdi/share/classes/com/sun/jdi/ModuleReference.java

Review Request: JDK-8169816 Move src.zip and jrt-fs.jar under the lib directory

This patch moves src.zip and jrt-fs.jar from the top-level into the `lib` directory in the run-time image as we proposed [1]. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169816/webrev.00/ thanks Mandy [1] http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-November/010128.html

hg: jigsaw/jake/jdk: Residual references to exports-private in tests

Changeset: 73203c93bde8 Author:alanb Date: 2016-11-22 20:50 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/73203c93bde8 Residual references to exports-private in tests ! test/java/awt/patchlib/java.desktop/java/awt/Helper.java ! test/java/lang/instrument/RedefineModuleAg

hg: jigsaw/jake/hotspot: Missing since= in new JVMTI functions

Changeset: 4e2ce42c5ef8 Author:alanb Date: 2016-11-22 17:53 + URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/4e2ce42c5ef8 Missing since= in new JVMTI functions ! src/share/vm/prims/jvmti.xml

Re: [9] RFR: 8170120: Several new jimage testcases should be excluded due to their failures

+1 > On Nov 22, 2016, at 8:51 AM, Denis Kononenko > wrote: > > > Hi, > > Could someone please review this small fix. > > Recently several failing test cases were delivered into jdk9-dev by my > mistake (https://bugs.openjdk.java.net/browse/JDK-8167240). I've just renamed > these test case

hg: jigsaw/jake/jdk: 4 new changesets

Changeset: 755476f7207d Author:alanb Date: 2016-11-22 13:31 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/755476f7207d @throws SecurityManager instead of SecurityException ! src/java.base/share/classes/java/lang/invoke/MethodHandles.java Changeset: f5010fea98b1 Author:

Re: [9] RFR: 8170120: Several new jimage testcases should be excluded due to their failures

Hi, Looks OK to me. —Andrey > On 22 Nov 2016, at 15:51, Denis Kononenko wrote: > > > Hi, > > Could someone please review this small fix. > > Recently several failing test cases were delivered into jdk9-dev by my > mistake (https://bugs.openjdk.java.net/browse/JDK-8167240). I've just rename

hg: jigsaw/jake/langtools: --add-exports-private changed to --add-opens.

Changeset: 117386f5ab3e Author:jlahoda Date: 2016-11-22 15:35 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/117386f5ab3e --add-exports-private changed to --add-opens. ! test/tools/javac/platform/PlatformProviderTest.java

Re: Annotation processors and the --processor-module-path

On 22/11/2016 10:33, Eirik Bjørsnøs wrote: : So again, I added -J-add-modules=java.xml.bind, and now my annotation processor is loading. Given that my annotation processor does require java.xml.bind and I'm using --processor-module-path, my expectation would be that javac would resolve that re

[9] RFR: 8170120: Several new jimage testcases should be excluded due to their failures

Hi, Could someone please review this small fix. Recently several failing test cases were delivered into jdk9-dev by my mistake (https://bugs.openjdk.java.net/browse/JDK-8167240). I've just renamed these test cases to temporarily avoid their execution. They should be renamed back when approp

Re: Annotation processors and the --processor-module-path

Found it! With --processor-module-path, JavacProcessingEnvironment.java:257 throws an exception when calling fileManager.getServiceLoader(ANNOTATION_PROCESSOR_MODULE_PATH, Processor.class) There's a catch for this exception at JavaCompiler, line 982: catch (Abort ex) { if (devVerbose)