Skip to content

bpo-45019: Do some cleanup related to frozen modules. #28319

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ericsnowcurrently merged 19 commits into from
Sep 13, 2021
Merged

bpo-45019: Do some cleanup related to frozen modules. #28319

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
63cd6f6
Fix freeze_module() in freeze_modules.py.
ericsnowcurrently Sep 2, 2021
0223e32
Add FrozenSource and FrozenModule to freeze_modules.py.
ericsnowcurrently Sep 1, 2021
a2be976
Leave all non-required frozen modules uncommitted.
ericsnowcurrently Sep 2, 2021
7a9d113
Leave *all* frozen modules uncommitted.
ericsnowcurrently Sep 2, 2021
e0f264f
Generate the list of frozen modules used by test_ctypes.
ericsnowcurrently Sep 1, 2021
c4cea0f
Ignore frozen submodules in generate_stdlib_module_names.py.
ericsnowcurrently Sep 7, 2021
1336b4e
Add a comment to the frozen modules manifest file.
ericsnowcurrently Sep 7, 2021
122492a
Show how the frozen manifest changed.
ericsnowcurrently Sep 7, 2021
4c1bc81
Go back to keeping frozen modules in the repo.
ericsnowcurrently Sep 7, 2021
cd99280
Also stop tracking the frozen manifest.
ericsnowcurrently Sep 8, 2021
65a9562
Mark the frozen manifest as a generated file.
ericsnowcurrently Sep 8, 2021
122e642
Drop a superfluous prefix on makefile rule dependencies.
ericsnowcurrently Sep 8, 2021
0704664
Do not generate test code.
ericsnowcurrently Sep 8, 2021
81d70ea
Drop unused code from freeze_modules.py.
ericsnowcurrently Sep 8, 2021
a61afbd
Do not clear the frozen .h files with "make distclean", now that they…
ericsnowcurrently Sep 9, 2021
ca35ee8
Always flush the printed "title" when freezing modules.
ericsnowcurrently Sep 10, 2021
1bf4d2f
Add the frozen manifest back into the repo.
ericsnowcurrently Sep 10, 2021
bd129b5
On Windows, only freeze the essential modules for now.
ericsnowcurrently Sep 13, 2021
20d3b50
Drop an outdated call.
ericsnowcurrently Sep 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Drop an outdated call.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Sep 13, 2021
commit 20d3b506b0c64be2569e6a96a70d104941535229
3 changes: 1 addition & 2 deletions Lib/ctypes/test/test_values.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,7 @@ class struct_frozen(Structure):
self.assertGreater(abs(entry.size), 10)
self.assertTrue([entry.code[i] for i in range(abs(entry.size))])
# Check the module's package-ness.
with import_helper.frozen_modules(), captured_stdout():
spec = importlib.util.find_spec(modname)
spec = importlib.util.find_spec(modname)
if entry.size < 0:
# It's a package.
self.assertIsNotNone(spec.submodule_search_locations)
Expand Down