summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2023-07-20 02:48:53 +0300
committerSimon Glass <sjg@chromium.org>2023-07-24 18:34:11 +0300
commit4ec76822747a5cdd82fff9a2accfb2beb7be6328 (patch)
tree194abebd40aa50ac86b72f107a80f6d1f9430eb6 /tools
parentffd06d3d8dfdff2aba57aafad8866805f9f542df (diff)
downloadu-boot-4ec76822747a5cdd82fff9a2accfb2beb7be6328.tar.xz
buildman: Tweak commits and show_bloat
Move setting of show_bloat to adjust_options() and adjust how the commits variable is set. Together these remove the pylint warning about too many statements. Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
-rw-r--r--tools/buildman/control.py13
1 files changed, 5 insertions, 8 deletions
diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index ece54ccb09..b0cc7078c4 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -441,6 +441,10 @@ def adjust_options(options, series, selected):
if not options.step:
options.step = len(series.commits) - 1
+ # We can't show function sizes without board details at present
+ if options.show_bloat:
+ options.show_detail = True
+
def setup_output_dir(output_dir, work_in_output, branch, no_subdirs, col,
clean_dir):
@@ -581,18 +585,11 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None,
# Work out which boards to build
board_selected = brds.get_selected_dict()
- if series:
- commits = series.commits
- else:
- commits = None
-
+ commits = series.commits if series else None
if not options.ide:
tprint(get_action_summary(options.summary, commits, board_selected,
options.step, options.threads, options.jobs))
- # We can't show function sizes without board details at present
- if options.show_bloat:
- options.show_detail = True
builder.SetDisplayOptions(
options.show_errors, options.show_sizes, options.show_detail,
options.show_bloat, options.list_error_boards, options.show_config,