commit d46a195fe21d86e27110f9225a71be57691a253a
author Andy Scott <andy.g.scott@gmail.com> Tue May 28 08:30:52 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue May 28 08:31:53 2019 -0700
tree f4c35185ee89f0d5b19776adfdf0adc128c37ad8
parent 4d1f5a3abdd228a581827af22970aec22f872bab

Auto enable terminal color/curses for --color=auto for some Emacs configurations

Enables color when `--color=auto` for Emacs when `$TERM` is `eterm-color`, as is the case with `ansi-term` open.

Closes #8424.

PiperOrigin-RevId: 250286483

src/main/cpp/blaze_util_posix.cc

diff --git a/src/main/cpp/blaze_util_posix.cc b/src/main/cpp/blaze_util_posix.cc
index 6ed6a41..2bab99c 100644
--- a/src/main/cpp/blaze_util_posix.cc
+++ b/src/main/cpp/blaze_util_posix.cc
@@ -715,9 +715,14 @@
 // control characters is stderr, so we only care for the stderr descriptor type.
 bool IsStderrStandardTerminal() {
   string term = GetEnv("TERM");
+  bool isEmacs = IsEmacsTerminal();
+
+  if (isEmacs && term == "eterm-color") {
+    return true;
+  }
   if (term.empty() || term == "dumb" || term == "emacs" ||
       term == "xterm-mono" || term == "symbolics" || term == "9term" ||
-      IsEmacsTerminal()) {
+      isEmacs) {
     return false;
   }
   return isatty(STDERR_FILENO);