summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2012-12-15 10:42:01 +0000
committerSimon Glass <sjg@chromium.org>2013-04-04 14:04:32 -0700
commitbbd01435b9f3f1f60355b95f157170ec52c6353d (patch)
treef6f5bd098c6658aafda60b0072f485757d3ab727 /tools
parentc8142633e169665b246352918df5b76fd243bb71 (diff)
patman: Use ANSI colours only when outputting to a terminal
It is easy to detect whether or not the process is connected to a terminal, or piped to a file. Disable ANSI colours automatically when output is not to a terminal. Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
-rw-r--r--tools/patman/terminal.py11
1 files changed, 9 insertions, 2 deletions
diff --git a/tools/patman/terminal.py b/tools/patman/terminal.py
index 838c82845f4..8fad06eb550 100644
--- a/tools/patman/terminal.py
+++ b/tools/patman/terminal.py
@@ -24,6 +24,12 @@
This module handles terminal interaction including ANSI color codes.
"""
+import os
+import sys
+
+# Selection of when we want our output to be colored
+COLOR_IF_TERMINAL, COLOR_ALWAYS, COLOR_NEVER = range(3)
+
class Color(object):
"""Conditionally wraps text in ANSI color escape sequences."""
BLACK, RED, GREEN, YELLOW, BLUE, MAGENTA, CYAN, WHITE = range(8)
@@ -32,14 +38,15 @@ class Color(object):
BOLD_START = '\033[1m'
RESET = '\033[0m'
- def __init__(self, enabled=True):
+ def __init__(self, colored=COLOR_IF_TERMINAL):
"""Create a new Color object, optionally disabling color output.
Args:
enabled: True if color output should be enabled. If False then this
class will not add color codes at all.
"""
- self._enabled = enabled
+ self._enabled = (colored == COLOR_ALWAYS or
+ (colored == COLOR_IF_TERMINAL and os.isatty(sys.stdout.fileno())))
def Start(self, color):
"""Returns a start color code.