Skip to content

catch mintty error and add help for it#3537

Merged
vilmibm merged 5 commits intotrunkfrom
incorrect-function
Apr 29, 2021
Merged

catch mintty error and add help for it#3537
vilmibm merged 5 commits intotrunkfrom
incorrect-function

Conversation

@vilmibm
Copy link
Contributor

@vilmibm vilmibm commented Apr 29, 2021

This PR notices when MinTTY fails to gather input with "Incorrect function" and then suggests the user run "gh help mintty" to learn about workarounds.

I've tested each workaround (including running git+bash via Windows Terminal) on my Windows machine.

Ref. #404

@vilmibm vilmibm requested review from mislav and samcoe April 29, 2021 16:05
Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great. Thank you!

Be sure to check the box "Enable experimental support for pseudo consoles".

- Use a different terminal emulator with Git for Windows like Windows Terminal or cmd.exe.
Try running "C:\Program Files\Git\bin\bash.exe" in one of those other terminal emulators.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, is this really the only way to consistently open Git Bash from Windows Terminal? Isn't there some way to create a shortcut of some kind so that people don't have to type or paste the full path?

Copy link
Contributor Author

@vilmibm vilmibm Apr 29, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can set up a Windows Terminal profile to always run bash.exe, set up a shortcut, or add to PATH but all of those strategies seemed out of scope for this help topic.

nate smith added 2 commits April 29, 2021 16:01
@vilmibm vilmibm enabled auto-merge April 29, 2021 21:07
@vilmibm vilmibm merged commit 6b9a860 into trunk Apr 29, 2021
@mislav mislav deleted the incorrect-function branch May 2, 2021 09:06
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

Successfully merging this pull request may close these issues.

2 participants