-
-
Notifications
You must be signed in to change notification settings - Fork 33.9k
bpo-35321: Set the spec origin to frozen in frozen modules #11732
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.
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
Conversation
|
Are you sure that this PR is related to https://bugs.python.org/issue35537? |
This fix correctly sets the spec origin to "frozen" for the _frozen_importlib module. Note that the origin was already correctly set in _frozen_importlib_external.
172f816 to
eb3dac6
Compare
|
@vstinner drat, copied and pasted the wrong issue number. It's actually https://bugs.python.org/issue35321. I updated the commit message, GitHub issue title & message. |
|
Thank you @nnja for this PR! |
warsaw
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @nnja - LGTM.
|
I added the backport tags for 3.7 and 3.6, but I'd like to get @ned-deily 's take on that. |
|
@warsaw: Please replace |
|
Sorry, @nnja and @warsaw, I could not cleanly backport this to |
|
Sorry, @nnja and @warsaw, I could not cleanly backport this to |
|
AFAICT this is not a 3.6 regression: the behavior has been around for a long time. Even though it is minor and the risk of breakage is small, the benefit seems to be also small. My take is that this doesn't meet the criteria for a security-fix-only branch so it should not go into 3.6 or earlier. Sorry! |
|
Thank you @ned-deily - we'll just back port this to 3.7. |
|
I have a question about how to backport this into 3.7. (maybe @warsaw or @ned-deily can help?) I used the As expected, this file has irresolvable conflicts. What's the process for this type of backport? |
|
I'm not sure this is the right way to do it (does the devguide provide any help?). What I generally do is to use the tool to do the merge in the older branch, then revert I wonder if @Mariatta has any thoughts on adding this to @miss-islington by default? |
|
you can resolve the conflict manually (regenerate the files) and the do Currently miss-islington can't do this automatically. There is open ticket here: python/miss-islington#41. |
This fix correctly sets the spec origin to
"frozen" for the _frozen_importlib module. Note that the
origin was already correctly set in _frozen_importlib_external.
https://bugs.python.org/issue35321