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

Weird interaction with git add -p and Notepad++ (git hangs) #307

Closed
klylesatepic opened this issue Jan 18, 2024 · 2 comments
Closed

Weird interaction with git add -p and Notepad++ (git hangs) #307

klylesatepic opened this issue Jan 18, 2024 · 2 comments

Comments

@klylesatepic
Copy link

The following is mostly a copy of a Windows Terminal issue that I wrote up first. After some investigation, they pointed me to msys2, so here I am.

I didn't see a template to use here, so I'll stick with theirs. Please let me know if there's more information that would be helpful -- I'm happy to dig in further.

Windows Terminal version

1.18.3181.0

Windows build number

10.0.19044.3803
10.0.22621.2861

Other Software

Git (both versions 2.42.0.windows.2 and 2.43.0.windows.1)
Notepad++ (version 8.6)

Steps to reproduce

  1. Open Git Bash in Windows Terminal
  2. Navigate to a git repository
  3. Make changes in at least two separate places in a file
  4. Run GIT_EDITOR='"C:\Program Files\Notepad++\notepad++.exe" -multiInst -notabbar -nosession -noPlugin' git add -p to interactively stage parts of the file
    • Obviously you will need Notepad++ installed, and you may need to adjust the path to it
  5. Type e to edit the first hunk (Notepad++ will open)
  6. Make whatever change you like within one of the new lines (starting with a plus)
  7. Save and close Notepad++
  8. See that git moves on and prompts for the next hunk
  9. Try to type n to skip the next hunk (or anything else)
  10. See that git is stuck (Ctrl+C won't interrupt it)

The same issue happens with git add -i instead of git add -p, but requires more steps between steps 4/5.

The issue does not happen if any of the following are true:

  • You use vanilla Git Bash instead of Git Bash in Windows Terminal
  • You use cygwin instead of Git Bash in Windows Terminal (with a slightly different GIT_EDITOR value to account for differences in path handling)
  • You only edit the last hunk
  • You use a terminal editor instead of Notepad++ (I tried vim and nano)
  • You use /c/Windows/System32/notepad.exe instead of Notepad++
    • The bug still happens if you just use 'notepad', which is a shell script in Git Bash located at /usr/bin/notepad

Expected Behavior

Step 9 above would echo the character you typed, and git would not get stuck.

Actual Behavior

The character(s) you type are not echoed, nothing happens even after also pressing Enter, and even Ctrl+C will not interrupt git.

If you kill the git process with Task Manager, you do come back to a seemingly-working terminal.

@dscho
Copy link
Contributor

dscho commented Jan 25, 2024

This sounds like a Git for Windows issue, but maybe the problem lies in the MSYS2 runtime. Best report it over at https://github.com/git-for-windows/git/issues/new (I will try to find time to look at the bug, promise!).

@klylesatepic
Copy link
Author

Closing in favor of the Git for Windows issue

@klylesatepic klylesatepic closed this as not planned Won't fix, can't repro, duplicate, stale Jan 25, 2024
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

2 participants