From: Carl Worth <cworth@cworth.org>
Date: Mon, 13 Aug 2012 20:24:03 +0000 (-0700)
Subject: diff: Don't use colorized line-format options when not on a tty
X-Git-Url: https://git.notmuchmail.org/git?a=commitdiff_plain;h=f45f1e2f0113a78a64b909ad87801f9f63a1fd28;p=apitrace

diff: Don't use colorized line-format options when not on a tty

The color output is convenient when manually looking at the
output. But when saving output to a file, (for example, for use in the
test suite), it's useful to have a much more minimal output.
---

diff --git a/scripts/tracediff.py b/scripts/tracediff.py
index 86d1e00..2639587 100755
--- a/scripts/tracediff.py
+++ b/scripts/tracediff.py
@@ -70,6 +70,8 @@ else:
 
 def diff(ref_trace, src_trace):
 
+    isatty = sys.stdout.isatty()
+
     ref_dumper = Dumper(ref_trace, options.ref_calls)
     src_dumper = Dumper(src_trace, options.src_calls)
 
@@ -78,6 +80,9 @@ def diff(ref_trace, src_trace):
         diff_args = [
                 'diff',
                 '--speed-large-files',
+            ]
+        if isatty:
+            diff_args += [
                 '--old-line-format=' + start_delete + '%l' + end_delete + '\n',
                 '--new-line-format=' + start_insert + '%l' + end_insert + '\n',
             ]
@@ -92,6 +97,9 @@ def diff(ref_trace, src_trace):
                 'wdiff',
                 #'--terminal',
                 '--avoid-wraps',
+            ]
+        if isatty:
+            diff_args += [
                 '--start-delete=' + start_delete,
                 '--end-delete=' + end_delete,
                 '--start-insert=' + start_insert,
@@ -105,7 +113,7 @@ def diff(ref_trace, src_trace):
     src_dumper.dump.wait()
 
     less = None
-    if sys.stdout.isatty():
+    if isatty:
         less = subprocess.Popen(
             args = ['less', '-FRXn'],
             stdin = subprocess.PIPE