-
Notifications
You must be signed in to change notification settings - Fork 185
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
crash when starting user command if used MINTTY_... variable contains tab char? #1269
Comments
Note that option UserCommands is deprecated - does it also happen with CtxMenuCommands? |
Also, as long as I can't reproduce this, please try to strip down the test case further:
|
Thanks for looking into this! First, good news: CopyTab optionI have narrowed the issue down to a ~/.minttyrc with only these two lines:
mintty only crashes with the CopyTab=yes option. AnswersBefore I found out about CopyTab, I already wrote down answers to your questions. Maybe you don't need them anymore, but here they are:
Yes.
But all that would equally apply to note 1 "Same with printf 'a b\n' (space instead of tab) works". By that I meant: The new mintty instance starts and runs less with the expected contents. So my PATH is not the problem, and I should probably just have left out note 2, which is probably irrelevant. In the meantime, I have verified with a deliberately (not by my test script) startet WSL bash, that Cygwin pstree does not show WSL bash, so the bash inside the user command is really Cygwin.
"CYGWIN_NT-10.0-22631 <my hostname> 3.5.1-1.x86_64 2024-02-27 11:54 UTC x86_64 Cygwin"
No. This mintty-usercmd-test is also sufficient to crash:
Yes.
No. Same behavior without it. That was just the separator I usually use when I have multiple user commands configured.
No. This mintty-usercmd-test is also sufficient to crash:
Still crashes with
Yes. I checked with
I have built it, but Windows Security does not let me start it yet. I probably will in a few days, though. |
Thanks. And the extra script isn't needed either, the following config is sufficient:
|
I confused config features here. But actually |
Released 3.7.2. |
For a long time, I sometimes had mintty crash when invoking a user command. Now I have finally narrowed the problem down to write a proper bug report.
mintty version: 3.7.1 (x86_64-pc-cygwin) [Windows 22631]
Preparation:
Put this in an executable script called mintty-usercmd-test in $PATH:
Set
UserCommands=�&test:mintty-usercmd-test
(first char after "=" is Ctrl a) andMenuMenu=u
in ~/.minttyrc.Steps to reproduce:
c:\cygwin\bin\mintty.exe -e env HOME=/tmp bash
printf 'a\tb\n'
⇒ mintty crashes.
Notes
printf 'a b\n'
(space instead of tab) works.bash
command might not be the intended Cygwin bash. But I think it is, becausepstree -ha
showsThe text was updated successfully, but these errors were encountered: