hgext/color.py
changeset 24249 5058e6962fcc
parent 24068 4e02418b4236
child 24266 c14b1d349af3
--- a/hgext/color.py	Thu Feb 26 20:54:05 2015 +0900
+++ b/hgext/color.py	Tue Mar 10 13:40:14 2015 -0400
@@ -254,7 +254,7 @@
     if realmode == 'win32':
         _terminfo_params = {}
         if not w32effects:
-            if mode == 'win32':
+            if mode == 'win32' and ui.interactive():
                 # only warn if color.mode is explicitly set to win32
                 ui.warn(_('warning: failed to set color mode to %s\n') % mode)
             return None
@@ -264,7 +264,7 @@
     elif realmode == 'terminfo':
         _terminfosetup(ui, mode)
         if not _terminfo_params:
-            if mode == 'terminfo':
+            if mode == 'terminfo' and ui.interactive():
                 ## FIXME Shouldn't we return None in this case too?
                 # only warn if color.mode is explicitly set to win32
                 ui.warn(_('warning: failed to set color mode to %s\n') % mode)