Skip to content
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

bpo-33169: Remove values of None from sys.path_importer_cache when invalidating caches #6402

Merged
merged 5 commits into from
Apr 6, 2018

Conversation

brettcannon
Copy link
Member

@brettcannon brettcannon commented Apr 6, 2018

An entry of None in sys.path_importer_cache represents a negative/missing finder for a path.

https://bugs.python.org/issue33169

@brettcannon brettcannon merged commit 9e2be60 into python:master Apr 6, 2018
@miss-islington
Copy link
Contributor

Thanks @brettcannon for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@brettcannon brettcannon deleted the invalidate-caches-None branch April 6, 2018 23:10
@miss-islington
Copy link
Contributor

Sorry, @brettcannon, I could not cleanly backport this to 3.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 9e2be60634914f23db2ae5624e4acc9335bf5fea 3.7

brettcannon added a commit to brettcannon/cpython that referenced this pull request Apr 6, 2018
… when invalidating caches (pythonGH-6402)

An entry of None in sys.path_importer_cache represents a negative/missing finder for a path, so clearing it out makes sense..
(cherry picked from commit 9e2be60)

Co-authored-by: Brett Cannon <[email protected]>
@bedevere-bot
Copy link

GH-6403 is a backport of this pull request to the 3.7 branch.

brettcannon added a commit that referenced this pull request Apr 7, 2018
… when invalidating caches (GH-6402) (GH-6403)

An entry of None in sys.path_importer_cache represents a negative/missing finder for a path, so clearing it out makes sense.
(cherry picked from commit 9e2be60)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants