diff options
author | Simon Glass <sjg@chromium.org> | 2020-07-10 03:39:39 +0300 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2020-07-25 23:46:57 +0300 |
commit | 38fdb4cb35e9260a6aa78ffcfa68d39bfc3523de (patch) | |
tree | b46104f229d43a8139f0d5ee6590d5f1bfd73f65 /tools | |
parent | 4f9f1056ecf2d9e54803b89fd0784240a8d89fd8 (diff) | |
download | u-boot-38fdb4cb35e9260a6aa78ffcfa68d39bfc3523de.tar.xz |
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>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/binman/ftest.py | 2 | ||||
-rw-r--r-- | tools/patman/tout.py | 5 |
2 files changed, 5 insertions, 2 deletions
diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index 7c8b3eb3a0..928d3608a3 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 c7e3272096..91a53f4073 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. |