-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Unable to run in Windows XP if path contains spaces #67
Comments
I ran into the same issue on my Windows 7 machine. I have a fix in a local fork, and will be submitting a pull request shortly. Update: I see an open pull request (#65) from @kjerk which makes all the changes I had in my own fork. I believe if that pull request is accepted, your issue should be resolved. |
I believe this should not be tagged "legacy" because I have this exact issue with Windows 8.1. |
I am packing |
@Bliker - tested 1.1.2, new cmder.exe launcher complains about lack of MSVCP120.dll, won't start. |
Oh damn, thanks, I guess there is a need to distribute that also. |
Tested v1.1.2 as well, the launcher is not recognized as a valid Win32 application. |
I managed to get a quick intro windows .dll files: does this work for you? Please say yes :-D |
I am sorry, but at work I cannot access Dropbox, so I cannot download the files. At home I use OS X, so the best I can do is download from home and test the new release tomorrow. I'll let you know. |
As a piece of good news, I finally was able to make the previous release, v1.1, to run from a path with spaces.
and the one to
Now, if it wasn't for the missing prompt, and the fact that clink crashes, it would be perfect... |
Actually, it kind of works! :) A vast improvement. The app starts without The command executes, but it looks like something's wonky with the path env On 11 December 2013 14:00, Samuel Vasko [email protected] wrote:
|
can you type |
λ path PATH=D:\My Files\Created\Today\cmder The system cannot find the path specified. On 11 December 2013 17:18, Samuel Vasko [email protected] wrote:
|
Tested the version downloaded from Dropbox. Again the launcher is not recognized as a valid Win32 application. |
@galli-a That is rather strange, can you give me your computer specs and md5 (or sha) of the .exe? Thanks |
mine is md5: |
The md5 I get is I tested both on a Windows XP 32bit with SP3 (Intel core i5), and on a Windows XP Professional x64 (Intel Xeon), and on both I get the same error: is not a valid Win32 application. |
I think WIndows XP is the critical part. I will take a look. |
Nope, having the same trouble on Windows 7 x64. I tried running cmder from different directories : C:\Program Files\cmder\ causes an error because of the space in "Program Files" %PATH% : So, it appears that only the first folder causes trouble. Any spaces after that are OK. |
@Bliker - for me it works fine now, the path error message had nothing to do with Cmder. Are you going to link to 1.1.2 from the Cmder homepage? Or is it not considered ready yet? (Asking because am considering a possible review.) |
I'd agree with @Bliker, the issue is on Windows XP. I got access to a Windows 7 machine, and tested there. It works fine, regardless of spaces in the path to the launcher. |
@ezuk I was waiting for confirmation about new launcher and dependencies. All seems to be clear right now. So in a few days... |
On Windows XP, the
Cmder.bat
file cannot be executed if the full path contains spaces. It does run succesfully if executed on a path without spaces.The text was updated successfully, but these errors were encountered: