Split the pip self-version check to get info before run#13923
Conversation
195cca2 to
af592ca
Compare
0ee71fd to
946120b
Compare
|
Thanks @ichard26 , I believe I've addressed all your issues. Thanks for catching the dead code, I'd tried a few different approaches and apparently hadn't done a thorough enough clean up. |
|
I won't have time to review this before the release. Do you still want it in 26.1? How risky is it? |
Yes.
Touching the self update check code has historically reared unexpected issue. I've done my absolute best to validate all the issues we've seen in the past (different upgrade/downgrade paths). Frankly though, if this PR is considered too risky I think we should disable the self update check, at least for 26.1, it's a one-line change. |
Given the core logic is wrapped defensively in a try except, I think it'll be fine...? I tested how it'd handle non-standard indices and an no/bad internet connection and it seemed fine. Also, the last issue we had with the self check involved pip replacing itself. Given we're now blocking it from running whenever a |
|
I did an extra pass on validating this doesn't create any new errors, I did find some existing bugs which I will put on my to do list as a follow up, but nothing new. |
|
|
Do you have a specific issue? If so please open a new issue.
|
|
Does the copy of pip that ships with Python need to be updated too? |
|
Yes, but that will be occur at a future date. #13795 will contain the most up to date details. |
This means that everything that needs lazily importing is done before installation happens, while still not importing the network stack for uninstall and list (#12637)