[issue45653] Freeze the encodings module.

Christian Heimes added the comment: Eric, I have a simple reproducer for the issue: This works: $ LC_ALL=en_US.utf-8 TESTPATH=$(pwd)/Lib:$(pwd)/build/lib.linux-x86_64-3.11 ./Programs/_testembed test_init_setpath_config This fails because it cannot load ISO-8859-1 / latin-1 codec $ LC_ALL=e

[issue45653] Freeze the encodings module.

Change by Christian Heimes : -- nosy: +christian.heimes nosy_count: 5.0 -> 6.0 pull_requests: +28255 pull_request: https://github.com/python/cpython/pull/30030 ___ Python tracker _

[issue45653] Freeze the encodings module.

Guido van Rossum added the comment: New changeset 02b5ac6091ada0c2df99c4e1eae37ddccbcd91f0 by Kumar Aditya in branch 'main': bpo-45653: fix test_embed on windows (GH-29814) https://github.com/python/cpython/commit/02b5ac6091ada0c2df99c4e1eae37ddccbcd91f0 -- nosy: +gvanrossum __

[issue45653] Freeze the encodings module.

Change by Kumar Aditya : -- pull_requests: +28047 pull_request: https://github.com/python/cpython/pull/29814 ___ Python tracker ___

[issue45653] Freeze the encodings module.

Change by Kumar Aditya : -- nosy: +kumaraditya303 nosy_count: 3.0 -> 4.0 pull_requests: +28024 pull_request: https://github.com/python/cpython/pull/29788 ___ Python tracker ___

[issue45653] Freeze the encodings module.

Change by Filipe Laíns : -- stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45653] Freeze the encodings module.

Filipe Laíns added the comment: I have already opened up the PR, but I can change if desired. -- stage: patch review -> needs patch ___ Python tracker ___ _

[issue45653] Freeze the encodings module.

Marc-Andre Lemburg added the comment: On 30.10.2021 17:54, Filipe Laíns wrote: > > I just tested partially freezing the package, and it seems to working fine :) FWIW: I think it's best not bother and simply freeze the whole thing. It's mostly char mappings which compress well and there's a ben

[issue45653] Freeze the encodings module.

Change by Filipe Laíns : -- keywords: +patch pull_requests: +27599 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/29331 ___ Python tracker _

[issue45653] Freeze the encodings module.

Filipe Laíns added the comment: I just tested partially freezing the package, and it seems to working fine :) -- ___ Python tracker ___

[issue45653] Freeze the encodings module.

Eric Snow added the comment: On Thu, Oct 28, 2021 at 12:15 PM Marc-Andre Lemburg wrote: > encodings is a package. I think you first have to check whether mixing > frozen and non-frozen submodules are even supported. I've never tried > having only part of a package frozen. It works as long as

[issue45653] Freeze the encodings module.

Marc-Andre Lemburg added the comment: encodings is a package. I think you first have to check whether mixing frozen and non-frozen submodules are even supported. I've never tried having only part of a package frozen. Freezing the whole package certainly works. -- nosy: +lemburg _

[issue45653] Freeze the encodings module.

New submission from Eric Snow : Currently we freeze all the modules imported during runtime initialization, except for the encodings module. It has a lot of submodules and this results in a lot of extra noise in builds. We hadn't frozen it yet because we were still ironing out changes relat