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

2.1.3 breakes conemu #466

Closed
fhoshino opened this issue Jul 31, 2015 · 5 comments
Closed

2.1.3 breakes conemu #466

fhoshino opened this issue Jul 31, 2015 · 5 comments

Comments

@fhoshino
Copy link

Mintty updated to 2.1.3, conemu will fail to capture the window due to spawning of process.
2.1.2 works fine with conemu.

Maximus5/ConEmu#239

@Maximus5
Copy link

The problem was with daemonizing. Well, adding -d switch will solve the problem, but...

  1. This switch leads to an error if is used with previous mintty builds.
  2. It's wrong, I believe, to let users to add -d switch manually. Damn, nobody reads docs.
  3. I can't add the switch automatically because of (1).

So, may be you will not do forking/daemonizing if your app was started from ConEmu? Easiest way to check this - environment variable: http://conemu.github.io/en/ConEmuEnvironment.html#List_of_environment_variables_set_by_ConEmu for example ConEmuPID.

@mintty
Copy link
Owner

mintty commented Jul 31, 2015

Disable the process-spawning with command line option -d.
The spawning was introduced to ensure signals are being delivered within mintty, which would otherwise not happen if started from a console window. I have no idea whether being invoked from ConEmu could be detected automatically, thus the option will likely be needed in future.
Also, please tell me whether in mintty -d, or older mintty, signals work (e.g. Control-C interrupts).

@Maximus5
Copy link

Ctrl-C interrupts "ls -R" properly at least when mintty is running in ConEmu tab. Either old mintty build, or new one with -d switch doesn't matter, all work.

Checking environment variable, as I've said, is the simplest way.

If you need test, you may download ConEmu, unpack it and run ConEmu.exe -basic -cmd c:\cygwin\bin\mintty.exe -

@Maximus5
Copy link

Maximus5 commented Aug 1, 2015

Link to cygwin topic: http://comments.gmane.org/gmane.os.cygwin/154167

@mintty
Copy link
Owner

mintty commented Aug 3, 2015

As suggested, the workaround checks for $ConEmuPID although that's not strictly a safe indication; but starting an "outside" mintty from there works as well, so that should be fine.

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

No branches or pull requests

3 participants