From 8dae07aa404a08297c1b0a3ae3963c7758026227 Mon Sep 17 00:00:00 2001
From: Simon Glass <sjg@chromium.org>
Date: Wed, 19 Jul 2023 17:48:11 -0600
Subject: [PATCH] buildman: Exit with the return code consistently
Test should return a suitable exit code when they fail. Fix this and tidy
up the code a little.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
tools/buildman/main.py | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tools/buildman/main.py b/tools/buildman/main.py
index 5cbdce7cf3..096f5328a2 100755
--- a/tools/buildman/main.py
+++ b/tools/buildman/main.py
@@ -62,14 +62,16 @@ def run_buildman():
# Run our meagre tests
if cmdline.HAS_TESTS and options.test:
- RunTests(options.skip_net_tests, options.debug, options.verbose, args)
+ return RunTests(options.skip_net_tests, options.debug, options.verbose,
+ args)
# Build selected commits for selected boards
else:
bsettings.Setup(options.config_file)
ret_code = control.DoBuildman(options, args)
- sys.exit(ret_code)
+ return ret_code
+ return 0
if __name__ == "__main__":
- run_buildman()
+ sys.exit(run_buildman())
--
2.39.5