Message 178335 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	sbt
Recipients	asksol, brandon-rhodes, catalin.iacob, christian.heimes, dholth, gregory.p.smith, jnoller, mrmekon, ned.deily, neologix, numbernine, pitrou, rcoyner, sbt, vsekhar
Date	2012-12-27.21:07:56
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1356642476.62.0.512734651229.issue8713@psf.upfronthosting.co.za>
In-reply-to

Content
> The safest default would be fork+exec though we need to implement the > fork+exec code as a C extension module or have it use subprocess (as I > noted in the mb_fork_exec.patch review). That was an old version of the patch. In the branch http://hg.python.org/sandbox/sbt#spawn _posixsubprocess is used instead of fork+exec, and all unnecessary fds are closed. See http://hg.python.org/sandbox/sbt/file/8f08d83264a0/Lib/multiprocessing/popen_spawn_posix.py

> The safest default would be fork+exec though we need to implement the 
> fork+exec code as a C extension module or have it use subprocess (as I 
> noted in the mb_fork_exec.patch review).

That was an old version of the patch.

In the branch

    http://hg.python.org/sandbox/sbt#spawn

_posixsubprocess is used instead of fork+exec, and all unnecessary fds are closed.  See

    http://hg.python.org/sandbox/sbt/file/8f08d83264a0/Lib/multiprocessing/popen_spawn_posix.py

History
Date	User	Action	Args
2012-12-27 21:07:56	sbt	set	recipients: + sbt, gregory.p.smith, pitrou, christian.heimes, ned.deily, jnoller, rcoyner, asksol, dholth, brandon-rhodes, neologix, catalin.iacob, numbernine, vsekhar, mrmekon
2012-12-27 21:07:56	sbt	set	messageid: <1356642476.62.0.512734651229.issue8713@psf.upfronthosting.co.za>
2012-12-27 21:07:56	sbt	link	issue8713 messages
2012-12-27 21:07:56	sbt	create