Hi All

Webrev updated at http://cr.openjdk.java.net/~weijun/8056174/webrev.01/.

Major changes:

1. JarSignerException is now a RuntimeException, no more ErrorCode

2. Action.java and Builder.java moved into sun.security.tools.jarsigner. Hopefully the jarsigner tool can use them.

The code changes in the root repo remain unchanged. The unshuffle_list.txt also remains. We are considering backporting the enhancement to jdk8u to ease the transition from calling internal jarsigner class to this new API.

Thanks to Mandy and Sean for the code review.

--Max

On 3/24/2015 5:47 PM, Wang Weijun wrote:
Hi All

Please review the code change at

    http://cr.openjdk.java.net/~weijun/8056174/webrev.00/

It provides a new jdk.Exported API JarSigner to perform jar signing. The class 
contains popular functions of the jarsigner tool. The jarsigner tool is 
unchanged and currently independent of the new class.

The root repo also needs a little change:

diff --git a/common/bin/unshuffle_list.txt b/common/bin/unshuffle_list.txt
--- a/common/bin/unshuffle_list.txt
+++ b/common/bin/unshuffle_list.txt
@@ -1305,6 +1304,7 @@
  jdk/src/jdk.httpserver/share/classes/com/sun/net/httpserver : 
jdk/src/share/classes/com/sun/net/httpserver
  jdk/src/jdk.httpserver/share/classes/sun/net/httpserver : 
jdk/src/share/classes/sun/net/httpserver
  jdk/src/jdk.jartool/share/classes/com/sun/jarsigner : 
jdk/src/share/classes/com/sun/jarsigner
+jdk/src/jdk.jartool/share/classes/jdk/security/jarsigner : 
jdk/src/share/classes/jdk/security/jarsigner
  jdk/src/jdk.jartool/share/classes/sun/security/tools/jarsigner : 
jdk/src/share/classes/sun/security/tools/jarsigner
  jdk/src/jdk.jartool/share/classes/sun/tools/jar : 
jdk/src/share/classes/sun/tools/jar
  jdk/src/jdk.jcmd/share/classes/sun/tools/jcmd : 
jdk/src/share/classes/sun/tools/jcmd
diff --git a/modules.xml b/modules.xml
--- a/modules.xml
+++ b/modules.xml
@@ -1634,6 +1645,9 @@
      <export>
        <name>com.sun.jarsigner</name>
      </export>
+    <export>
+      <name>jdk.security.jarsigner</name>
+    </export>
    </module>
    <module>
      <name>jdk.javadoc</name>

Thanks
Max

Reply via email to