[PATCH v3 33/81] bulldman: Set up output_dir earlier
Simon Glass
sjg at chromium.org
Sun Jul 16 02:36:01 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 2bc235e130a2..b6cd9787fbbd 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -410,14 +410,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.maintainer_check, options.threads,
options.verbose)
if isinstance(brds, int):
@@ -460,13 +461,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.455.g037347b96a-goog
More information about the U-Boot
mailing list