[PATCH] buildman: Fix some newer pylint warnings

Tom Rini trini at konsulko.com
Wed Feb 12 23:23:28 CET 2025


With a newer pylint we get a warning about "color" or "col" being used
before assigned. This is most easily handled by setting them to None to
indicate no specific color is needed, only the default.

Signed-off-by: Tom Rini <trini at konsulko.com>
---
Cc: Simon Glass <sjg at chromium.org>
---
 tools/buildman/builder.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index cbf1345281bf..492697310808 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -1093,6 +1093,7 @@ class Builder:
             printed_target = False
             for name in sorted(result):
                 diff = result[name]
+                color = None
                 if name.startswith('_'):
                     continue
                 if diff != 0:
@@ -1359,6 +1360,8 @@ class Builder:
                     col = self.col.RED
                 elif line[0] == 'c':
                     col = self.col.YELLOW
+                else:
+                    col = None
                 tprint('   ' + line, newline=True, colour=col)
 
         def _output_err_lines(err_lines, colour):
-- 
2.43.0



More information about the U-Boot mailing list