-
Notifications
You must be signed in to change notification settings - Fork 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
Using pip launcher on Windows when path contains spaces #1997
Comments
Hi, I managed to solve the issue by removing the quotes in the "ScriptMaker" used to generate the exe launchers. The problem is that Python is not working well when reading paths with quotes in the "shebang" section of the exe launchers. Here's the proof of concept:
|
Using spaces in Windows is common. This patch fixes an issue in the exe launch wrappers generated in the "Scripts" directory where unnecessary quoting causes the following error. Fatal error in launcher: Unable to create process using '""C:\Program Files (x86)\Cloudbase Solutions\OpenStack\Nova\Python27\python.exe"" "C:\Program Files (x86)\Cloudbase Solutions\OpenStack\Nova\Python27\Scripts\nova.exe" ' Closes issue: pypa#1997
I have a feeling this was fixed recently in distlib, and vendored into pip. Could you check if the issue still exists with the development version of pip from github? |
Did some quick tests and it worked. Closing the pull request by now, I'll investigate further and reopen it in case. Thanks! |
Did some further investigations, the issue was in the "t32.exe" launcher and has been fixed by this patch in distlib: IMO the issue can be considered closed. |
Still experiencing the problem when my pip had no spaces: |
Comfirmed this on Windows 7 with Python 3.5. My pip is in |
@laike9m could you describe the exact error you're getting? |
|
Confirmed for Windows 10 as well. No spaces. I am running it within a virtual environment, but that shouldn't be an issue. Fatal error in launcher: Unable to create process using '"' |
Same here and no virtual environment, windows 8.1 [edit] my path was polluted by a python version which was not properly uninstalled. It works with pyton3.4 anaconda |
pip PIP_FIND_LINKS env variable dosen't support the windows space, and it resulted in a virtualenv bugs. Because virtualenv can't install setuptools, wheel through the pip installed in C:/Program files/Python27 when I create an virtual evironment. I think pip should support the double " , like windows cmd, such as And use the space to split PIP_FIND_LINKS into different paths which is not a good choice. |
Just found this workaround http://stackoverflow.com/a/26428562/670565 hope it will be useful to someone 👍 |
It's not only workaround. Sometimes it's preferred to call pip using |
Same shiat here when installing python with chocolatey. Here's output in cmder after
A workaround for this issue is overriding --installargs (short -ia): |
Windows 10, x86. I also have Python 3.5.1 (local user install, username does not have spaces in path), it had pip 7.1.2, which had this problem. Once I upgraded to pip 8.1.2, I was able to run: This issue's summary is pretty vague (it's missing "what's wrong"), the long description points to a stackoverflow question with a bunch of things. I'd suggest a maintainer clarify this issue summary to be limited to a single problem. If that problem is pip 7.1.2 and below yielding: |
Closing this, I can't reproduce this. If you're still having issues please open a new issue with reproduction steps. It appears this was fixed in distlib. |
Steps to reproduce:
That's it reproduced |
There is a problem when trying to run pip.exe when the path of Python installation contain spaces. Some launcher code adds extra quotes, which results in incorrect path and results in fatal error in launcher. See for example http://stackoverflow.com/questions/24627525/fatal-error-in-launcher-unable-to-create-process-using-c-program-files-x86. I understand that this is probably issue of the code producing the launcher (there is the same problem with IPython launcher), but I'm not sure where to report it.
The text was updated successfully, but these errors were encountered: