Hi Tom, On Tue, 15 Aug 2023 at 08:42, Tom Rini <[email protected]> wrote: > > On Thu, Aug 03, 2023 at 12:59:55PM -0600, Simon Glass wrote: > > > The -v option enables display of build results while building, instead of > > needing to do a separate summary step later (or in another terminal). But > > this resets the build results after each commit, so that all errors are > > reported fresh, without trying to diff them against previous errors > > already shown. > > > > Drop this reset, as an experiment, so see if this is useful. > > > > Signed-off-by: Simon Glass <[email protected]> > > --- > > > > tools/buildman/builder.py | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py > > index ecbd368c47a..5ee2b5959b2 100644 > > --- a/tools/buildman/builder.py > > +++ b/tools/buildman/builder.py > > @@ -536,7 +536,6 @@ class Builder: > > if self._verbose: > > terminal.print_clear() > > boards_selected = {target : result.brd} > > - self.reset_result_summary(boards_selected) > > self.produce_result_summary(result.commit_upto, > > self.commits, > > boards_selected) > > else: > > Nope, this doesn't change any of the output I see in how I usually do my > world builds.
For me it allows using -S and -B during the build phase, and seeing some output. However, the patch is not correct...it is just a concept. I should have marked it as RFC. Regards, Simon
