aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2023-07-05 13:18:36 +0100
committerSimon Glass <sjg@chromium.org>2023-07-12 07:57:20 -0600
commitdd6630b245cb2e12e83799ecc440cda9b317592f (patch)
tree746ee7414dbc0b9c9077089a8a7d83765726311c
parent90c8c1dad52946ac1945cf2007425af53002352f (diff)
downloadu-boot-dd6630b245cb2e12e83799ecc440cda9b317592f.zip
u-boot-dd6630b245cb2e12e83799ecc440cda9b317592f.tar.gz
u-boot-dd6630b245cb2e12e83799ecc440cda9b317592f.tar.bz2
buildman: Drop use of builder in show_actions()
This function only needs the output directory from the builder. This is passed into the builder, so just pass the same value to show_actions(). The avoids needing a builder to call show_actions(). Signed-off-by: Simon Glass <sjg@chromium.org>
-rw-r--r--tools/buildman/control.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index fd136e5..7beb5bc 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -48,7 +48,7 @@ def get_action_summary(is_summary, commits, selected, options):
return msg
# pylint: disable=R0913
-def show_actions(series, why_selected, boards_selected, bldr, options,
+def show_actions(series, why_selected, boards_selected, output_dir, options,
board_warnings):
"""Display a list of actions that we would take, if not a dry run.
@@ -61,7 +61,7 @@ def show_actions(series, why_selected, boards_selected, bldr, options,
the value would be a list of board names.
boards_selected: Dict of selected boards, key is target name,
value is Board object
- bldr: The builder that will be used to build the commits
+ output_dir (str): Output directory for builder
options: Command line options object
board_warnings: List of warnings obtained from board selected
"""
@@ -74,7 +74,7 @@ def show_actions(series, why_selected, boards_selected, bldr, options,
commits = None
print(get_action_summary(False, commits, boards_selected,
options))
- print(f'Build directory: {bldr.base_dir}')
+ print(f'Build directory: {output_dir}')
if commits:
for upto in range(0, len(series.commits), options.step):
commit = series.commits[upto]
@@ -416,7 +416,7 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None,
# For a dry run, just show our actions as a sanity check
if options.dry_run:
- show_actions(series, why_selected, selected, builder, options,
+ show_actions(series, why_selected, selected, output_dir, options,
board_warnings)
else:
builder.force_build = options.force_build