Quantcast

RFR: 8176834: jdk/nio/zipfs/MultiReleaseJarTest.java test fails after JDK-8176709

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR: 8176834: jdk/nio/zipfs/MultiReleaseJarTest.java test fails after JDK-8176709

Claes Redestad
Hi,

it appears the test we wrote to test the Multi-Release checking in
JarFile, based on the jar manifest specification[1] and now adapted
for JDK-8176709 is too strict when actually using
java.util.jar.Manifest, which means the test I added for JDK-8176709
fails on the version that I pushed (but not on the version I tested it
on - shame! shame!)

Webrev: http://cr.openjdk.java.net/~redestad/8176834/jdk.01/
Bug: https://bugs.openjdk.java.net/browse/JDK-8176834

I've filed a bug to re-examine the Manifest
specification/implementation here, which may or may not need to be
higher priority:

https://bugs.openjdk.java.net/browse/JDK-8176843

Thanks!

/Claes
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: 8176834: jdk/nio/zipfs/MultiReleaseJarTest.java test fails after JDK-8176709

Mandy Chung

> On Mar 15, 2017, at 3:01 PM, Claes Redestad <[hidden email]> wrote:
>
> Hi,
>
> it appears the test we wrote to test the Multi-Release checking in
> JarFile, based on the jar manifest specification[1] and now adapted
> for JDK-8176709 is too strict when actually using
> java.util.jar.Manifest, which means the test I added for JDK-8176709
> fails on the version that I pushed (but not on the version I tested it
> on - shame! shame!)
>
> Webrev: http://cr.openjdk.java.net/~redestad/8176834/jdk.01/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8176834
>

It’s okay to exclude those test cases until JDK-8176843 is resolved.

Thanks
Mandy
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: 8176834: jdk/nio/zipfs/MultiReleaseJarTest.java test fails after JDK-8176709

Claes Redestad


On 2017-03-15 23:05, Mandy Chung wrote:

>
>> On Mar 15, 2017, at 3:01 PM, Claes Redestad <[hidden email]> wrote:
>>
>> Hi,
>>
>> it appears the test we wrote to test the Multi-Release checking in
>> JarFile, based on the jar manifest specification[1] and now adapted
>> for JDK-8176709 is too strict when actually using
>> java.util.jar.Manifest, which means the test I added for JDK-8176709
>> fails on the version that I pushed (but not on the version I tested it
>> on - shame! shame!)
>>
>> Webrev: http://cr.openjdk.java.net/~redestad/8176834/jdk.01/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8176834
>>
>
> It’s okay to exclude those test cases until JDK-8176843 is resolved.
>
> Thanks
> Mandy
>

Thanks Mandy!

/Claes
Loading...