commit 78578595de217166fbd8fd69cd9c994ec2c16108
author Simon Glass <sjg@chromium.org> Wed Jul 19 17:48:51 2023 -0600
committer Simon Glass <sjg@chromium.org> Mon Jul 24 09:34:11 2023 -0600
tree 05ad7759bb49806964cffbe47c3133c267662adb
parent 1b14049251f73ea7fe1f38946a1627d4ad03a34d

buildman: Avoid too many returns in do_buildman()

Fix the pylint warning by using a variable instead of lots of 'return'
statements.

Signed-off-by: Simon Glass <sjg@chromium.org>

tools/buildman/control.py

diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index 51dc6fe..44a96cf 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -602,15 +602,16 @@
         options.show_bloat, options.list_error_boards, options.show_config,
         options.show_environment, options.filter_dtb_warnings,
         options.filter_migration_warnings, options.ide)
+    retval = 0
     if options.summary:
         builder.ShowSummary(commits, board_selected)
     else:
         fail, warned, excs = builder.BuildBoards(
             commits, board_selected, options.keep_outputs, options.verbose)
         if excs:
-            return 102
+            retval = 102
         if fail:
-            return 100
+            retval = 100
         if warned and not options.ignore_warnings:
-            return 101
-    return 0
+            retval = 101
+    return retval