-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
allow use of not-embedded pip #6062
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). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 update Fails 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.