[PATCH v2 14/60] bulldman: Set up output_dir earlier
Simon Glass
sjg at chromium.org
Wed Jul 5 14:18:42 CEST 2023
Set up output_dir at the start of the main function, instead of updating
the options.output_dir option.
Signed-off-by: Simon Glass <sjg at chromium.org>
---
(no changes since v1)
tools/buildman/control.py | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index f319dc8724e2..7c356ec5020e 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -402,14 +402,15 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None,
toolchains = get_toolchains(toolchains, col, options.override_toolchain,
options.fetch_arch, options.list_tool_chains,
options.verbose)
- if not options.output_dir:
+ output_dir = options.output_dir
+ if not output_dir:
if options.work_in_output:
sys.exit(col.build(col.RED, '-w requires that you specify -o'))
- options.output_dir = '..'
+ output_dir = '..'
# Work out what subset of the boards we are building
if not brds:
- brds = get_boards_obj(options.output_dir, options.regen_board_list,
+ brds = get_boards_obj(output_dir, options.regen_board_list,
options.threads, options.verbose)
if isinstance(brds, int):
return brds
@@ -451,13 +452,12 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None,
options.branch)
# Create a new builder with the selected options.
- output_dir = options.output_dir
if options.branch:
dirname = options.branch.replace('/', '_')
# As a special case allow the board directory to be placed in the
# output directory itself rather than any subdirectory.
if not options.no_subdirs:
- output_dir = os.path.join(options.output_dir, dirname)
+ output_dir = os.path.join(output_dir, dirname)
if clean_dir and os.path.exists(output_dir):
shutil.rmtree(output_dir)
--
2.41.0.255.g8b1d071c50-goog
More information about the U-Boot
mailing list