[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

STINNER Victor added the comment: See also bpo-38865: "Can Py_Finalize() be called if the current interpreter is not the main interpreter?". -- nosy: +vstinner ___ Python tracker

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Change by Maciej Szulik : -- nosy: +maciej.szulik ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Change by Lewis Gaul : -- pull_requests: +17048 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/17575 ___ Python tracker ___

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Joannah Nanjekye added the comment: Am abit swamped and sick atm. You can go on and submit a fix. -- ___ Python tracker ___ ___

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Lewis Gaul added the comment: I've put together a test along the lines of what Nick suggested, see the attached patch. Running this hits the Fatal 'remaining subinterpreters' error as expected: ``` > ./Programs/_testembed test_bpo36225 --- Pass 0 --- interp 0 <0x1A561A0>, thread state

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Joannah Nanjekye added the comment: I remember julien wanting to check this out during a discussion we had at the sprints hence the loop in. -- nosy: +mdk ___ Python tracker

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Joannah Nanjekye added the comment: The test request is moved to issue37776. -- ___ Python tracker ___ ___ Python-bugs-list

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Joannah Nanjekye added the comment: I am investigating this but in the meantime. > It also occurs to me that we don't currently have a test case for what > happens if you call Py_Finalize from a sub-interpreter rather than the main > interpreter. @ncoghlan Am moving this test request in a

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Ned Deily added the comment: Ping. It was marked as a 3.7regression but perhaps it should now be just targeted for 3.8. -- nosy: +ned.deily versions: +Python 3.9 ___ Python tracker

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Nick Coghlan added the comment: I think test_embed would be the right home for this, as there's an existing test case there for subinterpreter lifecycles and repeated init/finalize cycles:

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Eric Snow added the comment: test__xxsubinterpreters is a great place for tests that exercise use of subinterpreters, including most lifecycle operations. There are also one or two subinterpreter-related tests in test_embed. However, for this issue the interplay with runtime finalization

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Eric Snow added the comment: Interestingly, I noticed this independently today. :) Here's what I wrote in #36477 (which I've closed as a duplicate): When using subinterpreters, any that exist when Py_FinalizeEx() is called do not appear to get cleaned up during runtime finalization. Maybe

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Petr Viktorin added the comment: Joannah, yes, that looks like a good place. Eric Snow might have more info; he wrote that module. As for testing Py_FatalError, there's an assert_python_failure function in test.support.script_helper. -- ___

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Joannah Nanjekye added the comment: I have been wondering where the regression to test this can be put..in test__xxsubinterpreters.py may be? -- ___ Python tracker ___

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

Change by Joannah Nanjekye : -- nosy: +nanjekyejoannah ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue36225] Lingering subinterpreters should be implicitly cleared on shutdown

New submission from Nick Coghlan : https://docs.python.org/3/c-api/init.html#c.Py_EndInterpreter states that "Py_FinalizeEx() will destroy all sub-interpreters that haven’t been explicitly destroyed at that point." As discussed in https://github.com/hexchat/hexchat/issues/2237, Python 3.7+