Re: RFR: 8156499 Update jlink to support creating images with modules that are packaged as multi-release JARs

> On Oct 27, 2016, at 5:22 PM, Steve Drach wrote: > > I’ve put out another webrev, > http://cr.openjdk.java.net/~sdrach/8156499/webrev.05/, that addresses Mandy’s > concerns. In particular I demonstrate that the resultant image is “runnable” > and that a Main class

hg: jigsaw/jake/langtools: fix test for provides table

Changeset: 74b4b6a2ae2b Author:jjg Date: 2016-10-31 16:50 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/74b4b6a2ae2b fix test for provides table ! test/tools/javac/classfiles/attributes/Module/ModuleTest.java !

hg: jigsaw/jake/langtools: initial support for new provides syntax and provides table in Module attribute

Changeset: 08e003561fba Author:jjg Date: 2016-10-31 16:00 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/08e003561fba initial support for new provides syntax and provides table in Module attribute !

hg: jigsaw/jake/jdk: basic support for new provides table in Module attribute

Changeset: 9fcb953ee79d Author:jjg Date: 2016-10-31 15:59 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/9fcb953ee79d basic support for new provides table in Module attribute ! src/java.base/share/classes/java/lang/module/ModuleInfo.java !

hg: jigsaw/jake/langtools: update javac tests

Changeset: 2fb22956b1d1 Author:jjg Date: 2016-10-31 15:22 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/2fb22956b1d1 update javac tests + test/tools/javac/diags/examples/HasBeenDeprecatedRemovalModule/HasBeenDeprecatedRemovalModule.java +

Re: RFR: 8159523. Fix tests depending on absence of -limitmods in VM arguments.

Alan, > On Oct 26, 2016, at 12:53 PM, Alexandre (Shura) Iline > wrote: > > >> On Oct 26, 2016, at 12:00 PM, Alan Bateman wrote: >> >> On 26/10/2016 19:33, Alexandre (Shura) Iline wrote: >> >>> : >>> Like this: >>>

Re: Request Review: JDK-6479237 (cl) Add support for classloader names

> On Oct 31, 2016, at 6:51 AM, Alan Bateman wrote: >> >> if (VM.isModuleSystemInited() && !HashedModules.contains(m)) { > Looks okay, an alternative would be to move that that check to L439. What do you think about this?

hg: jigsaw/jake/langtools: 17 new changesets

Changeset: 0a91e3198c97 Author:jlahoda Date: 2016-10-18 16:00 +0200 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/0a91e3198c97 8167461: jshell tool: Scanner#next() hangs tool Summary: PipeInputStream.read(byte[]...) should only read available bytes; properly resending

hg: jigsaw/jake/nashorn: 8 new changesets

Changeset: 0d5081c856bc Author:sundar Date: 2016-10-19 13:27 +0530 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/0d5081c856bc 8071588: The spec for javax.script.ScriptEngineFactory.getProgram() should specify NPEs thrown Reviewed-by: alanb, hannesw !

hg: jigsaw/jake/jaxws: 2 new changesets

Changeset: 1c988e708a06 Author:lana Date: 2016-10-27 16:29 + URL: http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/1c988e708a06 Added tag jdk-9+142 for changeset 59101416d901 ! .hgtags Changeset: 6e956bc711cb Author:mchung Date: 2016-10-31 13:03 -0700 URL:

hg: jigsaw/jake/jaxp: 3 new changesets

Changeset: bdafa0cc34a9 Author:aefimov Date: 2016-10-21 02:53 +0300 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/bdafa0cc34a9 8167179: Make XSL generated namespace prefixes local to transformation process Reviewed-by: joehw !

hg: jigsaw/jake/hotspot: 11 new changesets

Changeset: 4567b6a9e7e9 Author:jiangli Date: 2016-05-12 17:37 -0400 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/4567b6a9e7e9 8156794: Extend data sharing Reviewed-by: iklam, hseigel, acorn, mschoene ! src/share/vm/runtime/arguments.cpp Changeset: 601485e5640a Author:

hg: jigsaw/jake: 8 new changesets

Changeset: fbead0d3c815 Author:mchung Date: 2016-10-18 13:26 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/fbead0d3c815 8167558: Add new JMOD section for header files and man pages Reviewed-by: alanb ! make/CreateJmods.gmk ! make/Images.gmk Changeset: 1c9b94ce045e Author:

hg: jigsaw/jake/corba: 3 new changesets

Changeset: 408c9c621938 Author:rriggs Date: 2016-10-24 14:53 -0400 URL: http://hg.openjdk.java.net/jigsaw/jake/corba/rev/408c9c621938 8164908: ReflectionFactory support for IIOP and custom serialization Summary: Update IIOP Streams to use ReflectionFactory and Unsafe Reviewed-by:

Re: New proposal for #ReflectiveAccessToNonExportedTypes: Open modules & open packages

On 31/10/2016 17:44, Stephen Felts wrote: That means in the next build as of tomorrow the class format is changed and classes need to be re-generated? Yes, any module-info.java from previous builds will need to be re-compiled. There is also an update to the Module attribute to rev the

Re: Request Review: JDK-6479237 (cl) Add support for classloader names

On 10/26/2016 10:43 AM, David M. Lloyd wrote: On 10/26/2016 10:39 AM, Alan Bateman wrote: On 26/10/2016 16:28, David M. Lloyd wrote: : Some background is in order I guess. I've been reworking a couple of our projects to support both Java 9 and Java 8, using MR JARs and also without MR JARs.

Re: Apache Maven Compiler Plugin 3.6.0 Released

2016/10/30 10:41:36 -0700, Robert Scholte : > The Apache Maven team is pleased to announce the release of the Apache > Maven Compiler Plugin, version 3.6.0 > > The Compiler Plugin is used to compile the sources of your project. > > This version introduces support of

Re: New proposal for #ReflectiveAccessToNonExportedTypes: Open modules & open packages

On 31/10/2016 17:06, Paul Bakker wrote: The proposal looks very good! Thanks for listening to the voiced concerns. When can we expect a new prototype containing the proposed features? We have an initial implementation in the jake forest and expect to have new builds + javadoc on the java.net

Re: New proposal for #ReflectiveAccessToNonExportedTypes: Open modules & open packages

On 31/10/16 17:06, Paul Bakker wrote: > The proposal looks very good! Thanks for listening to the voiced concerns. > When can we expect a new prototype containing the proposed features? Alan just pushed an 'initial implementation of open modules' into the jake forest.

Re: jdk.unsupported module missing from compact profiles

Alan Bateman wrote: On 29/10/2016 23:09, Simon Nash wrote: My application uses sun.misc.SignalHandler which is now in the jdk.unsupported module. This class is available when running with the full JRE but not when running with the compact3 JRE. The sun.misc.* classes are available in the

Re: New proposal for #ReflectiveAccessToNonExportedTypes: Open modules & open packages

The proposal looks very good! Thanks for listening to the voiced concerns. When can we expect a new prototype containing the proposed features? Thanks, Paul On Fri, Oct 28, 2016 at 3:38 PM John Rose wrote: > On Oct 27, 2016, at 9:07 AM, mark.reinh...@oracle.com wrote:

hg: jigsaw/jake/nashorn: Initial implementation of open modules

Changeset: 5401bbd15280 Author:alanb Date: 2016-10-31 15:59 + URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/5401bbd15280 Initial implementation of open modules ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/scripts/ModuleGraphManipulator.java

hg: jigsaw/jake/jaxws: Initial implementation of open modules

Changeset: e4e3f6e64a91 Author:alanb Date: 2016-10-31 15:47 + URL: http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/e4e3f6e64a91 Initial implementation of open modules ! src/java.xml.bind/share/classes/module-info.java ! src/java.xml.ws/share/classes/module-info.java !

hg: jigsaw/jake/langtools: 2 new changesets

Changeset: ce2af21982c1 Author:jlahoda Date: 2016-10-31 15:47 + URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ce2af21982c1 Initial implementation of open modules ! src/java.compiler/share/classes/javax/lang/model/element/ModuleElement.java !

hg: jigsaw/jake/jdk: Initial implementation of open modules

Changeset: eb57bd291112 Author:alanb Date: 2016-10-31 15:46 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/eb57bd291112 Initial implementation of open modules ! make/launcher/Launcher-jdk.jconsole.gmk !

Re: RFR 8160063: Provide a means to disable a plugin via the command line

+1 > On Oct 31, 2016, at 12:11 PM, Sundararajan Athijegannathan > wrote: > > Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.02/ for > https://bugs.openjdk.java.net/browse/JDK-8160063 > > PS. Earlier review thread on slightly different

Re: Request Review: JDK-6479237 (cl) Add support for classloader names

> On Oct 28, 2016, at 2:09 PM, David Holmes wrote: > > : > > Given the BootClassLoader instance is not in fact the boot loader at all I > think it would have been clearer and avoid potential confusion to call this > something more representative of its purpose -

RFR 8160063: Provide a means to disable a plugin via the command line

Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.02/ for https://bugs.openjdk.java.net/browse/JDK-8160063 PS. Earlier review thread on slightly different fix: http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-September/009392.html Thanks, -Sundar

Re: Request Review: JDK-6479237 (cl) Add support for classloader names

On 28/10/2016 04:01, Mandy Chung wrote: On Oct 27, 2016, at 3:01 AM, Alan Bateman wrote: One thing to check is getHashedModuleNames where it assumes the boot Layer exists, I assume this has the potential to NPE if we have any cases where a stack trace is printed

Re: Request Review: JDK-6479237 (cl) Add support for classloader names

On 10/28/2016 4:44 PM, Mandy Chung wrote: On Oct 28, 2016, at 11:11 AM, Brent Christian wrote: Should something be done for STEs returned from StackFrameInfo.toStackTraceElement() ? Good catch - I missed it. I added package-private static methods in