Michael,

That looks like a fine patch to the BSD version of UNIXProcess_md.c,
but my idea was to avoid having to have a BSD version of this file
at all.  The hope is eventually to have BSD support integrated into the
standard OpenJDK sources, hopefully without special case code
specifically for BSDs.

Martin

On Sat, Oct 25, 2008 at 13:55, Michael Franz <mvfranz@gmail.com> wrote:
> Martin,
>
> Here are the changes that work for OS X.
>
> Michael
>
>
> On Sat, Oct 25, 2008 at 12:47 PM, Martin Buchholz <martinrb@google.com>
> wrote:
>>
>> Hi Michael,
>>
>> I am the author of the changes to UNIXProcess_md.c.
>> File descriptor 3 is used to communicate any cause of failure
>> back to the parent process.
>>
>> One might think that the openjdk7 code would work on
>> BSD because if the standard way of opening /proc/self/fd,
>> it falls back to closing all descriptors up to sysconf(_SC_OPEN_MAX),
>> which one would think the BSDs would have implemented.
>>
>> In any case, even if the current jdk7 code doesn't work on BSD,
>> it should be easy to create a small patch to it that would be
>> portable to more Unix systems, that could be accepted upstream.
>> I would be willing to review.
>>
>> Martin
>
>