From: Simon Glass <sjg@chromium.org>
Date: Fri, 10 Jul 2020 00:39:39 +0000 (-0600)
Subject: patman: Update errors and warnings to use stderr
X-Git-Tag: v2020.10~98^2~33
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=38fdb4cb35e9260a6aa78ffcfa68d39bfc3523de;p=platform%2Fkernel%2Fu-boot.git

patman: Update errors and warnings to use stderr

When warnings and errors are produced by tools they should be written to
stderr. Update the tout implementation to handle this.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
---

diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py
index 7c8b3eb..928d360 100644
--- a/tools/binman/ftest.py
+++ b/tools/binman/ftest.py
@@ -3232,7 +3232,7 @@ class TestFunctional(unittest.TestCase):
         with test_util.capture_sys_output() as (stdout, stderr):
             control.ReplaceEntries(updated_fname, None, outdir, [])
         self.assertIn("Skipping entry '/u-boot' from missing file",
-                      stdout.getvalue())
+                      stderr.getvalue())
 
     def testReplaceCmdMap(self):
         """Test replacing a file fron an image on the command line"""
diff --git a/tools/patman/tout.py b/tools/patman/tout.py
index c7e3272..91a53f4 100644
--- a/tools/patman/tout.py
+++ b/tools/patman/tout.py
@@ -83,7 +83,10 @@ def _Output(level, msg, color=None):
         ClearProgress()
         if color:
             msg = _color.Color(color, msg)
-        print(msg)
+        if level < NOTICE:
+            print(msg, file=sys.stderr)
+        else:
+            print(msg)
 
 def DoOutput(level, msg):
     """Output a message to the terminal.