Problems using native editors via $env:EDITOR with Git for Windows.

[rkitover]

Setup

• Which version of Git for Windows are you using? Is it 32-bit or 64-bit?

git version 2.34.1.windows.1
cpu: x86_64
built from commit: 2ca94ab318509b3c271e82889938816bad76dfea
sizeof-long: 4
sizeof-size_t: 8
shell-path: /bin/sh
feature: fsmonitor--daemon

• Which version of Windows are you running? Vista, 7, 8, 10? Is it 32-bit or 64-bit?

Microsoft Windows [Version 10.0.22526.1000]

• What options did you set as part of the installation? Or did you choose the
  defaults?

Editor Option: VIM
Custom Editor Path:
Default Branch Option:
Path Option: Cmd
SSH Option: OpenSSH
Tortoise Option: false
CURL Option: OpenSSL
CRLF Option: CRLFAlways
Bash Terminal Option: MinTTY
Git Pull Behavior Option: Merge
Use Credential Manager: Enabled
Performance Tweaks FSCache: Enabled
Enable Symlinks: Enabled
Enable Pseudo Console Support: Disabled
Enable FSMonitor: Disabled

Details

• Which terminal/shell are you running Git from? e.g Bash/CMD/PowerShell/other

I am using the new Windows Terminal with PowerShell Core.

• What commands did you run to trigger this issue? If you can provide a
  Minimal, Complete, and Verifiable example
  this will help us understand the issue.

$env:EDITOR = 'C:/tools/neovim/Neovim/bin/nvim.exe'
git commit -a --verbose --signoff -S

• What did you expect to occur after running these commands?

I expected Neovim to start with my commit message in the same mode as it starts when run from the PowerShell prompt.

• What actually happened instead?

Neovim started with terminal support for Cygwin, with much worse colors etc..

Running the following inside Neovim shows:

:echo $TERM
" cygwin
:echo &term
" builtin_cygwin

, while starting it from PowerShell shows:

:echo $TERM
" vtpcon
:echo &term
" builtin_vtpcon

The issue is that $env:TERM is set to cygwin when Neovim launches from Git, causing it to use its builtin Cygwin terminal support which is much worse than the native vtpcon support for the new Windows console.

I opened an issue related to this in the neovim repository:

neovim/neovim#17021

There are two workarounds that I found for this issue, the first is creating a .bat file and setting it as $env:EDITOR.

This my ~/.local/bin/nvim.bat:

@echo off
set TERM=
/tools/neovim/neovim/bin/nvim %*

. Now, the following would work correctly:

$env:EDITOR = 'C:/Users/rkitover/.local/bin/nvim.bat'
git commit -a --verbose --signoff -S

. The other workaround is adding the following to ~/.gitconfig:

[core]
    editor = sh -c "'"TERM= c:/tools/neovim/Neovim/bin/nvim.exe \"$@\""'" --

.

Other Problems with $env:EDITOR on Windows

• A normal Windows path with backslashes in $env:EDITOR does not work at all:

$env:EDITOR = 'C:\tools\neovim\Neovim\bin\nvim.exe'
git commit -a --verbose --signoff -S

hint: Waiting for your editor to close the file... C:\tools\neovim\Neovim\bin\nvim.exe: C:toolsneovimNeovimbinnvim.exe: command not found
error: There was a problem with the editor 'C:\tools\neovim\Neovim\bin\nvim.exe'.
Please supply the message using either -m or -F option.

[dscho]

The issue is that $env:TERM is set to cygwin when Neovim launches from Git, causing it to use its builtin Cygwin terminal support which is much worse than the native vtpcon support for the new Windows console.

Isn't the real problem, then, that Neovim's defaults for the cygwin terminal are suboptimal?

[rkitover]

Isn't the real problem, then, that Neovim's defaults for the cygwin terminal are suboptimal?

This too, the new cygwin terminal support has almost full support for the new Windows console. However, they may want to support older systems. Further complicating this is that this is the native version of Neovim and not the Cygwin or MSYS2 version (MSYS2 is just Cygwin with a small patch for parameter-destroying.)

I should mention that the regular version of native vim does not have this problem, it ignores $env:TERM and uses its Windows console support.

I will bring this up in the Neovim issue I linked.

Regardless, I consider Git modifying $env:TERM a bug, if mine is empty I want it to stay empty when my editor or any external program launches.

Native Windows apps generally don't use $env:TERM, and there is no standard for this at present.

[dscho]

If native Windows programs don't use TERM, why do we even bother with this discussion?

[rkitover]

I tried setting:

$env:TERM = 'vtpcon'

and that works correctly with Neovim, and Git does not modify it in that case.

[dscho]

You could use one of the two approaches outlined in https://stackoverflow.com/questions/59733731/how-to-detect-if-running-in-the-new-windows-terminal or microsoft/terminal#6269 to detect in your PowerShell profile that it is running inside a Windows Terminal, and then set `$env:TERM = 'vtpcon':

1. use the presence of $env:WT_SESSION (although this might be a false positive if you spawn, say, a MinTTY window from your session)
2. walk the ParentProcessId chain to figure out whether the current session is a (transitive) child process of the Windows Terminal (it does run the same risk of a false positive, of course).

Of course, you could also change your Windows Terminal PowerShell Profile to execute with a startup command that sets this (I think -NoExit -Command '$env:TERM="vtpcon"' might work, but I haven't verified this).

[rkitover]

I took a different approach, I am setting:

$env:TERM = 'xterm-256color'
$env:COLORTERM = 'truecolor'

, in my $profile. This is currently the most widely used method on Linux as well. The $env:COLORTERM is not necessary in my case, but I set it just in case.

Both the Windows Terminal and versions of conhost made in the last many years support this just fine. All features I care about including 24 bit color work fine in Neovim. Regular vim also works fine, to the extent that it does on Windows, nano works fine.

So I have a solution for my problem and for the document I am maintaining.

But that does not solve the issue here, which is $env:TERM being set to cygwin when it is unset for native, non-Cygwin (including MSYS2 and related) apps.

I consider this a bug, if you don't, that's for you to decide.

Thank you for your work on this project.

[rkitover]

One of the terminal developers confirmed that these are good settings to use, and this works fine with the vim you ship with Git, other than the mouse not working, but that is likely a terminal issue. The mouse does not work in the native regular vim either.

[dscho]

But that does not solve the issue here, which is $env:TERM being set to cygwin when it is unset for native, non-Cygwin (including MSYS2 and related) apps.

How could the MSYS2/Cygwin runtime detect that it is connected to a Windows Terminal?

FWIW here is what the MSYS2 runtime does:

  /* If console has 24 bit color capability, TERM=xterm-256color,
     otherwise, TERM=cygwin */
  if (!sawTERM)
    envp[i++] = strdup (wincap.has_con_24bit_colors () ? xterm : cygterm);

That has_con_24bit_colors() method returns the value for the respective Windows version, as defined in wincap.cc. From what I can see, it should safely return true in your setup...

[rkitover]

I did some experiments and it seems that your version of Cygwin is setting $TERM correctly, the problem seems to be with how Git is launching $env:EDITOR.

I could take a look at this if you like, this is in editor.c.

[dscho]

I could take a look at this if you like, this is in editor.c.

If you mean to say that https://github.com/git-for-windows/git/blob/v2.35.0.windows.1/editor.c sets TERM=cygwin, I would like to learn how it does that.

[phil-blain]

Other Problems with $env:EDITOR on Windows

• A normal Windows path with backslashes in $env:EDITOR does not work at all:

$env:EDITOR = 'C:\tools\neovim\Neovim\bin\nvim.exe'
git commit -a --verbose --signoff -S

hint: Waiting for your editor to close the file... C:\tools\neovim\Neovim\bin\nvim.exe: C:toolsneovimNeovimbinnvim.exe: command not found
error: There was a problem with the editor 'C:\tools\neovim\Neovim\bin\nvim.exe'.
Please supply the message using either -m or -F option.

I also noticed that, and despite looking through the wiki and searching for "windows" in git-config(1), I could not find where this is documented.. @dscho is it documented somewhere ?

[rkitover]

I also noticed that, and despite looking through the wiki and searching for "windows" in git-config(1), I could not find where this is documented.. @dscho is it documented somewhere ?

My guess is that EDITOR is invoked with bash without proper quoting which treats backslashes as escapes. It seems that MSYS2/Cygwin bash is capable if launching Windows paths if they are properly quoted, for example, this will work:

'C:\Windows\notepad.exe'

.