Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Suggestion: respect default terminal emulator #18

Closed
polygonica opened this issue Aug 5, 2014 · 5 comments
Closed

Suggestion: respect default terminal emulator #18

polygonica opened this issue Aug 5, 2014 · 5 comments

Comments

@polygonica
Copy link

Users using ConEmu can configure it to be the system's default terminal emulator. I think it would be a good idea for the "Git Bash" Cheetah option to respect this choice.

@kusma
Copy link
Member

kusma commented Aug 5, 2014

What does "the system's default terminal emulator" even mean? Windows doesn't really have a concept of replaceable terminal emulators, so this sounds like ConEmu doing something to divert calls to cmd.exe (or something similar). Git Bash doesn't start cmd.exe, so I don't think such a concept would even work, and even if it would it'd probably be an issue with ConEmu's implementation of that concept.

@dscho dscho closed this as completed Aug 5, 2014
@dscho
Copy link
Member

dscho commented Aug 5, 2014

Having said that, I could imagine that somebody motivated enough (@polygonica, this is a hint for you!) could make it configurable which terminal emulator is offered.

@polygonica
Copy link
Author

I did some digging around in the source. It looks like create_bash_argv in explorer/menu.c handles the command for the creation of the new window and pulls the cmd it uses from get_cmd. Is there a particular reason the command is "start sh" and not just "sh"? Forcing the creation of a new window with start seems to trip up ConEmu but doesn't seem to have any purpose in the stock setup.

(Removing one parameter is simpler than allowing user configuration)

@kusma
Copy link
Member

kusma commented Aug 5, 2014

There's a comment at the top of that function saying "start is required because exec_program does not create a window", originating from 915e185.

@polygonica
Copy link
Author

Missed that -- thanks. I'll see if I can patch configuration in then.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants