[8u-dev] Request for approval: 6545422: TEST BUG: NativeErrors.java uses wrong path name in exec

[Ivan Gerasimov]

Thanks Seán!

On 07.07.2014 13:49, Seán Coffey wrote:
> Approved.
>
> regards,
> Sean.
>
> On 01/07/14 22:32, Ivan Gerasimov wrote:
>> Corrected the subject line: I'm requesting an approval to push the 
>> fix into jdk8u-dev repo, not jdk8u20
>>
>> On 02.07.2014 1:29, Ivan Gerasimov wrote:
>>> Hello!
>>>
>>> I'm rerequesting an approval to backport this test fix into jdk8u.
>>> To address the issue with NoClassDefFoundError I added the implicit 
>>> @build line to the test (as it was done in JDK-8043520).
>>>
>>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-6545422
>>> WEBREV: http://cr.openjdk.java.net/~igerasim/6545422/0/webrev/
>>> Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/efeab0eae50f
>>> Review: 
>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-February/014152.html
>>>
>>> Would you please approve the backport?
>>>
>>> Sincerely yours,
>>> Ivan
>>>
>>> On 24.02.2014 13:15, Ivan Gerasimov wrote:
>>>>
>>>> On 24.02.2014 13:03, Alan Bateman wrote:
>>>>> On 24/02/2014 08:56, Ivan Gerasimov wrote:
>>>>>> Hello!
>>>>>>
>>>>>> Would you please approve porting back this test bug fix?
>>>>>> The fix applies cleanly to jdk8u.
>>>>>>
>>>>>> Master Bug: https://bugs.openjdk.java.net/browse/JDK-6545422
>>>>>> Jdk9 change: 
>>>>>> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/efeab0eae50f
>>>>>> Review: 
>>>>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-February/014152.html
>>>>> Ivan - would it be possible to hold back on this one for a bit. 
>>>>> The updated test is dependent on implicit compilation of the 
>>>>> classes in the @library and this is causing problems for 
>>>>> concurrent test runs. The same pattern has slipped into 15-20 
>>>>> other tests with the result that they all fail intermittently due 
>>>>> to NoClassDefFoundError issues. I think they should be fixed in 
>>>>> jdk9-dev but any of these "improved" tests are backported.
>>>>>
>>>>
>>>> Yes, sure!
>>>> Actually, I was going to port this fix into jdk6 (the test fails 
>>>> there), but modified in such a way that it does not depend on 
>>>> testlibrary.
>>>> I can still work on jdk6 port without first integrating it into 
>>>> jdk8u and jdk7u.
>>>>
>>>> Sincerely yours,
>>>> Ivan
>>>>
>>>>
>>>>> -Alan.
>>>>>
>>>>>
>>>>
>>>>
>>>>
>>>>
>>>
>>>
>>>
>>
>
>
>