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

2012-01-31 Thread Jonathan Lu
On 01/31/2012 06:40 PM, Neil Richards wrote: On Thu, 2012-01-26 at 17:45 +0800, Jonathan Lu wrote: I think we forgot to create a bug for this, I've created it now: 7133301: (process) UNIXProcess_md.c should include sys/wait.h rather than wait.h Thanks a lo

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

2012-01-31 Thread Neil Richards
On Thu, 2012-01-26 at 17:45 +0800, Jonathan Lu wrote: > > > I think we forgot to create a bug for this, I've created it > now: > > 7133301: (process) UNIXProcess_md.c should include sys/wait.h > rather than wait.h > > Thanks a lot, Alan! > >

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

2012-01-26 Thread Jonathan Lu
> I think we forgot to create a bug for this, I've created it now: > > 7133301: (process) UNIXProcess_md.c should include sys/wait.h rather than > wait.h > Thanks a lot, Alan! > > I don't mind pushing it for you but maybe this is something that Neil > wants to do? > > Hi Neil, could you please

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

2012-01-25 Thread Alan Bateman
On 13/01/2012 07:38, Jonathan Lu wrote: Hello David and Alan, Thanks for the review. Do you plan to push it? I think we forgot to create a bug for this, I've created it now: 7133301: (process) UNIXProcess_md.c should include sys/wait.h rather than wait.h I don't mind pushing it for you but

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

2012-01-12 Thread Jonathan Lu
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 which does not seem to be compli

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

2012-01-12 Thread Alan Bateman
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 which does not seem to be compliant with POSIX specification, in which the expected header file name should be . see http://en.wikipedia.org/wiki/C_POSIX_library. I

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

2012-01-12 Thread David Holmes
Hi Jonathon, On 12/01/2012 7:35 PM, Jonathan Lu wrote: It was found that solaris/native/java/lang/UNIXProcess_md.c includes which does not seem to be compliant with POSIX specification, in which the expected header file name should be . see http://en.wikipedia.org/wiki/C_POSIX_library. I also p