[issue45020] Freeze all modules imported during startup.

2021-11-24 Thread Christian Heimes
Christian Heimes added the comment: New changeset 5c4b19ec49a5fbad65a682225f7cfed8b78f2a2f by Christian Heimes in branch 'main': bpo-45020: Fix strict-prototypes warning (GH-29755) https://github.com/python/cpython/commit/5c4b19ec49a5fbad65a682225f7cfed8b78f2a2f --

[issue45020] Freeze all modules imported during startup.

2021-11-24 Thread Christian Heimes
Change by Christian Heimes : -- nosy: +christian.heimes, christian.heimes, christian.heimes nosy_count: 24.0 -> 25.0 pull_requests: +27992, 27993, 27994 pull_request: https://github.com/python/cpython/pull/29755 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-11-24 Thread Christian Heimes
Change by Christian Heimes : -- nosy: +christian.heimes, christian.heimes nosy_count: 24.0 -> 25.0 pull_requests: +27992, 27993 pull_request: https://github.com/python/cpython/pull/29755 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-11-24 Thread Christian Heimes
Change by Christian Heimes : -- nosy: +christian.heimes nosy_count: 24.0 -> 25.0 pull_requests: +27992 pull_request: https://github.com/python/cpython/pull/29755 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-11-10 Thread Guido van Rossum
Guido van Rossum added the comment: New changeset 1cbaa505d007e11c4a1f0d2073d72b6c02c7147c by Guido van Rossum in branch 'main': bpo-45696: Deep-freeze selected modules (GH-29118) https://github.com/python/cpython/commit/1cbaa505d007e11c4a1f0d2073d72b6c02c7147c --

[issue45020] Freeze all modules imported during startup.

2021-10-28 Thread Eric Snow
Eric Snow added the comment: I consider this done. There is some lingering follow-up work, for which I've created a number of issues: * https://bugs.python.org/issue45396 - -X frozen_modules=off ignores custom frozen modules * https://bugs.python.org/issue45395 - custom modules override

[issue45020] Freeze all modules imported during startup.

2021-10-28 Thread Eric Snow
Eric Snow added the comment: On Mon, Oct 18, 2021 at 7:14 PM Guido van Rossum wrote: > Is gh-28940 only for UNIX? > I built on Windows with default options (PCbuild\build.bat) and it looks like > the frozen modules are used by default even though I am running in the source > directory. (I

[issue45020] Freeze all modules imported during startup.

2021-10-25 Thread Eric Snow
Eric Snow added the comment: New changeset 6afb285ff0790471a6858e44f85d143f07fda70c by Eric Snow in branch 'main': bpo-45020: Add tests for the -X "frozen_modules" option. (gh-28997) https://github.com/python/cpython/commit/6afb285ff0790471a6858e44f85d143f07fda70c --

[issue45020] Freeze all modules imported during startup.

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

[issue45020] Freeze all modules imported during startup.

2021-10-19 Thread Eric Snow
Eric Snow added the comment: On Mon, Oct 18, 2021 at 7:14 PM Guido van Rossum wrote: > Is gh-28940 only for UNIX? It wasn't meant to be. :( > I built on Windows with default options (PCbuild\build.bat) and it looks like > the frozen modules are used by default even though I am running in

[issue45020] Freeze all modules imported during startup.

2021-10-18 Thread Guido van Rossum
Guido van Rossum added the comment: Is gh-28940 only for UNIX? I built on Windows with default options (PCbuild\build.bat) and it looks like the frozen modules are used by default even though I am running in the source directory. (I put a printf() call in unmarshal_frozen_code().) I also

[issue45020] Freeze all modules imported during startup.

2021-10-18 Thread Gregory P. Smith
Gregory P. Smith added the comment: could changes related to this be the cause of https://bugs.python.org/issue45506 ? out of tree builds in main usually cannot pass key tests today. they often hang or blow up with strange exceptions. -- nosy: +gregory.p.smith

[issue45020] Freeze all modules imported during startup.

2021-10-16 Thread Eric Snow
Eric Snow added the comment: New changeset b9cdd0fb9c463c2503a4d854bb6529a9db58fe1b by Eric Snow in branch 'main': bpo-45020: Default to using frozen modules unless running from source tree. (gh-28940) https://github.com/python/cpython/commit/b9cdd0fb9c463c2503a4d854bb6529a9db58fe1b

[issue45020] Freeze all modules imported during startup.

2021-10-16 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +27280 pull_request: https://github.com/python/cpython/pull/28997 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-10-13 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +27229 pull_request: https://github.com/python/cpython/pull/28940 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-10-06 Thread Eric Snow
Eric Snow added the comment: On Tue, Oct 5, 2021 at 11:31 AM Guido van Rossum wrote: > Whoa. os.path is not always an alias for posixpath, is it? Steve brought this to my attention a couple weeks ago. Bottom line: the frozen module entry is only there for checks, not for actual import, but

[issue45020] Freeze all modules imported during startup.

2021-10-06 Thread Larry Hastings
Larry Hastings added the comment: Nope. On Windows, os.path is "ntpath". -- ___ Python tracker ___ ___ Python-bugs-list mailing

[issue45020] Freeze all modules imported during startup.

2021-10-05 Thread Guido van Rossum
Guido van Rossum added the comment: Whoa. os.path is not always an alias for posixpath, is it? -- ___ Python tracker ___ ___

[issue45020] Freeze all modules imported during startup.

2021-10-05 Thread Eric Snow
Eric Snow added the comment: New changeset 08285d563e64c179a56ab2f952345b3dbcdb54f3 by Eric Snow in branch 'main': bpo-45020: Identify which frozen modules are actually aliases. (gh-28655) https://github.com/python/cpython/commit/08285d563e64c179a56ab2f952345b3dbcdb54f3 --

[issue45020] Freeze all modules imported during startup.

2021-10-01 Thread Guido van Rossum
Guido van Rossum added the comment: And it's most definitely 3.11, not 3.9. (Did you mean to change a different issue?) -- versions: +Python 3.11 -Python 3.9 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-10-01 Thread Guido van Rossum
Guido van Rossum added the comment: @santhu_reddy12, why did you assign this to the Parser category? IMO this issue is clearly in the Build category. (We haven't met, I assume you have triage permissions?) -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-10-01 Thread santhosh
Change by santhosh : -- components: +Parser -Build nosy: +lys.nikolaou, pablogsal, santhu_reddy12 versions: +Python 3.9 -Python 3.11 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-10-01 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +27045 pull_request: https://github.com/python/cpython/pull/28655 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-30 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +27030 pull_request: https://github.com/python/cpython/pull/28665 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-30 Thread Eric Snow
Eric Snow added the comment: New changeset 7e5c107541726b90d3f2e6e69ef37180cf58335d by Eric Snow in branch 'main': bpo-45020: Add more test cases for frozen modules. (gh-28664) https://github.com/python/cpython/commit/7e5c107541726b90d3f2e6e69ef37180cf58335d --

[issue45020] Freeze all modules imported during startup.

2021-09-30 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +27029 pull_request: https://github.com/python/cpython/pull/28664 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-29 Thread Eric Snow
Eric Snow added the comment: New changeset 45ca1c04139300ec0289a32f78c7ac922a4f7b07 by Eric Snow in branch 'main': bpo-45020: Do not freeze /__init__.py twice. (gh-28635) https://github.com/python/cpython/commit/45ca1c04139300ec0289a32f78c7ac922a4f7b07 --

[issue45020] Freeze all modules imported during startup.

2021-09-29 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +27004 pull_request: https://github.com/python/cpython/pull/28635 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-27 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26972 pull_request: https://github.com/python/cpython/pull/28590 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-27 Thread Barry A. Warsaw
Change by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45020] Freeze all modules imported during startup.

2021-09-27 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26965 pull_request: https://github.com/python/cpython/pull/28583 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-25 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: On 25.09.2021 18:20, STINNER Victor wrote: > > STINNER Victor added the comment: > > Marc-Andre: I suppose that you're talking about LANDMARK in Modules/getpath.c > and PC/getpathp.c. Now that you mention it: yes, that as well :-) But os.py is used in

[issue45020] Freeze all modules imported during startup.

2021-09-25 Thread STINNER Victor
STINNER Victor added the comment: Marc-Andre: I suppose that you're talking about LANDMARK in Modules/getpath.c and PC/getpathp.c. -- ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-25 Thread Guido van Rossum
Guido van Rossum added the comment: The plan is to add __file__ back, based on where the stdlib lives (we won’t do any stat() calls for frozen files). -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-25 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: Eric, I noticed that you are freezing os.py. Please be aware that this module is often being used as indicator for where the stdlib was installed (the stdlib itself does this in site.py to read the LICENSE and the test suite also uses os.__file__ in a

[issue45020] Freeze all modules imported during startup.

2021-09-24 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26937 pull_request: https://github.com/python/cpython/pull/28554 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-24 Thread Eric Snow
Eric Snow added the comment: New changeset 7c801e0fa603b155eab3fd19698aa90854ac5a7b by Eric Snow in branch 'main': bpo-45020: Fix some corner cases for frozen module generation. (gh-28538) https://github.com/python/cpython/commit/7c801e0fa603b155eab3fd19698aa90854ac5a7b --

[issue45020] Freeze all modules imported during startup.

2021-09-23 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26923 pull_request: https://github.com/python/cpython/pull/28538 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-22 Thread Gregory Szorc
Gregory Szorc added the comment: I'll throw out that __file__ is strictly optional, per https://docs.python.org/3/reference/datamodel.html. IMO it is more important to define importlib.abc.InspectLoader than __file__ (so tracebacks have source context). Given that __file__ is optional,

[issue45020] Freeze all modules imported during startup.

2021-09-22 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: On 22.09.2021 20:47, Brett Cannon wrote: > What about if there isn't a pre-computed location for __file__? I could > imagine a self-contained CPython build where there is no concept of a file > location on disk for anything using this. This does work

[issue45020] Freeze all modules imported during startup.

2021-09-22 Thread Brett Cannon
Brett Cannon added the comment: What about if there isn't a pre-computed location for __file__? I could imagine a self-contained CPython build where there is no concept of a file location on disk for anything using this. -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-21 Thread Guido van Rossum
Guido van Rossum added the comment: I'm still torn about the need for a __file__ attribute. Assuming it's more likely that people write code that *reads* stdlib source for some reason (maybe printing more context for error messages) than *writing* it (which would be truly strange), the

[issue45020] Freeze all modules imported during startup.

2021-09-19 Thread STINNER Victor
STINNER Victor added the comment: runpy startup time matters a lot for "python3 -m module" startup time! In Python 3.10 (bpo-41006 and bpo-41718), I reduced the number of modules imported by runpy: "The runpy module now imports fewer modules. The python3 -m module-name command startup

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Raymond Hettinger
Raymond Hettinger added the comment: > Raymond, do you think we should also freeze the dependencies > of runpy (so "python -m " also starts faster)? Yes, please. The '-m' load option can be considered core startup functionality. It is often the recommended way to launch command line

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Guido van Rossum
Guido van Rossum added the comment: > It would be nice to freeze argparse.py and its dependencies. For > command-line tools, startup time is important. I quickly checked, and argparse has at least these dependencies: argparse re enum types operator functools collections keyword reprlib

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Raymond Hettinger
Raymond Hettinger added the comment: It would be nice to freeze argparse.py and its dependencies. For command-line tools, startup time is important. -- nosy: +rhettinger ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Eric Snow
Eric Snow added the comment: On Fri, Sep 17, 2021 at 4:22 PM Guido van Rossum wrote: > BTW, why does the script still run Programs/_freeze_module over all the > modules to be frozen? Isn't that up to the Makefile or its Windows equivalent? Yeah, it used to matter but we probably don't need

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Eric Snow
Eric Snow added the comment: New changeset 090591636c4f03ce06a039079bd7716a5b23631e by Eric Snow in branch 'main': bpo-45020: Freeze os, site, and codecs. (gh-28398) https://github.com/python/cpython/commit/090591636c4f03ce06a039079bd7716a5b23631e --

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Guido van Rossum
Guido van Rossum added the comment: BTW, why does the script still run Programs/_freeze_module over all the modules to be frozen? Isn't that up to the Makefile or its Windows equivalent? -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Guido van Rossum
Guido van Rossum added the comment: (Ah, now I recall how the $(srcdir) mechanism works again. Generated files do *not* have a $(srcdir) prefix, since they are generated in the destination directory. Hence the fix for the buildbot failure a few days ago. I take back my skepticism about

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread Eric Snow
Eric Snow added the comment: Thanks for fixing that, Victor! -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread STINNER Victor
STINNER Victor added the comment: New changeset 41551ee7e24fb6c58846836d3655dbb212281206 by Victor Stinner in branch 'main': bpo-45020: Fix build out of source tree (GH-28410) https://github.com/python/cpython/commit/41551ee7e24fb6c58846836d3655dbb212281206 --

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +26822 pull_request: https://github.com/python/cpython/pull/28410 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread STINNER Victor
STINNER Victor added the comment: There is also a minor issue, "make regen-frozen" changes the end of line of two PCbuild/ files: see bpo-45231. -- ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-17 Thread STINNER Victor
STINNER Victor added the comment: It's no longer possible to build Python from a different directory. Example: git clean -fdx mkdir build cd build/ ../configure --with-pydebug make It fails with: ../Programs/_freeze_module importlib._bootstrap ../Lib/importlib/_bootstrap.py

[issue45020] Freeze all modules imported during startup.

2021-09-16 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26812 pull_request: https://github.com/python/cpython/pull/28398 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-16 Thread Eric Snow
Eric Snow added the comment: New changeset fdc6b3d9316501d2f0068a1bf4334debc1949e62 by Eric Snow in branch 'main': bpo-45020: Drop the frozen .h files from the repo. (gh-28392) https://github.com/python/cpython/commit/fdc6b3d9316501d2f0068a1bf4334debc1949e62 --

[issue45020] Freeze all modules imported during startup.

2021-09-16 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26804 pull_request: https://github.com/python/cpython/pull/28392 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-16 Thread Eric Snow
Eric Snow added the comment: I left a comment on bpo-45220. -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue45020] Freeze all modules imported during startup.

2021-09-16 Thread STINNER Victor
STINNER Victor added the comment: Is bpo-45220 "Windows builds sometimes fail on Azure and GitHub Action: fatal error RC1116: RC terminating after preprocessor errors" related to this issue? -- nosy: +vstinner ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: New changeset 9fd87a5fe5c468cf94265365091267838b004b7f by Eric Snow in branch 'main': bpo-45020: Revert "Drop the frozen .h files from the repo." (gh-28380) https://github.com/python/cpython/commit/9fd87a5fe5c468cf94265365091267838b004b7f --

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26795 pull_request: https://github.com/python/cpython/pull/28380 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: Never mind. It's pretty late here so I'm going to revert it and sort it out in the morning. -- ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Guido van Rossum
Guido van Rossum added the comment: FWIW, Python/frozen_modules/__hello__.h is still in the repo somehow. -- ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: Looks like that last commit broke one of the buildbots: https://buildbot.python.org/all/#/builders/483/builds/812 I'll fix that right away. -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: New changeset a9757bf34d8b4cb3c24bbb70d50a06c815e2e8f3 by Eric Snow in branch 'main': bpo-45020: Drop the frozen .h files from the repo. (gh-28375) https://github.com/python/cpython/commit/a9757bf34d8b4cb3c24bbb70d50a06c815e2e8f3 --

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: On Wed, Sep 15, 2021 at 12:03 PM Guido van Rossum wrote: > I would move "default to "on" (except if actually running out of the source > tree)" to the "maybe" category. I left a few comments in other deps. I think > we should start by turning this on by default

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26790 pull_request: https://github.com/python/cpython/pull/28375 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Guido van Rossum
Guido van Rossum added the comment: I would move "default to "on" (except if actually running out of the source tree)" to the "maybe" category. I left a few comments in other deps. I think we should start by turning this on by default in PGO builds. Separately, I encourage you to collect

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: At this point the fundamental work is done. Here are some follow-up tasks to wrap up this issue: * freeze the remaining stdlib modules imported during startup (os, site, codecs, encodings.*) + blocked by bpo-45186 and bpo-45188 * default to "on" (except if

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Eric Snow
Eric Snow added the comment: New changeset cbeb81971057d6c382f45ecce92df2b204d4106a by Eric Snow in branch 'main': bpo-45020: Freeze some of the modules imported during startup. (gh-28335) https://github.com/python/cpython/commit/cbeb81971057d6c382f45ecce92df2b204d4106a --

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread miss-islington
miss-islington added the comment: New changeset f71b86e0ae194613d235086755c6a44266978be1 by Miss Islington (bot) in branch '3.9': bpo-45020: Don't test IDLE with frozen module. (GH-28344) https://github.com/python/cpython/commit/f71b86e0ae194613d235086755c6a44266978be1 --

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread miss-islington
miss-islington added the comment: New changeset 8a9396cf1d9e1ce558841095e1ce0d3c23b7a8aa by Miss Islington (bot) in branch '3.10': bpo-45020: Don't test IDLE with frozen module. (GH-28344) https://github.com/python/cpython/commit/8a9396cf1d9e1ce558841095e1ce0d3c23b7a8aa --

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread miss-islington
Change by miss-islington : -- nosy: +miss-islington nosy_count: 15.0 -> 16.0 pull_requests: +26758 pull_request: https://github.com/python/cpython/pull/28345 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread miss-islington
Change by miss-islington : -- pull_requests: +26759 pull_request: https://github.com/python/cpython/pull/28346 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-15 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset 369bf949ccbb689cd4638b29b4c0c12db79b927c by Terry Jan Reedy in branch 'main': bpo-45020: Don't test IDLE with frozen module. (GH-28344) https://github.com/python/cpython/commit/369bf949ccbb689cd4638b29b4c0c12db79b927c --

[issue45020] Freeze all modules imported during startup.

2021-09-14 Thread Terry J. Reedy
Change by Terry J. Reedy : -- nosy: +terry.reedy nosy_count: 14.0 -> 15.0 pull_requests: +26756 pull_request: https://github.com/python/cpython/pull/28344 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-14 Thread Eric Snow
Eric Snow added the comment: New changeset a65c86889e208dddb26a7ebe7840c24edbcca775 by Eric Snow in branch 'main': bpo-45020: Add -X frozen_modules=[on|off] to explicitly control use of frozen modules. (gh-28320)

[issue45020] Freeze all modules imported during startup.

2021-09-14 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26745 pull_request: https://github.com/python/cpython/pull/28335 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-13 Thread Eric Snow
Eric Snow added the comment: On Mon, Sep 13, 2021 at 2:59 PM Guido van Rossum wrote: > If you reduce the number of modules being frozen you could probably manage > to land this (or most of it) before tackling those other issues. Yeah, that's what I'm doing. :) --

[issue45020] Freeze all modules imported during startup.

2021-09-13 Thread Eric Snow
Change by Eric Snow : -- pull_requests: +26729 pull_request: https://github.com/python/cpython/pull/28320 ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-13 Thread Guido van Rossum
Guido van Rossum added the comment: If you reduce the number of modules being frozen you could probably manage to land this (or most of it) before tackling those other issues. -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-13 Thread Eric Snow
Change by Eric Snow : -- dependencies: +De-couple the Windows builds from freezing modules., Marshal output isn't completely deterministic. ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-09-04 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: FWIW, I've not found the importer for frozen modules to be lacking features. When using frozen modules, you don't expect to see source code, so the whole part about finding source code is not really relevant for that use case. The only lacking part I

[issue45020] Freeze all modules imported during startup.

2021-09-03 Thread Nick Coghlan
Nick Coghlan added the comment: For the module metadata problem: one potential approach to that for "designed to be frozen" stdlib modules is to set the values directly in the module code, rather than trying to set them automatically in the frozen import machinery. It should also be

[issue45020] Freeze all modules imported during startup.

2021-09-01 Thread Eric Snow
Eric Snow added the comment: On Tue, Aug 31, 2021 at 12:14 PM Brett Cannon wrote: > > set __file__ (and __path__) on frozen modules? > > See https://bugs.python.org/issue21736 Great! I'll take a look. -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-09-01 Thread Eric Snow
Eric Snow added the comment: On Tue, Aug 31, 2021 at 10:05 PM Inada Naoki wrote: > I don't want all frozen header files to be committed in git repository. > Can't we just build them during build process? That's a good point (and an interesting one). Only two of the frozen modules are

[issue45020] Freeze all modules imported during startup.

2021-08-31 Thread Inada Naoki
Inada Naoki added the comment: I don't want all frozen header files to be committed in git repository. Can't we just build them during build process? -- ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-08-31 Thread Eric Snow
Change by Eric Snow : -- keywords: +patch pull_requests: +26549 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/28107 ___ Python tracker

[issue45020] Freeze all modules imported during startup.

2021-08-31 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: On 31.08.2021 20:14, Brett Cannon wrote: > > Brett Cannon added the comment: > >> set __file__ (and __path__) on frozen modules? > > See https://bugs.python.org/issue21736 The patch on that ticket is straight from PyRun, where the __file__ location is

[issue45020] Freeze all modules imported during startup.

2021-08-31 Thread Brett Cannon
Brett Cannon added the comment: > set __file__ (and __path__) on frozen modules? See https://bugs.python.org/issue21736 -- ___ Python tracker ___

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread hai shi
Change by hai shi : -- nosy: +shihai1991 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Eric Snow
Eric Snow added the comment: On Mon, Aug 30, 2021 at 2:22 PM Guido van Rossum wrote: > I propose to only opt in by default in **PGO builds**. After all what we're > doing is another extreme optimization. > > It should always be possible to opt in using some -X flag (e.g. to debug the >

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Guido van Rossum
Guido van Rossum added the comment: FWIW, I'd be okay with doing the -X flag in a separate PR. -- ___ Python tracker ___ ___

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Guido van Rossum
Guido van Rossum added the comment: > At this point, here are the open questions I'm seeing: > + The editing-stdlib-.py-files problem: [...] > + Compatibility: [...] ? + Penalty for too many frozen modules: [...] > FWIW, I think the ideal mechanism for a dev build will be to opt in to

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Guido van Rossum
Guido van Rossum added the comment: [Gregory Szorc] > What do you set __file__ to? [...] Exactly. I think it should not be set, just like it's not set for builtin modules. > I have some observations about the implications of this. I typed up a long > comment but then realized someone

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Guido van Rossum
Guido van Rossum added the comment: > FWIW, I asked Ned Batchelder about this and he said this approach > ("fullcoverage" [1]) was added to support running coverage on the stdlib. > [...] The docs you pointed out in [3] (where it talks about a *horrible hack you should never use" :-)

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Gregory Szorc
Gregory Szorc added the comment: > For stdlib modules it wouldn't be a big problem to set __file__ on > frozen modules. > Would that be enough to solve the problem? What do you set __file__ to? Do you still materialize the .py[c] files on disk and set to that? (This would make the most

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Eric Snow
Eric Snow added the comment: > * tricks to inject hooks ASAP (e.g. coverage.py swaps the encodings module) > may lose their entry point FWIW, I asked Ned Batchelder about this and he said this approach ("fullcoverage" [1]) was added to support running coverage on the stdlib. It doesn't

[issue45020] Freeze all modules imported during startup.

2021-08-30 Thread Eric Snow
Eric Snow added the comment: At this point, here are the open questions I'm seeing: + The editing-stdlib-.py-files problem: * use a command-line flag to opt-out of frozen modules? * use a build flag to opt out (e.g. a configure flag or a new Py_NO_FROZEN or even Py_DEBUG)? * ignore

  1   2   >