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

Revert "Merge pull request #12637 from ichard26/refactor/imports" #12676

Closed
wants to merge 2 commits into from

Conversation

ichard26
Copy link
Member

@ichard26 ichard26 commented May 6, 2024

This reverts commit 71a08a7, reversing changes made to 22142d6.

One possible fix for #12675.

This patch made the pip self-version check importpip._internal.self_outdated_check on demand. This blows up when downgrading (and possibly upgrading) pip as it imports the module after the downgrade, which is problematic as the older self-check logic is incompatible with the rest of the pip already loaded into memory.

This reverts commit 71a08a7, reversing
changes made to 22142d6.

This patch made the pip self-version check import
`pip._internal.self_outdated_check` on demand. This blows up when
downgrading (and possibly upgrading) pip as it imports the module
_after_ the downgrade, which is problematic as the older self-check logic
is incompatible with the rest of the pip already loaded into memory.
@ichard26 ichard26 closed this May 9, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2024
@ichard26 ichard26 deleted the revert-lazy branch July 10, 2024 23:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant