POSIX compatibility change for including wait.h in UNIXProcess_md.c

Jonathan Lu luchsh at linux.vnet.ibm.com
Fri Jan 13 07:38:45 UTC 2012


Hello David and Alan,

Thanks for the review.
Do you plan to push it?

Cheers
- Jonathan

On 01/12/2012 07:33 PM, Alan Bateman wrote:
> On 12/01/2012 09:35, Jonathan Lu wrote:
>> Hi core-libs-dev,
>>
>> It was found that solaris/native/java/lang/UNIXProcess_md.c includes 
>> <wait.h> which does not seem to be compliant with POSIX 
>> specification, in which the expected header file name should be 
>> <sys/wait.h>. see http://en.wikipedia.org/wiki/C_POSIX_library. I 
>> also performed a 'grep' for the code base, it seems nowhere else 
>> needs to be changed so far.
>>
>> So here is a simple patch to adjust this issue, any suggestions?
>> http://cr.openjdk.java.net/~luchsh/wait_h_POSIX_comp/
>>
>> Cheers
>> - Jonathan
>>
> This looks fine to me.
>
> -Alan




More information about the core-libs-dev mailing list