hg: jigsaw/jake/jdk: Add test for checking StackTraceElement format

2016-09-15 Thread mandy . chung
Changeset: e0864f0eb268 Author:mchung Date: 2016-09-15 15:17 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/e0864f0eb268 Add test for checking StackTraceElement format + test/java/lang/StackTraceElement/SerialTest.java

hg: jigsaw/jake/jaxp: 6 new changesets

2016-09-15 Thread mandy . chung
Changeset: bba703e3281b Author:joehw Date: 2016-09-07 11:00 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/bba703e3281b 8165116: redirect function is not allowed even with enableExtensionFunctions Reviewed-by: lancea !

hg: jigsaw/jake: 7 new changesets

2016-09-15 Thread mandy . chung
Changeset: 24efaaddd376 Author:sundar Date: 2016-09-08 14:35 +0530 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/24efaaddd376 8165595: Main class should be set for nashorn modules Reviewed-by: jlaskey, erikj ! make/CreateJmods.gmk Changeset: 6e62bbd02e6b Author:lana Date:

hg: jigsaw/jake/langtools: 7 new changesets

2016-09-15 Thread mandy . chung
Changeset: 589ff4d43428 Author:vromero Date: 2016-09-06 17:04 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/589ff4d43428 8162546: change hidden options -Xdebug to --debug, -XshouldStop to --should-stop, and -diags to --diags Reviewed-by: mcimadamore !

hg: jigsaw/jake/nashorn: 5 new changesets

2016-09-15 Thread mandy . chung
Changeset: 925e7b26b363 Author:hannesw Date: 2016-09-07 22:48 +0200 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/925e7b26b363 8077149: __noSuchProperty__ and __noSuchMethod__ invocations are not properly guarded Reviewed-by: jlaskey, mhaupt !

hg: jigsaw/jake/jdk: 37 new changesets

2016-09-15 Thread mandy . chung
Changeset: 7c15548ab9d6 Author:shurailine Date: 2016-09-06 17:07 -0400 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/7c15548ab9d6 8148859: Fix module dependences for java/time tests Reviewed-by: alanb, rriggs ! test/java/time/TEST.properties Changeset: 76ba1b74f268 Author:

hg: jigsaw/jake/corba: 2 new changesets

2016-09-15 Thread mandy . chung
Changeset: 258cf18fa7fc Author:lana Date: 2016-09-15 17:15 + URL: http://hg.openjdk.java.net/jigsaw/jake/corba/rev/258cf18fa7fc Added tag jdk-9+136 for changeset aa053a3faf26 ! .hgtags Changeset: cbdc0699e357 Author:mchung Date: 2016-09-15 13:57 -0700 URL:

hg: jigsaw/jake/hotspot: 2 new changesets

2016-09-15 Thread mandy . chung
Changeset: 6bddcf692e1d Author:lana Date: 2016-09-15 17:15 + URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/6bddcf692e1d Added tag jdk-9+136 for changeset a20da289f646 ! .hgtags Changeset: 89d191b6c6ee Author:mchung Date: 2016-09-15 13:57 -0700 URL:

hg: jigsaw/jake/jdk: Fixup test @modules to access private field

2016-09-15 Thread mandy . chung
Changeset: 570f2e8d2b6c Author:mchung Date: 2016-09-15 13:14 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/570f2e8d2b6c Fixup test @modules to access private field ! test/sun/security/krb5/tools/KtabZero.java

Re: #ServiceLoaderEnhancements

2016-09-15 Thread Alex Buckley
Best to wait for the spec. On 9/15/2016 12:34 PM, Peter Levart wrote: On 09/15/2016 05:51 PM, Alan Bateman wrote: On 15/09/2016 07:54, Peter Levart wrote: One thing that is not clear is whether the yyy in "provides xxx with yyy" directive of module declaration must be a concrete class and a

Re: #ServiceLoaderEnhancements

2016-09-15 Thread Peter Levart
On 09/15/2016 05:51 PM, Alan Bateman wrote: On 15/09/2016 07:54, Peter Levart wrote: One thing that is not clear is whether the yyy in "provides xxx with yyy" directive of module declaration must be a concrete class and a subtype of service type when the service is obtained via a static

hg: jigsaw/jake/hotspot: 8162998: Check at module definition that a package named java is not being defined to a class loader other than the boot or platform loader

2016-09-15 Thread lois . foltan
Changeset: de2b914558cd Author:lfoltan Date: 2016-09-15 11:25 -0400 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/de2b914558cd 8162998: Check at module definition that a package named java is not being defined to a class loader other than the boot or platform loader

hg: jigsaw/jake/jdk: 8162998: Check at module definition that a package named java is not being defined to a class loader other than the boot or platform loader

2016-09-15 Thread lois . foltan
Changeset: b25ebe289b7c Author:lfoltan Date: 2016-09-15 11:27 -0400 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/b25ebe289b7c 8162998: Check at module definition that a package named java is not being defined to a class loader other than the boot or platform loader Summary:

JDK 9 Early Access with Project Jigsaw, build 135 on 09-14-2016 (#5500)

2016-09-15 Thread Alan Bateman
The EA download [1] has been refreshed with new builds that are mostly aligned with the current proposals for JPMS issues [2]. Not everything is there yet of course (and some areas need a few more iterations) but there is enough to test and try things out. For

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

2016-09-15 Thread Mandy Chung
-—disable-plugin can allow any plugin if present, regardless whether it’s auto-enabled or not. If the plugin is disabled, -—disable-plugin will just be nop. Looking at Plugin.State::DISABLED: DISABLED: The plugin is not exposed in help and will be not called. It’s unclear this is needed.

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

2016-09-15 Thread Sundararajan Athijegannathan
With "--disable ", user can pass any plugin name - even the plugins that are *not* auto-enabled. We may have to check and report. But, --disable- options are added only for auto-enabled plugins. --disable- will automatically fail as "unsupported option". -Sundar On 9/15/2016 9:57 PM, Paul

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

2016-09-15 Thread Mandy Chung
> On Sep 15, 2016, at 8:26 AM, Alan Bateman wrote: > > On 15/09/2016 07:31, Sundararajan Athijegannathan wrote: > >> Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.01/ for >> https://bugs.openjdk.java.net/browse/JDK-8160063 >> >> * Adding --disable-

Re: #ServiceLoaderEnhancements

2016-09-15 Thread Alan Bateman
On 15/09/2016 07:54, Peter Levart wrote: One thing that is not clear is whether the yyy in "provides xxx with yyy" directive of module declaration must be a concrete class and a subtype of service type when the service is obtained via a static method. For example, is the following a valid

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

2016-09-15 Thread Alan Bateman
On 15/09/2016 07:31, Sundararajan Athijegannathan wrote: Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.01/ for https://bugs.openjdk.java.net/browse/JDK-8160063 * Adding --disable- option for any plugin that is auto-enabled. Should this be --disable-plugin rather than

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

2016-09-15 Thread Mandy Chung
> On Sep 15, 2016, at 7:31 AM, Sundararajan Athijegannathan > wrote: > > Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.01/ for > https://bugs.openjdk.java.net/browse/JDK-8160063 > > * Adding --disable- option for any plugin that is

Re: #ServiceLoaderEnhancements

2016-09-15 Thread Peter Levart
Hi, Reading number (2) of the proposal... the static provider() method (declared by "provider class") and returning a "provider object" sounds confusing when coupled with number (3) which proposes a ServiceLoader.Provider interface. Should Provider interface be renamed to ProviderFactory?

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

2016-09-15 Thread Jim Laskey (Oracle)
+1 > On Sep 15, 2016, at 11:31 AM, Sundararajan Athijegannathan > wrote: > > Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.01/ for > https://bugs.openjdk.java.net/browse/JDK-8160063 > > * Adding --disable- option for any plugin that

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

2016-09-15 Thread Claes Redestad
+1 I agree it's good with a standard way to disable plugins like this on a high level. This also solves an issue I was just about to file a bug for where the GenerateJLIClassesPlugin generates a few basic forms even when "disabled" with =none /Claes On 2016-09-15 16:31, Sundararajan

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

2016-09-15 Thread Sundararajan Athijegannathan
Also piggybacking clean-up in unusued code in JLinkTask.java (Section enum) -Sundar On 15/09/16, 8:01 PM, Sundararajan Athijegannathan wrote: Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.01/ for https://bugs.openjdk.java.net/browse/JDK-8160063 * Adding --disable- option

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

2016-09-15 Thread Sundararajan Athijegannathan
Please review http://cr.openjdk.java.net/~sundar/8160063/webrev.01/ for https://bugs.openjdk.java.net/browse/JDK-8160063 * Adding --disable- option for any plugin that is auto-enabled. * Removed ad-hoc "none" option of generate-jli-classes plugin. * Simple test to make sure