diff options
author | Simon Glass <sjg@chromium.org> | 2023-07-20 02:48:36 +0300 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2023-07-24 18:34:10 +0300 |
commit | d233dfb07d43ddc75458796d4f7b14d5eb3093d0 (patch) | |
tree | 035added5cc535436f7a58213469351c993d6f37 /tools/buildman | |
parent | d4366b11ad556f4e662afd4fb90af5e51d5137e5 (diff) | |
download | u-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.py | 8 |
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 |