Commit 7798e2285fda17aa2268cbe18498678cf857905f

Authored by Simon Glass
1 parent 6b1978f8a1

buildman: Fix the logic for the bloat command

This check should now be done whatever mode buildman is running in, since
we may be displaying information while building.

Signed-off-by: Simon Glass <sjg@chromium.org>

Showing 1 changed file with 3 additions and 3 deletions Side-by-side Diff

tools/buildman/control.py
... ... @@ -244,13 +244,13 @@
244 244 Print(GetActionSummary(options.summary, commits, board_selected,
245 245 options))
246 246  
  247 + # We can't show function sizes without board details at present
  248 + if options.show_bloat:
  249 + options.show_detail = True
247 250 builder.SetDisplayOptions(options.show_errors, options.show_sizes,
248 251 options.show_detail, options.show_bloat,
249 252 options.list_error_boards)
250 253 if options.summary:
251   - # We can't show function sizes without board details at present
252   - if options.show_bloat:
253   - options.show_detail = True
254 254 builder.ShowSummary(commits, board_selected)
255 255 else:
256 256 fail, warned = builder.BuildBoards(commits, board_selected,