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

report successfully installed versions of requirements even if existi… #3723

Merged
merged 3 commits into from
Oct 27, 2016

Conversation

BrownTruck
Copy link
Contributor

…ng installations have been ignored

currently, the "Successfully installed ..." message of a pip install can report a wrong version when either the -I or the --target options are in use: when another installation of the package exists on sys.path and the new installation goes to a location not on sys.path or in a later position, the pre-existing version will be reported. The patch fixes this for me.

This change is


This was automatically migrated from #3612 to reparent it to the master branch. Please see original pull request for any previous discussion.

Original Submitter: @wm75

@xavfernandez xavfernandez added this to the 8.2 milestone Sep 29, 2016
@xavfernandez xavfernandez reopened this Oct 27, 2016
@xavfernandez xavfernandez merged commit 1aa5e6a into pypa:master Oct 27, 2016
@xavfernandez
Copy link
Member

@wm75 Thanks a lot for your PR !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants