Bug#742207: [PATCH] Update the handling of colour output for more use-cases.

2014-03-30 Thread Lucas De Marchi
On Sat, Mar 22, 2014 at 3:05 AM, Paul Wise pa...@bonedaddy.net wrote:
 Disable colours when stdout is not a terminal by default.

 Add an option to enable colours when stdout is not a terminal.

 Fixes: https://bugs.debian.org/742207
 ---

Applied. Thanks.

Lucas De Marchi


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#742207: [PATCH] Update the handling of colour output for more use-cases.

2014-03-22 Thread Paul Wise
Disable colours when stdout is not a terminal by default.

Add an option to enable colours when stdout is not a terminal.

Fixes: https://bugs.debian.org/742207
---
 codespell.py | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/codespell.py b/codespell.py
index fdbd5cb..1fd8409 100755
--- a/codespell.py
+++ b/codespell.py
@@ -189,9 +189,13 @@ class FileOpener:
 def parse_options(args):
 parser = OptionParser(usage=USAGE, version=VERSION)
 
+parser.set_defaults(colors = sys.stdout.isatty())
 parser.add_option('-d', '--disable-colors',
-action = 'store_true', default = False,
+action = 'store_false', dest = 'colors',
 help = 'Disable colors even when printing to terminal')
+parser.add_option('-c', '--enable-colors',
+action = 'store_true', dest = 'colors',
+help = 'Enable colors even when not printing to 
terminal')
 parser.add_option('-w', '--write-changes',
 action = 'store_true', default = False,
 help = 'write changes in place if possible')
@@ -478,7 +482,7 @@ def main(*args):
 
 build_dict(options.dictionary)
 colors = TermColors();
-if options.disable_colors:
+if not options.colors:
 colors.disable()
 
 if options.summary:
-- 
1.9.0


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org