Request for review: 7201156 : jar tool fails to convert file separation characters for list and extract
Chris Hegarty
chris.hegarty at oracle.com
Fri Nov 9 09:57:03 UTC 2012
Sean,
Thank you for adding a test. Rather than creating a dependency on rt.jar
it may be best to have the test use the java.util.jar API to create a
small temporary jar file.
The problem with the dependency on rt.jar is that the test will fail if
run against a development build where the images have not been created (
run against an exploded classes directory ).
-Chris.
On 09/11/2012 04:31, Sean Chou wrote:
> Hello Chris, Alan, and Xueming,
>
> I added the testcase, please take a look.
>
> webrev: http://cr.openjdk.java.net/~zhouyx/7201156/webrev.01/
>
>
>
>
> On Wed, Nov 7, 2012 at 6:16 PM, Chris Hegarty <chris.hegarty at oracle.com
> <mailto:chris.hegarty at oracle.com>> wrote:
>
> The change looks fine to me.
>
> I wonder if it is worth creating an automatic regression test to
> verify this change ( so an future regression in behavior gets caught
> early ). You could call sun.tools.jar.Main directly passing suitable
> streams to check the output.
>
> -Chris.
>
>
> On 07/11/2012 08:31, Sean Chou wrote:
>
> Hello,
>
> This is the suggested fix described in sun bug 7201156 page.
> Please take a
> look.
>
> sunbug:
> http://bugs.sun.com/__bugdatabase/view_bug.do?bug___id=7201156
> <http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7201156>
> webrev: http://cr.openjdk.java.net/~__zhouyx/7201156/webrev.00/
> <http://cr.openjdk.java.net/~zhouyx/7201156/webrev.00/>
>
>
>
>
> --
> Best Regards,
> Sean Chou
>
More information about the core-libs-dev
mailing list