Review request for 5049299 (original) (raw)

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

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 posixspawn() 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