-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Fix for #373 #4495
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
Fix for #373 #4495
Conversation
d02acd6
to
501b774
Compare
@NiklasRosenstein Thanks for checking! Could you provide a set of steps to reproduce the issue? I'd like to add a test for this behaviour to ensure it doesn't break again in the future. |
I've tried to reproduce the issue on Ubuntu 16.04 on Python 3.5.2 and Pip 9.0.1, but wasn't able to. I can only reproduce it on Windows 10 using the same Python and Pip versions. Again, using this command
The installation succeeds after upgrading to Pip with your patch.
|
Awesome! Thanks a lot! I don't have a Windows 10 machine handy. :/ I'll add a test based on what you describe. I'm hoping I can have it breaking on master and passing on this branch. I guess I'll use AppVeyor. |
@dstufft Can this be merged without a test? |
I hope to resolve #373 in this PR.