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

Launcher does not escape whitespace in set env variables. #728

Closed
MartiUK opened this issue Nov 24, 2015 · 4 comments
Closed

Launcher does not escape whitespace in set env variables. #728

MartiUK opened this issue Nov 24, 2015 · 4 comments

Comments

@MartiUK
Copy link
Member

MartiUK commented Nov 24, 2015

This is a somewhat old bug IIRC, ideally we wouldn't want users sticking anything into their Program Files since Microsoft has taken considerable effort to lock it down; even for Administrator users (for good reason) but there will be other cases where someone will have a directory containing whitespace.

Can be reproduced by putting cmder into a directory with whitespace e.g. C:\Program Files\.

Since CMDER_ROOT et al are set in the launcher, this would be the best place to resolve it, but it probably could be fixed in the startup scripts instead.

Related Issues:

#727

Workarounds:

Don't put cmder in a directory containing whitespace

@Blasz
Copy link

Blasz commented May 21, 2016

This issue is still happening for bash tasks:

'C:\space' is not recognized as an internal or external command,
operable program or batch file.


Current directory:
C:\Users\Michael

Command to be executed:
"C:\Windows\system32\cmd.exe"  /c "C:\space folder\cmder\vendor\conemu-maximus5\..\git-for-windows\bin\bash --login -i"


ConEmuC: Root process was alive less than 10 sec, ExitCode=1.
Press Enter or Esc to close console...

@DanHarman
Copy link

Just hit this issue trying to change my bash task to use git for windows bash installed in the default location (C;\Program Files\Git\bin\bash". Is there a workaround?

@DanHarman
Copy link

Ok found the old standby of using Progra~1 works fine :)

@Stanzilla Stanzilla removed the C++ label Apr 20, 2018
@daxgames
Copy link
Member

daxgames commented Dec 2, 2018

Works with current version. Closing

@daxgames daxgames closed this as completed Dec 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants