[issue34651] Disallow fork in a subinterpreter.

miss-islington added the comment: New changeset a4be5aae6e587f5310f1fc0d66d37e032621ce39 by Miss Islington (bot) in branch '3.8': bpo-38778: Document that os.fork is not allowed in subinterpreters (GH-17123) https://github.com/python/cpython/commit/a4be5aae6e587f5310f1fc0d66d37e032621ce39

[issue34651] Disallow fork in a subinterpreter.

Change by miss-islington : -- pull_requests: +16688 pull_request: https://github.com/python/cpython/pull/17179 ___ Python tracker ___

[issue34651] Disallow fork in a subinterpreter.

miss-islington added the comment: New changeset b22030073b9327a3aeccb69507694bce078192aa by Miss Islington (bot) (Phil Connell) in branch 'master': bpo-38778: Document that os.fork is not allowed in subinterpreters (GH-17123)

[issue34651] Disallow fork in a subinterpreter.

Change by Roundup Robot : -- pull_requests: +16630 pull_request: https://github.com/python/cpython/pull/17123 ___ Python tracker ___

[issue34651] Disallow fork in a subinterpreter.

Change by STINNER Victor : -- resolution: -> fixed status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue34651] Disallow fork in a subinterpreter.

Miro Hrončok added the comment: The problem with subprocess.Popen has been fixed in https://bugs.python.org/issue37951 -- ___ Python tracker ___

[issue34651] Disallow fork in a subinterpreter.

STINNER Victor added the comment: See also bpo-37266: "Daemon threads must be forbidden in subinterpreters". -- ___ Python tracker ___

[issue34651] Disallow fork in a subinterpreter.

Eric Snow added the comment: FYI, I plan on looking into this either today or next Friday. -- ___ Python tracker ___ ___

[issue34651] Disallow fork in a subinterpreter.

STINNER Victor added the comment: > I'd start by documenting the limitation and keeping a future feature request > of "Allow subprocess to work from subinterpreters". That is doable. > Supporting os.fork() is not. subprocess is able to use os.posix_spawn() in Python 3.8. In that case,

[issue34651] Disallow fork in a subinterpreter.

Gregory P. Smith added the comment: I'd start by documenting the limitation and keeping a future feature request of "Allow subprocess to work from subinterpreters". That is doable. Supporting os.fork() is not. -- ___ Python tracker

[issue34651] Disallow fork in a subinterpreter.

STINNER Victor added the comment: I reopen the issue to let Eric answer ;-) If the behavior is deliberate, maybe it should just be documented somewhere? -- nosy: +vstinner resolution: fixed -> status: closed -> open ___ Python tracker

[issue34651] Disallow fork in a subinterpreter.

Miro Hrončok added the comment: It appears that as a result of this, subprocess.Popen cannot be called from within a subinterpreter either. Is that an obvious and desired limitation? -- nosy: +hroncok ___ Python tracker

[issue34651] Disallow fork in a subinterpreter.

Change by Eric Snow : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue34651] Disallow fork in a subinterpreter.

Eric Snow added the comment: New changeset 5903296045b586b9cd1fce0b1e02caf896028d1d by Eric Snow in branch 'master': bpo-34651: Only allow the main interpreter to fork. (gh-9279) https://github.com/python/cpython/commit/5903296045b586b9cd1fce0b1e02caf896028d1d --

[issue34651] Disallow fork in a subinterpreter.

Gregory P. Smith added the comment: +1 agreed. this is the simplest approach to start with. Code to restrict: os.fork itself and disallowing the use of preexec_fn on subprocess within subinterpreters. feel free to ignore preexec_fn in subprocess for the time being if desired, we already

[issue34651] Disallow fork in a subinterpreter.

Change by Eric Snow : -- keywords: +patch pull_requests: +8710 stage: needs patch -> patch review ___ Python tracker ___ ___

[issue34651] Disallow fork in a subinterpreter.

New submission from Eric Snow : os.fork() potentially has some problematic behavior when called from a subinterpreter. In additional to the normal fork+threads madness, there's the question of what to do with existing subinterpreters. The simplest solution is to simply disallow fork in a