diff options
author | Simon Glass <sjg@chromium.org> | 2012-12-15 10:42:01 (GMT) |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2013-04-04 21:04:32 (GMT) |
commit | bbd01435b9f3f1f60355b95f157170ec52c6353d (patch) | |
tree | f6f5bd098c6658aafda60b0072f485757d3ab727 /tools | |
parent | c8142633e169665b246352918df5b76fd243bb71 (diff) | |
download | u-boot-fsl-qoriq-bbd01435b9f3f1f60355b95f157170ec52c6353d.tar.xz |
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.py | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/tools/patman/terminal.py b/tools/patman/terminal.py index 838c828..8fad06e 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. |