diff options
author | Simon Glass <sjg@chromium.org> | 2023-07-05 13:18:36 +0100 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2023-07-12 07:57:20 -0600 |
commit | dd6630b245cb2e12e83799ecc440cda9b317592f (patch) | |
tree | 746ee7414dbc0b9c9077089a8a7d83765726311c | |
parent | 90c8c1dad52946ac1945cf2007425af53002352f (diff) | |
download | u-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.py | 8 |
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 |