RFR 8060432: tools/pack200/TestNormal.java fails on Windows with java.io.FileNotFoundException after JDK-8058854
Amy Lu
amy.lu at oracle.com
Thu Oct 16 07:36:55 UTC 2014
Thank you Kumar for your review.
I need a sponsorfor push this fix. Alan, may I get your help?
Thanks,
Amy
On 10/16/14, 8:31 AM, Kumar Srinivasan wrote:
> Looks good!
>
> Kumar
>
> On 10/14/2014 7:42 PM, Amy Lu wrote:
>> On 10/15/14, 4:44 AM, Kumar Srinivasan wrote:
>>> Amy,
>>>
>>> The modifications you have made will not test pack200 compression
>>> and normalization correctly, as the test needs ".class" files.
>>
>> Sorry, I missed that.
>> Please review the updated version, test works on ".class" files
>> (Utils.TEST_CLS_DIR)
>> http://cr.openjdk.java.net/~weijun/8060432/webrev.01/
>>
>>> Do you know
>>> why the test fails on windows ?
>>
>> extractJar(JarFile jf, File where) try to extract jar files to folder
>> "testdir", as the jar contains files of
>> C:/blabla
>> extractJar method run into issue on Windows as it try to extract it
>> to testdir\\C:\\blabla
>>
>> Anyway, as there already has some ".class" files to working on
>> (Utils.TEST_CLS_DIR), extractJar method is not needed.
>>
>> Thanks,
>> Amy
>>>
>>> Kumar
>>>
>>>
>>> On 10/14/2014 7:19 AM, Amy Lu wrote:
>>>> Please review the test fix.
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8060432
>>>> webrev: http://cr.openjdk.java.net/~weijun/8060432/webrev.00/
>>>>
>>>> This test is to test
>>>> compareJars(new JarFile("normalized.jar"), new
>>>> JarFile("repacked.jar"));
>>>> where the jar files (normalized.jar repacked.jar and original.jar)
>>>> are created by "jar cnf" "jar cf" or "pack200 -r”.
>>>>
>>>> extractJar(JarFile jf, File where) is not really needed as this
>>>> method just try to provide some files for the test to do "jar cnf”.
>>>> Actually, "jar cnf" can work on any files, test just need some
>>>> files to jar with.
>>>>
>>>> Test can just simply created some files for that purpose.
>>>>
>>>> Thanks,
>>>> Amy
>>>>
>>>
>>
>
More information about the core-libs-dev
mailing list