hg: jigsaw/jake: Post-process step to create jmod files

2013-09-10 Thread mandy . chung
Changeset: 80abe3cb308c Author:mchung Date: 2013-09-10 11:25 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/80abe3cb308c Post-process step to create jmod files ! common/makefiles/Main.gmk

Re: RFR 8058117: Missing jdk.deploy.osx from modules.xml

2014-09-10 Thread Mandy Chung
On 9/10/2014 8:43 AM, Chris Hegarty wrote: While looking at the parsing of modules.xml, I came across a potential issue; jdk.deploy.osx is not listed in modules.xml. This was just an oversight in the original generation of modules.xml, as it will only appear on Mac. Looks okay. Thanks

hg: jigsaw/m2/langtools: Update jdeps to work with modular image

2014-09-24 Thread mandy . chung
Changeset: 11a11f331a42 Author:mchung Date: 2014-09-24 12:16 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/11a11f331a42 Update jdeps to work with modular image ! src/jdk.dev/share/classes/com/sun/tools/jdeps/ClassFileReader.java !

hg: jigsaw/m2: 20 new changesets

2014-09-27 Thread mandy . chung
Changeset: 34df46455488 Author:lancea Date: 2014-09-12 17:46 -0400 URL: http://hg.openjdk.java.net/jigsaw/m2/rev/34df46455488 8058366: Export sun.misc to java.sql Reviewed-by: mchung ! modules.xml Changeset: a3ee36412163 Author:ehelin Date: 2014-09-15 16:30 +0200 URL:

hg: jigsaw/m2/corba: 4 new changesets

2014-09-27 Thread mandy . chung
Changeset: cfdac5887952 Author:katleman Date: 2014-09-25 12:40 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/corba/rev/cfdac5887952 Added tag jdk9-b32 for changeset b5b139354630 ! .hgtags Changeset: 1683040fc3fa Author:mchung Date: 2014-09-27 00:40 -0700 URL:

hg: jigsaw/m2/jaxws: 6 new changesets

2014-09-27 Thread mandy . chung
Changeset: d35ad0854f68 Author:mkos Date: 2014-09-12 17:20 +0200 URL: http://hg.openjdk.java.net/jigsaw/m2/jaxws/rev/d35ad0854f68 8054548: JAX-WS tools need to updated to work with modular image Summary: Removing java reflection API to get JavaCompiler; using standard javax.tools

hg: jigsaw/m2/hotspot: 70 new changesets

2014-09-27 Thread mandy . chung
Changeset: 6818c5298fab Author:dholmes Date: 2014-09-02 21:27 -0400 URL: http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/6818c5298fab 8046210: Missing memory barrier when reading init_lock Reviewed-by: fparain, dcubed, mdoerr Contributed-by: Bill Pittore bill.pitt...@oracle.com !

hg: jigsaw/m2/jaxp: 2 new changesets

2014-09-27 Thread mandy . chung
Changeset: 46b360454dad Author:katleman Date: 2014-09-25 12:40 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/jaxp/rev/46b360454dad Added tag jdk9-b32 for changeset b940ca3d2c7e ! .hgtags Changeset: bf5a6fa4e8c2 Author:mchung Date: 2014-09-27 00:40 -0700 URL:

hg: jigsaw/m2/langtools: 15 new changesets

2014-09-27 Thread mandy . chung
Changeset: 3eb8614e39b3 Author:sogoel Date: 2014-09-12 17:05 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/3eb8614e39b3 8055080: Group 9d: golden files for tests in tools/javac dir Reviewed-by: jjg, mcimadamore, jlahoda ! test/tools/javac/Parens1.java +

hg: jigsaw/m2/nashorn: 19 new changesets

2014-09-27 Thread mandy . chung
Changeset: a20309596c42 Author:hannesw Date: 2014-09-12 11:00 +0200 URL: http://hg.openjdk.java.net/jigsaw/m2/nashorn/rev/a20309596c42 8057743: Single quotes must be escaped in message resource file Reviewed-by: attila, lagergren, sundar !

Review Request: JDK-8059083: Remove jdk.compact3 from modules.xml

2014-10-02 Thread Mandy Chung
jdk.compact3 was meant to be removed when we integrated JEP 201: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8059083/webrev.00/ Mandy

Re: RFR 8046002: Move Ucrypto to the open jdk repo

2014-10-16 Thread Mandy Chung
On 10/15/14 3:20 PM, Valerie Peng wrote: Hi, experts, Please review the update for module.xml for 8046002: Move Ucrypto to the open jdk repo http://cr.openjdk.java.net/~valeriep/8046002_top/webrev.00/ Looks okay to me too. Mandy

hg: jigsaw/m2/jdk: 8059315: KeyStore and keytool tests failing with KeyStoreException; ...

2014-10-17 Thread mandy . chung
Changeset: d359a9bb2d3a Author:mchung Date: 2014-10-17 12:16 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/d359a9bb2d3a 8059315: KeyStore and keytool tests failing with KeyStoreException 8059316: closed/com/sun/crypto/provider/KeyStore/BackwardCompatibility.java failing

hg: jigsaw/m2/langtools: Remove jdeps legacy image support

2014-10-20 Thread mandy . chung
Changeset: 604f4be5fcea Author:mchung Date: 2014-10-20 19:24 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/604f4be5fcea Remove jdeps legacy image support ! src/jdk.dev/share/classes/com/sun/tools/jdeps/Archive.java !

hg: jigsaw/m2/jdk: jfxrt.jar moved from lib/ext to lib in the modular image

2014-10-21 Thread mandy . chung
Changeset: ca389ce52206 Author:mchung Date: 2014-10-21 11:53 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/ca389ce52206 jfxrt.jar moved from lib/ext to lib in the modular image ! src/java.base/share/classes/sun/misc/Launcher.java

hg: jigsaw/m2/langtools: jfxrt.jar moved from lib/ext to lib in the modular image

2014-10-21 Thread mandy . chung
Changeset: 8dd333dfc21c Author:mchung Date: 2014-10-21 11:53 -0700 URL: http://hg.openjdk.java.net/jigsaw/m2/langtools/rev/8dd333dfc21c jfxrt.jar moved from lib/ext to lib in the modular image ! src/jdk.compiler/share/classes/com/sun/tools/javac/file/Locations.java

Re: CFV: New Jigsaw Committer: Sundar Athijegannathan

2014-11-03 Thread Mandy Chung
Vote: yes On 11/3/14 6:02 AM, Alan Bateman wrote: I hereby nominate Sundar Athijegannathan to Jigsaw Committer. Sundar is currently a Reviewer on the jdk9, jdk8u and Nashorn projects. Sundar has been contributing to Project Jigsaw by way of the compression support in jimage container format

hg: jigsaw/m2/hotspot: Replace PrintExtDirsWarning with -XX:+CheckEndorsedAndExtDirs flag

2014-11-10 Thread mandy . chung
Changeset: 9531206c79a6 Author:mchung Date: 2014-11-10 12:05 -0800 URL: http://hg.openjdk.java.net/jigsaw/m2/hotspot/rev/9531206c79a6 Replace PrintExtDirsWarning with -XX:+CheckEndorsedAndExtDirs flag ! src/share/vm/runtime/arguments.cpp ! src/share/vm/runtime/globals.hpp

hg: jigsaw/m2/jdk: 8064782: Change jdeps to use jrt:/ instead of jimagefs

2014-11-17 Thread mandy . chung
Changeset: 8cb493f1bb25 Author:mchung Date: 2014-11-17 19:18 -0800 URL: http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/8cb493f1bb25 8064782: Change jdeps to use jrt:/ instead of jimagefs ! make/src/classes/build/tools/module/ModulesXmlReader.java !

hg: jigsaw/m2: ucrypto moved to the open

2014-11-17 Thread mandy . chung
Changeset: 7ec2be25e81e Author:mchung Date: 2014-11-17 19:52 -0800 URL: http://hg.openjdk.java.net/jigsaw/m2/rev/7ec2be25e81e ucrypto moved to the open ! make/Images.gmk

hg: jigsaw/m2/jdk: Remove @Deprecated from EXTENSION_NAME and EXTENSION_LIST

2014-11-26 Thread mandy . chung
Changeset: 3e198fd8bac3 Author:mchung Date: 2014-11-26 08:32 -0800 URL: http://hg.openjdk.java.net/jigsaw/m2/jdk/rev/3e198fd8bac3 Remove @Deprecated from EXTENSION_NAME and EXTENSION_LIST ! src/java.base/share/classes/java/util/jar/Attributes.java

Re: Review request 8069551: Move java.security.acl from compact3 to java.base

2015-01-30 Thread Mandy Chung
On 1/30/15 12:50 AM, Alan Bateman wrote: On 29/01/2015 20:58, Mandy Chung wrote: Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8069551/webrev.00 java.security.acl is superceded by java.security package since 1.2. This patch proposes to move java.security.acl package to java.base

Review request: JDK-8069414: Rename oracle.accessbridge to jdk.accessbridge

2015-01-22 Thread Mandy Chung
diff --git a/modules.xml b/modules.xml --- a/modules.xml +++ b/modules.xml @@ -737,7 +737,7 @@ /export export namesun.awt/name - tooracle.accessbridge/to + tojdk.accessbridge/to /export /module module Mandy

Re: Apache Maven JDeps Plugin

2015-02-17 Thread Mandy Chung
Hi Robert, Indeed this looks very useful. On 2/16/2015 10:45 AM, Alan Bateman wrote: On 16/02/2015 18:28, Robert Scholte wrote: Hi Alan, if you are referring to the -R / -recursive option of the jdeps tool, then yes you can. See

Re: [9] Review Request: 8072663: Remove the sun.security.acl package which is not used in the JDK

2015-02-18 Thread Mandy Chung
On 2/17/15 12:59 PM, Jason Uh wrote: Please review this fix, which removes the sun.security.acl package. webrev: http://cr.openjdk.java.net/~juh/8072663/00/ jbs: http://bugs.openjdk.java.net/browse/JDK-8072663 Looks fine. While the java.security.acl package is small, it'd be nice if this

Re: Apache Maven JDeps Plugin

2015-02-19 Thread Mandy Chung
can exclude a subset of dependenciesToAnalyzeIncludes. dependenciesToAnalyzeExcludes excludeorg.foo.test:*/exclude /dependenciesToAnalyzeExcludes This should match your requirements. Regards, Robert Scholte Op Wed, 18 Feb 2015 05:46:37 +0100 schreef Mandy Chung mandy.ch

Re: 8067867: Subsume module java.xml.soap into module java.xml.ws

2015-01-05 Thread Mandy Chung
On 1/3/15 1:24 AM, Alan Bateman wrote: There are a number of proposed updates to the module graph that we need reflected in the source code layout (and the module graph in JEP 200 [1] will need to be refreshed too). JDK-8067867 tracks dropping the module java.xml.soap and subsuming the API

Re: Policy providers in Java 9

2015-03-21 Thread Mandy Chung
On 3/21/2015 5:12 AM, Peter Firmstone wrote: Just wondering, with the removal of the extensions directory, what's the correct way of specifying a policy provider? We currently have a number of nested policy providers that are loaded by the extension classloader. Do you set the custom

Re: Review Request: 8074428, 8074429, 8074430 jdk.pack200, jdk.jartool, jdk.policytool modules

2015-03-13 Thread Mandy Chung
jdk.unpack200 and not jdk.pack200 ? since it contains only the unpacker, and the bin utilities are pack200 and unpack200. Kumar On 3/4/2015 5:13 PM, Mandy Chung wrote: As listed in an open issue in JEP 200: The jdk.dev and jdk.runtime modules contain miscellaneous tools that do not obviously belong to any

Re: Review Request: 8074428, 8074429, 8074430 jdk.pack200, jdk.jartool, jdk.policytool modules

2015-03-04 Thread Mandy Chung
it into jdk.security.tools and jdk.security.util. Put the executables into tools and APIs into util. --Max On Mar 5, 2015, at 09:13, Mandy Chung mandy.ch...@oracle.com wrote: As listed in an open issue in JEP 200: The jdk.dev and jdk.runtime modules contain miscellaneous tools that do not obviously belong

Review Request: 8074428, 8074429, 8074430 jdk.pack200, jdk.jartool, jdk.policytool modules

2015-03-04 Thread Mandy Chung
As listed in an open issue in JEP 200: The jdk.dev and jdk.runtime modules contain miscellaneous tools that do not obviously belong to any other module; these modules will eventually be either renamed or refactored. Currently there are jdk.javadoc, jdk.jconsole, jdk.jcmd modules in the JDK that

Re: Review Request: 8074428, 8074429, 8074430 jdk.pack200, jdk.jartool, jdk.policytool modules

2015-03-05 Thread Mandy Chung
Max, Since the new APIs you're working on are still in design phase, I think it's a bit early to discuss where these new APIs should be in. Just one thing to say about the new JarSigner API from your webrev. com.sun.jarsigner is an existing exported package that you should consider

Review request 8069551: Move java.security.acl from compact3 to java.base

2015-01-29 Thread Mandy Chung
Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8069551/webrev.00 java.security.acl is superceded by java.security package since 1.2. This patch proposes to move java.security.acl package to java.base module rather than complicating the module graph with a close-to-useless

Re: OS X 1.9 ea jdeps

2015-04-29 Thread Mandy Chung
On 4/13/15 1:32 AM, Michael Hall wrote: On Apr 13, 2015, at 2:32 AM, Alan Bateman alan.bate...@oracle.com wrote: On 12/04/2015 17:57, Michael Hall wrote: : Not exactly sure what you mean. But, which java /usr/bin/java ... I think the issue is that the installation creates sym links in

Re: jdeps command

2015-04-30 Thread Mandy Chung
On 04/30/2015 03:02 PM, Michael Hall wrote: Please file a bug at http://bugreporter.apple.com asking for this command to be added to /usr/bin, and send me the bug ID. We can ensure that these stubs are in the base OS without any Apple Java being installed. Thanks, Mike Swingler Apple Inc.

Re: jdeps command

2015-04-30 Thread Mandy Chung
On 04/30/2015 04:17 PM, Michael Hall wrote: On Apr 30, 2015, at 5:39 PM, Mandy Chung mandy.ch...@oracle.com mailto:mandy.ch...@oracle.com wrote: and existing tool may be removed (e.g. jhat [1]). Not through Java 9 ea? jhat -version jhat version 2.0 (java version 1.9.0-ea) So I would

Re: internal API usage: sun.awt.CausedFocusEvent

2015-05-13 Thread Mandy Chung
On May 13, 2015, at 10:51 AM, Alan Snyder javali...@cbfiddle.com wrote: I will do that, but a larger question remains: Are any of the AWT classes currently reported by jdeps as “JDK internal API” going to be accessible in JDK 9? No - there is no known RFE that I know of may provide a

Re: RFR 8042901: Allow com.sun.management to be in a different module to java.lang.management

2015-04-01 Thread Mandy Chung
On 3/31/2015 9:39 AM, shanliang wrote: Please review this fix: Bug: https://bugs.openjdk.java.net/browse/JDK-8042901 Webrev: http://cr.openjdk.java.net/~sjiang/JDK-8042901/00/ Thank you for doing this big change to separate com.sun.management API from java.management module. Looking really

Re: RFR: JDK-8080511 - Refresh of jimage support

2015-06-18 Thread Mandy Chung
http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-top modules.xml looks fine. test/lib/sun/hotspot/WhiteBox.java This adds the native entry points. Where are they implemented? It’s not obvious to me. http://cr.openjdk.java.net/~jlaskey/hs-rt-jimage/webrev-langtools

Re: RFR: JDK-8080511 - Refresh of jimage support

2015-06-19 Thread Mandy Chung
On Jun 19, 2015, at 6:27 AM, Alan Bateman alan.bate...@oracle.com wrote: On 18/06/2015 19:48, Mandy Chung wrote: : test/java/nio/Buffer/LimitDirectMemory.sh test/java/nio/file/spi/SetDefaultProvider.java Why this test is ignored? For LimitDirectMemory.sh then the test needs

Re: Review Request 8074432: Move jdeps to jdk.compiler module

2015-05-27 Thread Mandy Chung
='{ -J-ms8m$(COMMA) com.sun.tools.jdeps.Main$(COMMA) }')) On May 27, 2015, at 12:48 AM, Erik Joelsson erik.joels...@oracle.com wrote: Hello Mandy, I don't see a Launcher-jdk.jdeps.gmk. Should there be one? /Erik On 2015-05-27 08:00, Mandy Chung wrote: We discussed this offline and revised

Re: Review Request 8074432: Move jdeps to jdk.compiler module

2015-05-27 Thread Mandy Chung
We discussed this offline and revised the proposal to group javap and classfile library together with jdeps in jdk.jdeps module for static analysis tools to live. The jdk.compiler module will contain the compiler and a couple of small tools. Moving out javap and classfile library save about

Re: Jigsaw Hackday in London - Anything in particular you want us to look at?

2015-06-29 Thread Mandy Chung
FYI. The RFE concerning sun.misc.SignalHandler is: https://bugs.openjdk.java.net/browse/JDK-8087286 Mandy On Jun 29, 2015, at 5:43 AM, Richard Warburton richard.warbur...@gmail.com wrote: Hi, This is a good point actually. I have also used that class before as it seems to be the

Re: Review Request 8074432: Move jdeps to jdk.compiler module

2015-05-25 Thread Mandy Chung
Meant to include jigsaw-dev in the initial post. On May 25, 2015, at 12:16 AM, Alan Bateman alan.bate...@oracle.com wrote: On 24/05/2015 05:00, Mandy Chung wrote: This will move jdeps to the same module as other langtools javac and javap. Webrev at: http://cr.openjdk.java.net/~mchung

Re: Review for JDK-8132527

2015-08-18 Thread Mandy Chung
On 08/18/2015 01:34 AM, Jean-Francois Denise wrote: Hi, asking for review to fix jvm crashing when replacing jimage file with recreated one. The name of the module metadata resource was wrongly associated to the name of the recreated jimage file. The original name must be reused in the

hg: jigsaw/jake/langtools: Add jdeps -R -ct option.

2015-10-21 Thread mandy . chung
Changeset: f99c96f9f9f9 Author:mchung Date: 2015-10-21 22:29 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/f99c96f9f9f9 Add jdeps -R -ct option. Also remove -verify:access option. ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/Analyzer.java !

hg: jigsaw/jake/jdk: 8138626: java.awt.Window.applyResourceBundle fails to find resource bundle

2015-10-25 Thread mandy . chung
Changeset: 22839e63cbda Author:okutsu Date: 2015-10-25 19:56 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/22839e63cbda 8138626: java.awt.Window.applyResourceBundle fails to find resource bundle Reviewed-by: mchung ! src/java.desktop/share/classes/java/awt/Window.java

Re: RFR: 8140336: Add @modules for exported dependencies to jdk_core tests

2015-10-26 Thread Mandy Chung
> On Oct 26, 2015, at 10:13 AM, Alexandre (Shura) Iline > wrote: > > Hi. > > Could you please take a look on the suggested change in tests belonging to > jdk_core test group. > > Pls notice that the dependencies which would later be removed by fixing >

hg: jigsaw/jake/jdk: Jlink plugin for fast reconstitution of module descriptors of installed modules

2015-10-26 Thread mandy . chung
Changeset: 06bc945b812d Author:mchung Date: 2015-10-26 17:56 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/06bc945b812d Jlink plugin for fast reconstitution of module descriptors of installed modules To enable this plugin, use jlink --gen-installed-modules on option. -

Re: RFR 9: 8139390 : Very long classname in jimage causes SIGSEGV

2015-10-29 Thread Mandy Chung
Resource file of zero-length module name and package name should not be allowed to be written to jimage. The moduleName > 0 and packageName > 0 should be an assertion; exception if fails. Mandy > On Oct 29, 2015, at 1:09 PM, Dmitry Samersoff > wrote: > > Roger,

Re: RFR 9: 8139390 : Very long classname in jimage causes SIGSEGV

2015-10-29 Thread Mandy Chung
> On Oct 27, 2015, at 11:40 PM, Roger Riggs wrote: > > Please review an update to the jimage reader implementation to correct the > case where a class name is very long causing a SEGV due to buffer overruns. > > The fix will be pushed to the hs-comp repo; the bug was

Re: RFR 9: 8139390 : Very long classname in jimage causes SIGSEGV

2015-10-29 Thread Mandy Chung
> On Oct 29, 2015, at 4:22 PM, Roger Riggs <roger.ri...@oracle.com> wrote: > > Hi Mandy, > > Collapsing the emails. > > As for asserting that module name is never null; > it appears that the module name is not being supplied by the VM. (in hs-comp > or jdk9).

hg: jigsaw/jake/jdk: Jake patch for 8140336: Add @modules for exported dependencies to jdk_core tests

2015-10-27 Thread mandy . chung
Changeset: 1b632228f7e5 Author:shurailine Date: 2015-10-27 20:12 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/1b632228f7e5 Jake patch for 8140336: Add @modules for exported dependencies to jdk_core tests Reviewed-by: alanb, mchung !

Re: RFR: 8140336: Add @modules for exported dependencies to jdk_core tests

2015-10-27 Thread Mandy Chung
> On Oct 27, 2015, at 6:36 AM, Alexandre (Shura) Iline > wrote: > > I have created two separate reviews: > http://cr.openjdk.java.net/~shurailine/8140336/webrev.jdk9.02/ > http://cr.openjdk.java.net/~shurailine/8140336/webrev.jake.02/ Thanks. I have pushed

Re: Technical question about the Java Dependency Analysis Tool

2015-11-10 Thread Mandy Chung
jdeps already has the option finding the JAR files needed to compile A.jar. There is a new jdeps option added to Jigsaw EA build [1] to make this easier: $ jdeps -s -ct -R -cp lib/* A.jar This will transitively find all dependencies from the references in A.jar and shows the summary of the

Re: RFR: 8139430

2015-11-10 Thread Mandy Chung
> On Nov 10, 2015, at 10:11 AM, Alexandre (Shura) Iline > <alexandre.il...@oracle.com> wrote: > >> >> On Nov 10, 2015, at 9:04 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: >> >> Hi Shura, >> >> Thanks for doing it and it’s good

Re: RFR: 8139430

2015-11-10 Thread Mandy Chung
Hi Shura, Thanks for doing it and it’s good to see the unnecessary dependency to java.management eliminated. The new jdk.testlibrary.management package name is fine. It’s okay to keep the class name InputArguments as Jaroslav suggests and it’s easier to tell what this class is about. There

Re: Proposal to enhance jdeps tool to search split packages/add some kind of general API

2015-11-12 Thread Mandy Chung
> On Nov 12, 2015, at 2:39 AM, Patrick Reinhart wrote: > > Hi there, > > I like to do some early adoption tests with software using the new module > system. Now we got an unknown amount of split packages in our system, the we > will need to fix in order to migrate our

hg: jigsaw/jake/jdk: Backout compressed resource header length change that breaks windows build

2015-11-12 Thread mandy . chung
Changeset: 103e023995ba Author:mchung Date: 2015-11-12 18:51 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/103e023995ba Backout compressed resource header length change that breaks windows build !

hg: jigsaw/jake/jdk: 161 new changesets

2015-11-13 Thread mandy . chung
Changeset: 0198481aa9bd Author:lana Date: 2015-10-19 00:25 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/0198481aa9bd Added tag jdk9-b87 for changeset 110fc90bdfa0 ! .hgtags Changeset: e860e54043fd Author:aefimov Date: 2015-10-10 12:52 +0300 URL:

hg: jigsaw/jake/nashorn: 43 new changesets

2015-11-13 Thread mandy . chung
Changeset: 061682b25ca9 Author:lana Date: 2015-10-19 00:25 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/061682b25ca9 Added tag jdk9-b87 for changeset 0bf2fe0c7b32 ! .hgtags Changeset: 0cae16c0043d Author:attila Date: 2015-10-12 10:27 +0200 URL:

hg: jigsaw/jake: 65 new changesets

2015-11-13 Thread mandy . chung
Changeset: 0c140bff9257 Author:lana Date: 2015-10-19 00:24 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/0c140bff9257 Added tag jdk9-b87 for changeset fd4f4f756107 ! .hgtags Changeset: a7ec2278c13d Author:ihse Date: 2015-10-12 11:49 +0200 URL:

hg: jigsaw/jake/jdk: Add new modular resource bundle test

2015-11-13 Thread mandy . chung
Changeset: f83388bef364 Author:okutsu Date: 2015-11-13 22:20 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/f83388bef364 Add new modular resource bundle test + test/java/util/ResourceBundle/modules/visibility/src/embargo/jdk/embargo/TestWithNoModuleArg.java +

Re: JDeps: detecting offending packages

2015-11-14 Thread Mandy Chung
> On Nov 14, 2015, at 7:20 AM, Alan Bateman wrote: > > On 13/11/2015 21:58, Robert Scholte wrote: >> : >> >> Previous versions of JDK9 gave me the following output: >> >> classes -> java.base >>(classes) >> -> java.io >> -> java.lang >> -> sun.misc

hg: jigsaw/jake/langtools: 39 new changesets

2015-11-16 Thread mandy . chung
Changeset: 288f18dd9157 Author:lana Date: 2015-10-19 00:25 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/288f18dd9157 Added tag jdk9-b87 for changeset 45f796d8cdcd ! .hgtags Changeset: 79e637c1e083 Author:mcimadamore Date: 2015-10-12 12:24 +0100 URL:

hg: jigsaw/jake/hotspot: 226 new changesets

2015-11-16 Thread mandy . chung
Changeset: bc48b669bc66 Author:lana Date: 2015-10-19 00:24 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/bc48b669bc66 Added tag jdk9-b87 for changeset d7ffd16382fe ! .hgtags Changeset: b0e0a53226fd Author:lana Date: 2015-10-22 08:47 -0700 URL:

hg: jigsaw/jake/jaxp: 18 new changesets

2015-11-16 Thread mandy . chung
Changeset: 4700fd67e942 Author:lana Date: 2015-10-19 00:25 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/4700fd67e942 Added tag jdk9-b87 for changeset eb435c878c2c ! .hgtags Changeset: 27b625ce80f4 Author:lana Date: 2015-10-22 08:47 -0700 URL:

hg: jigsaw/jake/jdk: 3 new changesets

2015-11-16 Thread mandy . chung
Changeset: 5babde7bd078 Author:jjg Date: 2015-11-13 15:55 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/5babde7bd078 8142996: move jdk java/util/streams tests into java.base directories Reviewed-by: mchung ! test/java/util/stream/bootlib/TEST.properties +

hg: jigsaw/jake/hotspot: Add uses/provides for jdk.vm.ci module

2015-11-16 Thread mandy . chung
Changeset: 40952d9e4128 Author:mchung Date: 2015-11-16 22:11 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/40952d9e4128 Add uses/provides for jdk.vm.ci module ! src/jdk.vm.ci/share/classes/module-info.java

Re: RFR: 8139430

2015-11-17 Thread Mandy Chung
> On Nov 16, 2015, at 5:38 AM, Alan Bateman wrote: > > > > On 16/11/2015 13:01, Alexandre (Shura) Iline wrote: >> V6: >> http://cr.openjdk.java.net/~shurailine/8139430/webrev.06/ >> >> > This looks okay to me. For completeness then I assume the >

Re: RFR: 8139430

2015-11-10 Thread Mandy Chung
> On Nov 10, 2015, at 12:42 PM, Alan Bateman wrote: > > Would it break many tests if getProcessId were changed to return long to > match ProcessHandle::getPid? There are not small number of tests doing Integer.toString(ProcessTools.getProcessId())); Most of them

hg: jigsaw/jake/jdk: 8139067: Re-examine if a consumer module should load bundles from unnamed module

2015-10-29 Thread mandy . chung
Changeset: 1ac78e4a10d5 Author:okutsu Date: 2015-10-29 19:55 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/1ac78e4a10d5 8139067: Re-examine if a consumer module should load bundles from unnamed module Reviewed-by: alanb, mchung, naoto !

hg: jigsaw/jake/jdk: Enable jlink gen-installed-modules plugin by default

2015-11-07 Thread mandy . chung
Changeset: 3f859f4a29bf Author:mchung Date: 2015-11-07 13:02 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/3f859f4a29bf Enable jlink gen-installed-modules plugin by default !

Re: JOSM feedback on Java 7,8,9, including Jigsaw EA

2015-10-30 Thread Mandy Chung
Thanks for reporting these issues. > On Oct 30, 2015, at 8:29 AM, Vincent Privat > wrote: > > Concerning Jigsaw: > - We have reported 3 bugs. All made it to the public JIRA: 8138878, > 8140477, 8140481. The second one is a bit problematic for our tests as it >

Re: RFR: 8139067: Re-examine if a consumer module should load bundles from unnamed module

2015-10-29 Thread Mandy Chung
> On Oct 29, 2015, at 4:55 PM, Masayoshi Okutsu > wrote: > > Hello, > > Please review the change for JDK-8139067. A consumer module > (ResourceBundle.getBundle caller) will no longer load resource bundles from > the class-path if the consumer module is a named

hg: jigsaw/jake/langtools: Should not generate module-info.java if it contains an unnamed package

2015-10-21 Thread mandy . chung
Changeset: ce2237f87c34 Author:mchung Date: 2015-10-20 23:47 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ce2237f87c34 Should not generate module-info.java if it contains an unnamed package ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/ModuleInfoBuilder.java

hg: jigsaw/jake/langtools: Initial support for jdeps -genmoduleinfo option

2015-10-21 Thread mandy . chung
Changeset: 9504efc77849 Author:mchung Date: 2015-10-20 23:13 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/9504efc77849 Initial support for jdeps -genmoduleinfo option ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/Analyzer.java +

Re: RFR 7199353: Allow ConstructorProperties annotation from any package

2015-10-08 Thread Mandy Chung
On 10/08/2015 05:41 AM, Alan Bateman wrote: I'm not sure that I agree with logging a warning when java.beans.ConstructorProperties is used. I would be tempted to leave that out. The idea is that we want users to stop using @j.b.CP for JMX purposes. So we might as well warn them about the

Re: RFR 7199353: Allow ConstructorProperties annotation from any package

2015-10-08 Thread Mandy Chung
On 10/08/2015 04:49 AM, Jaroslav Bachorik wrote: Please, review the following change Issue : https://bugs.openjdk.java.net/browse/JDK-7199353 Webrev: http://cr.openjdk.java.net/~jbachorik/7199353/webrev.00/top http://cr.openjdk.java.net/~jbachorik/7199353/webrev.00/jdk I think #2 in the

Re: jlink plugin experiment to improve jake startup time

2015-10-06 Thread Mandy Chung
I realize I missed the attachment after seeing Claes’ question. This includes the heap usage after GC. > On Oct 5, 2015, at 2:13 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: > > I have been experimenting a jlink plugin to improve the module system startup > time

hg: jigsaw/jake/jdk: Fix InternalError thrown by Class.forName

2015-10-12 Thread mandy . chung
Changeset: f7e100339605 Author:mchung Date: 2015-10-12 20:22 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/f7e100339605 Fix InternalError thrown by Class.forName ! src/java.base/share/classes/java/lang/Class.java

hg: jigsaw/jake/jdk: 2 new changesets

2015-11-17 Thread mandy . chung
Changeset: e5fcea3aa679 Author:mchung Date: 2015-11-17 15:12 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/e5fcea3aa679 Enable the test case to verify unnamed module fails to load properties bundle from named module !

Re: jdeps and annotations parameters

2015-09-01 Thread Mandy Chung
ed nested classes as dependencies (even though they are in the API of the class). 3) Should annotations of APIs be considered APIs? (at least for some trivial cases they are) -Jeff *De :* Jeff Hain <jeffh...@rock

Re: jdeps and annotations parameters

2015-09-03 Thread Mandy Chung
Hi Jeff, Thanks for the feedback that is helpful. jdeps is static analysis tool reporting class dependencies to help existing code for modularization. So jdeps should report the list of class names matching the list that are needed during compilation as closely as possible. It looks at

Re: Project Jigsaw: Early-Access Builds available on jdk9.java.net/jigsaw

2015-09-10 Thread Mandy Chung
On 09/10/2015 01:55 PM, Jim Connors wrote: When you run the com.greetings.Main class via: $ java -mp mods -m com.greetings/com.greetings.Main The Quick Start guide shows the output as: class org.fastsocket.FastNetworkSocket The Main.java file referenced above prints nothing (at

hg: jigsaw/jake/langtools: 2 new changesets

2015-09-12 Thread mandy . chung
Changeset: 144597c5a68b Author:jlahoda Date: 2015-09-12 10:20 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/144597c5a68b Fixing langtools tests broken by recent merge. ! test/ProblemList.jake.txt ! test/tools/javac/failover/CheckAttributedTree.java !

Re: CFV: New Jigsaw Committer: Harold Seigel

2015-09-11 Thread Mandy Chung
Vote: yes Mandy

hg: jigsaw/jake/jaxp: 2 new changesets

2015-09-11 Thread mandy . chung
Changeset: 53fe3c103b6f Author:lana Date: 2015-09-11 10:26 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/53fe3c103b6f Added tag jdk9-b81 for changeset 6a418934997f ! .hgtags Changeset: faade39222ae Author:mchung Date: 2015-09-11 17:09 -0700 URL:

Re: CFV: New Jigsaw Committer: Lois Foltan

2015-09-16 Thread Mandy Chung
Vote: yes

Re: CFV: New Committer: Christian Tornqvist

2015-09-16 Thread Mandy Chung
Vote: yes

hg: jigsaw/jake/jdk: Exception typo in Proxy class javadoc

2015-09-17 Thread mandy . chung
Changeset: 3f905b5efebf Author:mchung Date: 2015-09-17 15:14 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/3f905b5efebf Exception typo in Proxy class javadoc ! src/java.base/share/classes/java/lang/reflect/Proxy.java

hg: jigsaw/jake/hotspot: fixing XoverrideCCDS.java test failure

2015-09-17 Thread mandy . chung
Changeset: e6b7e9cc28c0 Author:ccheung Date: 2015-09-17 14:39 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/e6b7e9cc28c0 fixing XoverrideCCDS.java test failure Summary: adjusted the expected error message during dump time Reviewed-by: iklam, jiangli !

hg: jigsaw/jake/jdk: Class::getPackage should return null for array type, primitive type or void

2015-10-02 Thread mandy . chung
Changeset: 375469dd1809 Author:mchung Date: 2015-10-02 08:24 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/375469dd1809 Class::getPackage should return null for array type, primitive type or void ! src/java.base/share/classes/java/lang/Class.java !

hg: jigsaw/jake/jdk: jdk.hprof.agent has been removed and module loader map needs update

2015-10-02 Thread mandy . chung
Changeset: 937f4cfe2cab Author:mchung Date: 2015-10-02 16:35 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/937f4cfe2cab jdk.hprof.agent has been removed and module loader map needs update ! make/gensrc/GensrcModuleLoaderMap.gmk

Re: Re-examine ClassLoader.getPackage(s) methods

2015-09-26 Thread Mandy Chung
> On Sep 26, 2015, at 7:33 AM, Alan Bateman wrote: > This looks good. I think I would lean more to deprecating getPackage as just > too fragile and leads to difficult to diagnose bugs. Are you also suggesting to deprecate ClassLoader::getPackages as well? > BTW: I

hg: jigsaw/jake/jaxws: Update jaxws to look up resources from named modules

2015-09-29 Thread mandy . chung
Changeset: 49c4220a2a43 Author:mkos Date: 2015-09-29 11:03 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/49c4220a2a43 Update jaxws to look up resources from named modules ! src/java.xml.ws/share/classes/com/sun/xml/internal/ws/api/server/SDDocumentSource.java !

hg: jigsaw/jake/jdk: 2 new changesets

2015-09-28 Thread mandy . chung
Changeset: 5f781c90d35b Author:mchung Date: 2015-09-28 16:31 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/5f781c90d35b New test to create proxies with layer + test/jdk/jigsaw/reflect/Proxy/ProxyLayerTest.java Changeset: 189f83e533d0 Author:mchung Date:

hg: jigsaw/jake/jdk: Deprecate Package::getPackage as ClassLoader::getPackage does

2015-09-29 Thread mandy . chung
Changeset: deb77851e5fd Author:mchung Date: 2015-09-29 12:36 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/deb77851e5fd Deprecate Package::getPackage as ClassLoader::getPackage does ! src/java.base/share/classes/java/lang/Package.java !

Re: Re-examine ClassLoader.getPackage(s) methods

2015-09-28 Thread Mandy Chung
On 09/27/2015 01:29 AM, Alan Bateman wrote: On 26/09/2015 19:28, Mandy Chung wrote: On Sep 26, 2015, at 7:33 AM, Alan Bateman <alan.bate...@oracle.com> wrote: This looks good. I think I would lean more to deprecating getPackage as just too fragile and leads to difficult to diagnos

hg: jigsaw/jake/jdk: Update test/java/util/ResourceBundle/Bug6299235Test.sh to use -Xpatch

2015-10-05 Thread mandy . chung
Changeset: 9a3cd31c64df Author:mchung Date: 2015-10-05 18:03 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/9a3cd31c64df Update test/java/util/ResourceBundle/Bug6299235Test.sh to use -Xpatch ! test/ProblemList.jake.txt ! test/java/util/ResourceBundle/Bug6299235Test.sh

  1   2   3   4   5   6   7   8   9   10   >