Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

FIX CMDER_ROOT for admin launch #311

Merged
merged 1 commit into from
Mar 18, 2015

Conversation

sescandell
Copy link
Contributor

When launching a cmd in admin mode, if %ConEmuDir%\..\.. contains any space, it breaks the init script.
Thanks to this patch, the path is correctly defined now.

When launching a cmd in admin mode, if %ConEmuDir%\..\.. contains any space, it breaks the scripts.
Thanks to this patch, the path is correctly defined now.
@killua99
Copy link

Maybe you don't need this comment but. Applying this change works with externals tools like PhpStorm when I setup the console with cmder

MartiUK added a commit that referenced this pull request Mar 18, 2015
@MartiUK MartiUK merged commit 7169f49 into cmderdev:development Mar 18, 2015
@FrankPeters
Copy link

@killua99 Could you explain how you added Cmder to PhpStorm to use as Terminal? When I use C:\cmder\Cmder.exe as the Shell path, the Terminal window opens and then closes but Cmder opens up in a separate window (but not in the correct directory).

@sescandell sescandell deleted the fix-adminlaunch branch April 2, 2015 07:35
@killua99
Copy link

killua99 commented Apr 7, 2015

Hi, I was on holidays I forgot to reply. I set on the terminal "Shell path"

cmd /k "C:\opt\cmder\vendor\init.bat"

Hope it work for you.

@FrankPeters
Copy link

Hey @killua99 it worked however the prompt is pretty awkward, looks corrupt or something. Well, I'll just continue with a separate instance on my 2nd monitor then, thanks for the reply! 👍

@killua99
Copy link

killua99 commented Apr 8, 2015

You must to be sure you had this patch in your init.bat file. If not it look broken

@FrankPeters
Copy link

Oh I see, it's not yet in stable release. Great, then I'll try it again when it's merged into master. Thanks!

@zerosurfer
Copy link

@killua99 I have the same issue , it looks awkward.
image

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants