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

[Seán Coffey]

On 24/07/2014 20:21, Peter Levart wrote:
> 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 ?
Looks good Peter. The '9-na' label is sufficient for this purpose and 
it's already present. You can add a comment in bug to same effect if you 
wish.

regards,
Sean.

>
> 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
>>>
>>
>