[issue41919] Modify test_codecs to use the new codecs.unregister() function

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Pablo Galindo Salgado added the comment: New changeset 14cdc215aa952d280c18626005d3aff967901d92 by Hai Shi in branch 'master': bpo-41919: Avoid resource leak in test_io (GH-22973) https://github.com/python/cpython/commit/14cdc215aa952d280c18626005d3aff967901d92 -- _

[issue41919] Modify test_codecs to use the new codecs.unregister() function

hai shi added the comment: > AS this is masking other issues in the build bots, we need to revert the > commit unless is fixed in 24 hours per the buildbot workflow Thanks, Pablo. I checked that only test_io.py have resource leak. I create PR-22973, pls take a look if you have free time, than

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Change by hai shi : -- pull_requests: +21888 pull_request: https://github.com/python/cpython/pull/22973 ___ Python tracker ___ ___ P

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Change by Pablo Galindo Salgado : -- pull_requests: +21877 stage: resolved -> patch review pull_request: https://github.com/python/cpython/pull/22961 ___ Python tracker ___ ___

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Pablo Galindo Salgado added the comment: AS this is masking other issues in the build bots, we need to revert the commit unless is fixed in 24 hours per the buildbot workflow -- ___ Python tracker _

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Change by Pablo Galindo Salgado : -- resolution: fixed -> status: closed -> open ___ Python tracker ___ ___ Python-bugs-list mailin

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Pablo Galindo Salgado added the comment: Commit c9f696cb96d1c362d5cad871f61da520572d9b08 introduced a reference leak in the test suite. See https://bugs.python.org/issue42145: c9f696cb96d1c362d5cad871f61da520572d9b08 is the first bad commit commit c9f696cb96d1c362d5cad871f61da520572d9b08 Autho

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Change by STINNER Victor : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ __

[issue41919] Modify test_codecs to use the new codecs.unregister() function

STINNER Victor added the comment: New changeset c9f696cb96d1c362d5cad871f61da520572d9b08 by Hai Shi in branch 'master': bpo-41919, test_codecs: Move codecs.register calls to setUp() (GH-22513) https://github.com/python/cpython/commit/c9f696cb96d1c362d5cad871f61da520572d9b08 --

[issue41919] Modify test_codecs to use the new codecs.unregister() function

Change by STINNER Victor : -- title: Move the codecs.register operation to testcases -> Modify test_codecs to use the new codecs.unregister() function ___ Python tracker ___ _