RFR 8020802: Need an ability to create jar files that are invariant to the pack200 packing/unpacking
Alan Bateman
Alan.Bateman at oracle.com
Sun Oct 20 19:15:06 UTC 2013
On 19/10/2013 16:14, Kumar Srinivasan wrote:
> Hi Alex,
>
> This looks good.
>
> Hi Sherman, Alan,
>
> Could one of you review this, please
>
The "-n" options seems okay but I wonder if there has any thought given
to having an option on jarsigner to normalize it before signing?
On the patch itself then I think it needs to be cleaned up before
pushing it. From a quick look:
- It's using a raw type for the packer properties, should be Map<String,
String>
- Is any reason not to use try-with-resources?
- I assume the catching of Exception in createTemporaryFile should be
IOException|SecurityException
- Is the @test tag in the test right?
- The test appears to extract jfr.jar but I don't think we can assume
this exists
- Some of the jar tests use sun.tools.jar.Main and this might be useful here
One other thing, Mathias's mail to jdk8-dev [1] asks for changes that
require translation to be in the master no longer than 10/24. Does the
update to jar.properties count?
-Alan
[1] http://mail.openjdk.java.net/pipermail/jdk8-dev/2013-October/003443.html
More information about the core-libs-dev
mailing list