[8u-dev] RFA: JDK-8000975 - (process) Merge UNIXProcess.java.bsd & UNIXProcess.java.linux

[Peter Levart]

Thanks Sean and Rob. I pushed the changeset and included both bugids in 
the message. I hope this is ok. I see both issues as Resolved now.

JDK-8000975 <https://bugs.openjdk.java.net/browse/JDK-8000975> has Fix 
Version: 9 and a backport issue for 8u40 JDK-8051918 
<https://bugs.openjdk.java.net/browse/JDK-8051918> has been created as 
resolved.
JDK-8047340 <https://bugs.openjdk.java.net/browse/JDK-8047340> has Fix 
Verison: 8u40 (and no mention of 9)

How to tell Jira that 8047340 has been superseded by 8000975 in 9 ?

Regards, Peter

On 07/24/2014 04:44 PM, Seán Coffey wrote:
> Changes look fine to me Peter. Thanks for backporting this one. Please 
> close out JDK-8047340 as a duplicate also.
>
> I took the opportunity to run your change through our JPRT build/test 
> system and all looks fine.
>
> regards,
> Sean.
>
> On 23/07/2014 12:53, Peter Levart wrote:
>> Hi,
>>
>> I would like to backport the fix for:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8000975
>>
>> Which seems to also fix the following issue:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8047340
>>
>> The discussion thread starts here:
>>
>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-March/026062.html 
>>
>>
>> The original fix was made for jdk9 and applied to:
>>
>> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/27e0909d3fa0
>> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/27e0909d3fa0
>>
>> It does not import cleanly into jdk8u-dev repository because a fix to 
>> UNIXProcess.java.solaris was applied to jdk8u-dev lately:
>>
>> http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/9189a6d49ac0
>>
>> ...which was not applied to jdk9-dev. Since the presented patch 
>> supersedes this fix (a fix for Solaris variant to use a thread pool 
>> to manage process reaper tasks), nothing is lost.
>>
>> Here's a webrev of the proposed backported fix:
>>
>> http://cr.openjdk.java.net/~plevart/jdk8u-dev/UNIXProcess/webrev.01/
>>
>> You can compare it to the webrev of original jdk9-dev fix:
>>
>> http://cr.openjdk.java.net/~plevart/jdk9-dev/UNIXProcess/webrev.10/
>>
>>
>> Regards, Peter
>>
>