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 meilyadam
Recipients gregory.p.smith, meilyadam, paul.moore, r.david.murray, steve.dower, tim.golden, vstinner, zach.ware
Date 2015-08-21.19:33:03
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1440185588.19.0.679369012065.issue24909@psf.upfronthosting.co.za>
In-reply-to
Content
Ok, I can re-implement the patch to meet what you all are looking for. I just want to double check that I'm on the same page:

I'll get rid of the lock, because the fix should really be done in the call to CreateProcessW. I imagine that I'll be editing Modules/_winapi.c:824. Essentially, the fix will include switching _winapi.c to use a STARTUPINFOEX structure, and manually passing in the three handles for stdin, stdout, and stderr to PROC_THREAD_ATTRIBUTE_HANDLE_LIST.

The one potential problem that I see is that it looks like specifying PROC_THREAD_ATTRIBUTE_HANDLE_LIST means that no other handles are inherited, even if they are inheritable. Doesn't this break applications that are explicitly creating inheritable handles and expecting them to be accessible within the subprocess? I could add Windows support for pass_fds, but then existing applications would have to be updated to use the argument on Windows.
History
Date User Action Args
2015-08-21 19:33:08meilyadamsetrecipients: + meilyadam, gregory.p.smith, paul.moore, vstinner, tim.golden, r.david.murray, zach.ware, steve.dower
2015-08-21 19:33:08meilyadamsetmessageid: <1440185588.19.0.679369012065.issue24909@psf.upfronthosting.co.za>
2015-08-21 19:33:07meilyadamlinkissue24909 messages
2015-08-21 19:33:04meilyadamcreate