summaryrefslogtreecommitdiff
path: root/tools/buildman
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2023-07-20 02:48:36 +0300
committerSimon Glass <sjg@chromium.org>2023-07-24 18:34:10 +0300
commitd233dfb07d43ddc75458796d4f7b14d5eb3093d0 (patch)
tree035added5cc535436f7a58213469351c993d6f37 /tools/buildman
parentd4366b11ad556f4e662afd4fb90af5e51d5137e5 (diff)
downloadu-boot-d233dfb07d43ddc75458796d4f7b14d5eb3093d0.tar.xz
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>
Diffstat (limited to 'tools/buildman')
-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 5762ec90c0..0aa19a2796 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]
@@ -427,7 +427,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