[8u-dev] RFR 8150070: TESTBUG: com/sun/jdi tests fail on Cygwin but pass on MKS

[Konstantin Shefov]

Alexander,

Direct backport form JDK 9 does not work on JDK 8, JDK 9 tests just hang 
on JDK 8 because of some JDK9-specific changes have been done there.

As for JDK 9 tests, they pass without error, although they use tasklist 
to search for Cygwin PID. Perhaps they do not fail because when I run 
them Cygwin PID is equal to Windows PID, so tasklist works.

-Konstantin

19.02.2016 17:57, Alexander Kulyakhtin пишет:
> Hi Konstantin,
>
> Thank you very much for your answer.
>
> The fix is not a backport of what we have in the JDK 9, is it? Since the issue is not observed in the JDK 9, why the backport would not be sufficient?
>
> (My concern is, if the backport is not sufficient, perhaps, some similar changes in the JDK 9 should also be done)
>
> Best regards,
> Alexander
>
> ----- Original Message -----
> From: konstantin.shefov at oracle.com
> To: alexander.kulyakhtin at oracle.com
> Cc: serviceability-dev at openjdk.java.net
> Sent: Friday, February 19, 2016 5:49:29 PM GMT +03:00 Iraq
> Subject: Re: [8u-dev] RFR 8150070: TESTBUG: com/sun/jdi tests fail on Cygwin but pass on MKS
>
> Yes, it is a fix for JDK 8u only.
>
> -Konstantin
>
> On 02/19/2016 05:47 PM, Alexander Kulyakhtin wrote:
>> Hi Konstantin,
>>
>> (not a reviewer)
>>
>> Could you, please, explain your fix a bit more, if possible?
>> Since the comment to the CR says "Cannot be reproduced with JDK 9.", is this fix for JDK 8 only?
>>
>> Best regards,
>> Alexander
>>
>> ----- Original Message -----
>> From: konstantin.shefov at oracle.com
>> To: serviceability-dev at openjdk.java.net
>> Sent: Thursday, February 18, 2016 7:09:26 PM GMT +03:00 Iraq
>> Subject: [8u-dev] RFR 8150070: TESTBUG: com/sun/jdi tests fail on Cygwin but pass on MKS
>>
>> Hello
>>
>> Please review a test bug fix.
>> The test fails because Cygwin PID returned by "$!" and Windows PID
>> returned by "tasklist" are not the same.
>> Need to use cygwin "ps".
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8150070
>> Webrev: http://cr.openjdk.java.net/~kshefov/8150070/webrev
>>
>> Thanks
>> -Konstantin