STINNER Victor added the comment:
The posix_spawn() bug is now fixed, I close again the issue.
Feel free to open a new issue if you want to follow-up on reworking the
posix_spawn() code path ;-)
--
resolution: -> fixed
stage: commit review -> resolved
status: open -> closed
STINNER Victor added the comment:
> As mentioned in the code review for Victor's fix, it feels like the big
> conditional on the use of self._posix_spawn in Popen._execute_child should
> perhaps be pulled into _posix_spawn, or at least refactored so it's closer to
> _posix_spawn, with some
Thomas Wouters added the comment:
As mentioned in the code review for Victor's fix, it feels like the big
conditional on the use of self._posix_spawn in Popen._execute_child should
perhaps be pulled into _posix_spawn, or at least refactored so it's closer to
_posix_spawn, with some
Thomas Wouters added the comment:
New changeset faca8553425c231d867dcabf6a69a9dd21118b6c by T. Wouters (Victor
Stinner) in branch 'master':
bpo-36046: posix_spawn() doesn't support uid/gid (GH-16384)
https://github.com/python/cpython/commit/faca8553425c231d867dcabf6a69a9dd21118b6c
Change by STINNER Victor :
--
resolution: fixed ->
status: closed -> open
title: support dropping privileges when running subprocesses -> subprocess: add
user, group and extra_groups parameters
___
Python tracker