Review request for 5049299

Florian Weimer fweimer at bfk.de
Thu May 28 08:56:09 UTC 2009


* Michael McMahon:

> webrev location: http://cr.openjdk.java.net/~michaelm/5049299/webrev.00/
>
> **I'd like to give an outline of the change here, to make reviewing
> the webrev a bit easier.  Basically, while posix_spawn() is a fairly
> elaborate and complicated system call, it can't quite do everything
> that the old fork()/exec() combination can do,

I've always assumed that the ProcessBuilder requirements could be
fulfilled by posix_spawn and friends.  Could you elaborate why this is
not the case?

The file descriptor setup race mentioned in the comments shouldn't
apply, really--the reshuffling should occur in the child, isolated
from the parent, otherwise the implementation is quite broken and
pointless.

-- 
Florian Weimer                <fweimer at bfk.de>
BFK edv-consulting GmbH       http://www.bfk.de/
Kriegsstraße 100              tel: +49-721-96201-1
D-76133 Karlsruhe             fax: +49-721-96201-99



More information about the core-libs-dev mailing list