bpo-35321: Set the spec origin to frozen in frozen modules#11732
Conversation
|
Are you sure that this PR is related to https://bugs.python.org/issue35537? |
Sorry, something went wrong.
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
February 2, 2019 20:13
|
@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. |
Sorry, something went wrong.
|
I added the backport tags for 3.7 and 3.6, but I'd like to get @ned-deily 's take on that. |
Sorry, something went wrong.
|
@warsaw: Please replace |
Sorry, something went wrong.
|
Sorry, @nnja and @warsaw, I could not cleanly backport this to |
Sorry, something went wrong.
|
Sorry, @nnja and @warsaw, I could not cleanly backport this to |
Sorry, something went wrong.
|
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! |
Sorry, something went wrong.
|
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? |
Sorry, something went wrong.
|
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? |
Sorry, something went wrong.
|
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. |
Sorry, something went wrong.
Sorry, something went wrong.
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