[issue45272] 'os.path' should not be a frozen module

Change by Guido van Rossum : -- pull_requests: +27884, 27885 pull_request: https://github.com/python/cpython/pull/29648 ___ Python tracker ___

[issue45272] 'os.path' should not be a frozen module

Change by Guido van Rossum : -- pull_requests: +27884, 27885, 27886 pull_request: https://github.com/python/cpython/pull/29648 ___ Python tracker ___

[issue45272] 'os.path' should not be a frozen module

Change by Guido van Rossum : -- pull_requests: +27884 pull_request: https://github.com/python/cpython/pull/29648 ___ Python tracker ___

[issue45272] 'os.path' should not be a frozen module

Change by Guido van Rossum : -- pull_requests: -27882 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45272] 'os.path' should not be a frozen module

Change by Guido van Rossum : -- pull_requests: +27882 pull_request: https://github.com/python/cpython/pull/29648 ___ Python tracker ___

[issue45272] 'os.path' should not be a frozen module

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

[issue45272] 'os.path' should not be a frozen module

Change by Filipe Laíns : -- nosy: +FFY00 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45272] 'os.path' should not be a frozen module

Guido van Rossum added the comment: It was a comment about my general lack of understanding of how the importlib bootstrap process works. I should probably start reading the docstrings before complaining more. :-) -- ___ Python tracker

[issue45272] 'os.path' should not be a frozen module

Eric Snow added the comment: On Wed, Oct 6, 2021 at 11:38 AM Guido van Rossum wrote: > I'm trying to understand the proposed solution, "have _imp.is_frozen() check > the module in sys.modules." Does that mean it would do a dict lookup first? Correct. We'd look up the module in sys.modules

[issue45272] 'os.path' should not be a frozen module

Guido van Rossum added the comment: I'm trying to understand the proposed solution, "have _imp.is_frozen() check the module in sys.modules." Does that mean it would do a dict lookup first? Maybe you should do that in the caller instead? importlib/_bootstrap.py calls it a few times, but I'm

[issue45272] 'os.path' should not be a frozen module

Change by Dong-hee Na : -- nosy: +corona10 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45272] 'os.path' should not be a frozen module

Eric Snow added the comment: The matter here boils down to the design of _imp.is_frozen() [1]. It only checks to see if the given module name shows up in the list of frozen modules in Python/frozen.c. This broke things when I froze os and posixpath/ntpath. The simplest solution was to

[issue45272] 'os.path' should not be a frozen module

New submission from Steve Dower : I noticed that Python/frozen.c includes posixpath as 'os.path'. This is not correct, and shouldn't be necessary anyway, because os.path is just an attribute in "os" and not a concrete module (see Lib/os.py#L95 for the bit that makes it importable, and