allow use of not-embedded pip#6062
Conversation
|
seeing #6205 reminded me of this one. Certainly this doesn't bring such large gains as are claimed there but I reckon it does take about 50% off a typical "poetry install" (with warm caches, downloaded artifacts). |
neersighted
left a comment
There was a problem hiding this comment.
I'm fine with this, but I'd like to hear from a few others.
|
I thought, it might make a difference when updating pip as a locked dependency (at least on Windows). But it seems it makes no difference if the embedded pip or the pip inside the venv is used. I always get the following error: So I'm fine with it, too. |
|
✨ This is an old work account. Please reference @brandonchinn178 for all future communication ✨ Note: it seems like this PR fixed a bug in 1.2.0 where an upgrade to Repro: docker run --rm -it python:3.10 bash
# in docker
python3 -m venv venv
venv/bin/pip install poetry==1.2.0 virtualenv==20.16.5
venv/bin/poetry self updateFails with: Opening an issue doesn't seem appropriate since this is now fixed, but maybe a test should be added to ensure this doesn't regress? |
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Fixes #6060
The code still falls back to embedded pip if no other pip can be found - see the implementation of
Env.pip().(Tests change because we save ourselves a call to
get_embedded_wheel()which causes a python script to be run to get environment info.)I realise that the discussion in #6060 hasn't yet reached a conclusion but this is trivial enough that we might as well have it available.