Hi Max/Mandy,
Here is the updated webrev:
http://cr.openjdk.java.net/~asmotrak/siba/8130360/webrev.05/
I tried to address all the comments bellow. The major change includes
abstracting the common behavior to ModularTest.java and extend the rest by each
specific test. i.e. SecurityProviderModul
Changeset: 37fefa14f3a7
Author:jjg
Date: 2015-12-21 19:03 -0800
URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/37fefa14f3a7
8145957: langtools tests failing on exploded image
! test/tools/javac/6589361/T6589361.java
! test/tools/lib/ToolBox.java
Changeset: 8c8e4774ff86
Author:jjg
Date: 2015-12-21 15:21 -0800
URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/8c8e4774ff86
Remove support for obsolete -Xoverride option
! src/jdk.compiler/share/classes/com/sun/tools/javac/file/Locations.java
! src/jdk.compiler/share/cla
Changeset: b5b8222bc934
Author:jlaskey
Date: 2015-12-21 11:47 -0400
URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/b5b8222bc934
Remove the use of UTF8String and use fast Modified-UTF8 conversion.
! src/java.base/share/classes/jdk/internal/jimage/BasicImageReader.java
! src/jav
Changeset: bb7babadf6ea
Author:jfdenise
Date: 2015-12-21 14:42 +0100
URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/bb7babadf6ea
jlink API rework, hide compression, merge File and Resource, ModuleData per
Module.
- src/java.base/share/classes/jdk/internal/jimage/StringTable.j
Changeset: 890589990d39
Author:sundar
Date: 2015-12-21 16:37 +0530
URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/890589990d39
8145880: Two tests involving Dynalink pluggable linker loading via service
loader fail
! src/jdk.dynalink/share/classes/jdk/dynalink/DynamicLinke
On 21/12/2015 08:58, Martin Balin wrote:
Thanks' for explanation.
Would not it make a sense to extend java.util.jar.Attributes.Name
class to support ConcealedPackages new optional attribute?
This is a class file attribute rather than a JAR file attribute. We have
bug JDK-8145775 tracking this
Thanks' for explanation.
Would not it make a sense to extend java.util.jar.Attributes.Name class
to support ConcealedPackages new optional attribute?
Martin
On 16.12.2015 11:19, Alan Bateman wrote:
On 16/12/2015 09:27, Martin Balin wrote:
I've encountered this problem with jlink as well. The
Changeset: 40b41f4e0d4b
Author:sundar
Date: 2015-12-21 14:06 +0530
URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/40b41f4e0d4b
8145879: ant clean test fails with too many failures
! buildtools/nasgen/project.properties
! make/build-nasgen.xml
! make/build.xml
! make/projec
