Skip to content

gh-89559: Change -X frozen_modules to -X frozen_stdlib. - #29301

Closed
ericsnowcurrently wants to merge 7 commits into
python:mainfrom
ericsnowcurrently:frozen-modules-change-x-option
Closed

gh-89559: Change -X frozen_modules to -X frozen_stdlib.#29301
ericsnowcurrently wants to merge 7 commits into
python:mainfrom
ericsnowcurrently:frozen-modules-change-x-option

Conversation

@ericsnowcurrently

@ericsnowcurrentlyericsnowcurrently commented Oct 28, 2021

Copy link
Copy Markdown
Member

With gh-28778python -X frozen_modules only applies to stdlib modules, rather than also affecting custom frozen modules (see Tools/freeze). So we're changing the name to avoid any confusion.

This -X option was added recently and hasn't gone out in any releases, so renaming it shouldn't be a problem.

(See #28633 (comment).)

https://bugs.python.org/issue45396

@ericsnowcurrently
ericsnowcurrentlyforce-pushed the frozen-modules-change-x-option branch from 1f1679c to 981e351CompareNovember 4, 2021 16:28
@brettcannon
brettcannon removed their request for review November 5, 2021 22:07
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actionsgithub-actionsBot added the stale Stale PR or inactive for long period of time. label Dec 6, 2021
@erlend-aaslanderlend-aasland added the pending The issue will be closed if no feedback is provided label Jul 27, 2022
@erlend-aasland

Copy link
Copy Markdown
Contributor

Marked as pending close, since the linked issue is closed.

@erlend-aaslanderlend-aasland changed the title bpo-45396: Change -X frozen_modules to -X frozen_stdlib.gh-89559: Change -X frozen_modules to -X frozen_stdlib.Jul 27, 2022
@erlend-aaslanderlend-aasland removed the pending The issue will be closed if no feedback is provided label Jul 27, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core reviewskip newsstaleStale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@ericsnowcurrently@erlend-aasland@the-knights-who-say-ni@ezio-melotti@bedevere-bot