Skip to content

gh-109515: Allow a large number of frozen modules on Windows #109516

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
zooba merged 21 commits into from
Oct 30, 2023
Merged

gh-109515: Allow a large number of frozen modules on Windows #109516

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
97c9b8c
fix windows command line limitations when freezing many modules
rghe Sep 9, 2023
5d640cb
Merge branch 'python:main' into allow-large-number-of-frozen-modules-…
rghe Sep 17, 2023
ddf9082
fix spacing
rghe Sep 17, 2023
9058b8d
📜🤖 Added by blurb_it.
blurb-it[bot] Sep 17, 2023
e98c3d0
missing regen
rghe Sep 17, 2023
86d1818
add generated file
rghe Sep 17, 2023
ca98aa1
add buildpath-dependent generated files to .gitignore
rghe Sep 17, 2023
5463e79
try relative paths
rghe Sep 17, 2023
7456fe5
fill mappings file with relative paths from vcxproj
rghe Sep 21, 2023
031ea3e
Implement S.Dower review suggestions
rghe Sep 23, 2023
e6fb7f5
Merge branch 'main' into allow-large-number-of-frozen-modules-vstudio
rghe Sep 23, 2023
b78bb97
clean gitignore
rghe Sep 23, 2023
93750fa
Merge branch 'main' into allow-large-number-of-frozen-modules-vstudio
rghe Sep 24, 2023
d584d2d
Update Tools/build/freeze_modules.py
rghe Sep 26, 2023
48112ab
Update Tools/build/deepfreeze.py
rghe Sep 26, 2023
74a255a
Update Tools/build/deepfreeze.py
rghe Sep 26, 2023
ecc334f
Update Tools/build/deepfreeze.py
rghe Sep 26, 2023
bb42f32
expand \t and align vcxfile by AA-Turner suggestion
rghe Sep 26, 2023
4cc3458
typo
rghe Sep 26, 2023
1c20fcd
reformat vcxproj
rghe Sep 27, 2023
5620f94
Update Tools/build/deepfreeze.py
rghe Sep 27, 2023
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
Next Next commit
Update Tools/build/freeze_modules.py 
Co-authored-by: Steve Dower <steve.dower@microsoft.com>
  • Loading branch information
@rghe @zooba
rghe and zooba authored Sep 26, 2023
commit d584d2d2d590ba145218a3f379ed1fa884b98d81
2 changes: 1 addition & 1 deletion Tools/build/freeze_modules.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -662,7 +662,7 @@ def regen_pcbuild(modules):
filterlines.append(f' <None Include="..\\{pyfile}">')
filterlines.append(' <Filter>Python Files</Filter>')
filterlines.append(' </None>')
deepfreezemappings.append(f'\t<FrozenModule Include="$(PySourcePath)\\{header}" FrozenId="{src.frozenid}" />\n')
deepfreezemappings.append(f' <FrozenModule Include="$(PySourcePath)\\{header}" FrozenId="{src.frozenid}" />\n')

@AA-Turner AA-Turner Sep 27, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this would solve Steve's point re indentation

Suggested change
deepfreezemappings.append(f' <FrozenModule Include="$(PySourcePath)\\{header}" FrozenId="{src.frozenid}" />\n')
deepfreezemappings.append(f' <FrozenModule Include="$(PySourcePath)\\{header}" FrozenId="{src.frozenid}" />\n')

@rghe rghe Sep 27, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reformatted the vcxproj, now everything seems aligned


corelines.append(f' <ClCompile Include="..\\Python\\deepfreeze\\deepfreeze.c" />')

Expand Down