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