logger: use color automatically for a tty

Signed-off-by: Joe Guo <joeg@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Wed Nov 21 10:46:20 CET 2018 on sn-devel-144

diff --git a/python/samba/logger.py b/python/samba/logger.py
index a3db3beeec89d358081fba609efc02e997c45003..667c6487a51a9acec883ef182611e56886025a7a 100644 (file)
--- a/python/samba/logger.py
+++ b/python/samba/logger.py
@@ -39,7 +39,7 @@ class ColoredFormatter(logging.Formatter):
 
 
 def get_samba_logger(
-        name='samba', stream=sys.stderr, use_color=True,
+        name='samba', stream=sys.stderr,
         level=None, verbose=False, quiet=False,
         fmt=('%(levelname)s %(asctime)s pid:%(process)d '
              '%(pathname)s #%(lineno)d: %(message)s'),
@@ -56,7 +56,10 @@ def get_samba_logger(
 
     logger.setLevel(level)
 
-    Formatter = use_color and ColoredFormatter or logging.Formatter
+    if (hasattr(stream, 'isatty') and stream.isatty()):
+        Formatter = ColoredFormatter
+    else:
+        Formatter = logging.Formatter
     formatter = Formatter(fmt=fmt, datefmt=datefmt)
 
     handler = logging.StreamHandler(stream=stream)