Re: RFR 8130302: jarsigner and keytool -providerClass needs be re-examined for modules

2016-02-16 Thread Weijun Wang
On 2/16/2016 22:54, Alan Bateman wrote: On 16/02/2016 14:44, Weijun Wang wrote: Please review the code change at http://cr.openjdk.java.net/~weijun/8130302/webrev.00/ I didn't abandon -providerClass and go all the way to -provideName because -providerClass has a sub-option -providerArg t

hg: jigsaw/jake/langtools: fill in missing implementations

2016-02-16 Thread jonathan . gibbons
Changeset: 7ee81202a226 Author:jjg Date: 2016-02-16 17:17 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/7ee81202a226 fill in missing implementations ! src/jdk.jdeps/share/classes/com/sun/tools/classfile/ClassWriter.java ! src/jdk.jdeps/share/classes/com/sun/tools/

hg: jigsaw/jake/langtools: 8149597: javadoc should use new doclet by default

2016-02-16 Thread jonathan . gibbons
Changeset: a2c2599ca449 Author:jjg Date: 2016-02-16 16:31 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/a2c2599ca449 8149597: javadoc should use new doclet by default ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/DocEnv.java ! src/jdk.javadoc/share/cla

hg: jigsaw/jake/jdk: 8149597: javadoc should use new doclet by default

2016-02-16 Thread jonathan . gibbons
Changeset: ebf7cc8f20c7 Author:ksrini Date: 2016-02-12 10:28 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/ebf7cc8f20c7 8149597: javadoc should use new doclet by default ! make/launcher/Launcher-jdk.javadoc.gmk

hg: jigsaw/jake: 8149597: javadoc should use new doclet by default

2016-02-16 Thread jonathan . gibbons
Changeset: 6f7fe35e932a Author:ksrini Date: 2016-02-12 10:28 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/6f7fe35e932a 8149597: javadoc should use new doclet by default ! common/autoconf/spec.gmk.in ! make/Javadoc.gmk

hg: jigsaw/jake/langtools: 8149033: ClassCastException in case of service implementation is nested static class

2016-02-16 Thread jonathan . gibbons
Changeset: f3d413211d69 Author:jjg Date: 2016-02-16 15:21 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/f3d413211d69 8149033: ClassCastException in case of service implementation is nested static class ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Mod

hg: jigsaw/jake/jdk: Formatting nits in jlink --list-plugins output

2016-02-16 Thread mandy . chung
Changeset: 199e89d721c8 Author:mchung Date: 2016-02-16 15:07 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/199e89d721c8 Formatting nits in jlink --list-plugins output ! src/jdk.jlink/share/classes/jdk/tools/jlink/resources/jlink.properties ! src/jdk.jlink/share/classes/

hg: jigsaw/jake/jdk: Minor formatting of jlink help

2016-02-16 Thread mandy . chung
Changeset: 96dbfff6196f Author:mchung Date: 2016-02-16 14:07 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/96dbfff6196f Minor formatting of jlink help ! src/jdk.jlink/share/classes/jdk/tools/jlink/resources/jlink.properties ! src/jdk.jlink/share/classes/jdk/tools/jlink/

hg: jigsaw/jake/jdk: Update ImageFileCreatorTest due to internal API change

2016-02-16 Thread mandy . chung
Changeset: 7d736133127e Author:mchung Date: 2016-02-16 08:45 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/7d736133127e Update ImageFileCreatorTest due to internal API change ! test/jdk/jigsaw/tools/jlink/ImageFileCreatorTest.java

hg: jigsaw/jake: Keep packaged modules in JDK image

2016-02-16 Thread mandy . chung
Changeset: 0a9fc49cde2e Author:mchung Date: 2016-02-16 08:32 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/0a9fc49cde2e Keep packaged modules in JDK image ! make/Images.gmk

Re: RFR 8130302: jarsigner and keytool -providerClass needs be re-examined for modules

2016-02-16 Thread Alan Bateman
On 16/02/2016 14:44, Weijun Wang wrote: Please review the code change at http://cr.openjdk.java.net/~weijun/8130302/webrev.00/ I didn't abandon -providerClass and go all the way to -provideName because -providerClass has a sub-option -providerArg that can be used to further configure the

RFR 8130302: jarsigner and keytool -providerClass needs be re-examined for modules

2016-02-16 Thread Weijun Wang
Please review the code change at http://cr.openjdk.java.net/~weijun/8130302/webrev.00/ I didn't abandon -providerClass and go all the way to -provideName because -providerClass has a sub-option -providerArg that can be used to further configure the provider. Also I think we still need to su

hg: jigsaw/jake/jdk: Adding jlink -o, -c options to use optimizer, compressor builtin plugins.

2016-02-16 Thread sundararajan . athijegannathan
Changeset: b025d08c1828 Author:sundar Date: 2016-02-16 19:53 +0530 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/b025d08c1828 Adding jlink -o, -c options to use optimizer, compressor builtin plugins. ! src/jdk.jlink/share/classes/jdk/tools/jlink/internal/TaskHelper.java ! src

Re: CFV: New Jigsaw Committer: Igor Ignatyev

2016-02-16 Thread dalibor topic
Vote: Yes. -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgerich

hg: jigsaw/jake/langtools: Add initial support for TargetPlatform to classfile API

2016-02-16 Thread alan . bateman
Changeset: 92b2b5ce18c3 Author:alanb Date: 2016-02-16 12:26 + URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/92b2b5ce18c3 Add initial support for TargetPlatform to classfile API ! src/jdk.jdeps/share/classes/com/sun/tools/classfile/Attribute.java ! src/jdk.jdeps/shar

hg: jigsaw/jake/jdk: 2 new changesets

2016-02-16 Thread alan . bateman
Changeset: cebe626ea814 Author:alanb Date: 2016-02-16 12:27 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/cebe626ea814 tools/pack200/ModuleAttributes.java failing ! test/tools/pack200/pack200-verifier/src/xmlkit/ClassReader.java Changeset: 6f336965e118 Author:alanb

Re: [9] RFR 8134487:updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-16 Thread Alan Bateman
On 16/02/2016 11:05, Tim Du wrote: Hi Alan: I tried @compile/module to fix this issue before, but it does not work ,Use CertStatusReqExtensionTests.java as example, the process as below: 1. Create java.base/sun/security/ssl folder in test/sun/security/ssl/StatusStapling. 2. Move the dependen

Re: [9] RFR 8134487:updated sun/security/ssl/StatusStapling/* to work with modules

2016-02-16 Thread Tim Du
Hi Alan: I tried @compile/module to fix this issue before, but it does not work ,Use CertStatusReqExtensionTests.java as example, the process as below: 1. Create java.base/sun/security/ssl folder in test/sun/security/ssl/StatusStapling. 2. Move the dependency class like TestCase.java TestUtil

hg: jigsaw/jake: Fixed merge mistake in jprt config

2016-02-16 Thread erik . joelsson
Changeset: b3736d835f24 Author:erikj Date: 2016-02-16 11:02 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/b3736d835f24 Fixed merge mistake in jprt config ! make/jprt.properties