Skip to content

Commit bd7b567

Browse files
rkitoverdscho
authored andcommitted
mingw: $env:TERM="xterm-256color" for newer OSes
For Windows builds >= 15063 set $env:TERM to "xterm-256color" instead of "cygwin" because they have a more capable console system that supports this. Also set $env:COLORTERM="truecolor" if unset. $env:TERM is initialized so that ANSI colors in color.c work, see 29a3963 (Win32: patch Windows environment on startup, 2012-01-15). See #3629 regarding problems caused by always setting $env:TERM="cygwin". This is the same heuristic used by the Cygwin runtime. Signed-off-by: Rafael Kitover <[email protected]> Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 38993b5 commit bd7b567

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

compat/mingw.c

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2795,9 +2795,20 @@ static void setup_windows_environment(void)
27952795
convert_slashes(tmp);
27962796
}
27972797

2798-
/* simulate TERM to enable auto-color (see color.c) */
2799-
if (!getenv("TERM"))
2800-
setenv("TERM", "cygwin", 1);
2798+
2799+
/*
2800+
* Make sure TERM is set up correctly to enable auto-color
2801+
* (see color.c .) Use "cygwin" for older OS releases which
2802+
* works correctly with MSYS2 utilities on older consoles.
2803+
*/
2804+
if (!getenv("TERM")) {
2805+
if ((GetVersion() >> 16) < 15063)
2806+
setenv("TERM", "cygwin", 0);
2807+
else {
2808+
setenv("TERM", "xterm-256color", 0);
2809+
setenv("COLORTERM", "truecolor", 0);
2810+
}
2811+
}
28012812

28022813
/* calculate HOME if not set */
28032814
if (!getenv("HOME")) {

0 commit comments

Comments
 (0)