[U-Boot] [PATCH 2/6] buildman: Add a space before the list of boards
Simon Glass
sjg at chromium.org
Fri Mar 6 00:18:08 CET 2015
On 5 February 2015 at 22:06, Simon Glass <sjg at chromium.org> wrote:
> Tweak the output slightly so we don't get things like:
>
> - board1 board2+ board3 board4
>
> There should be a space before the '+'.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> tools/buildman/builder.py | 2 +-
> tools/buildman/test.py | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
> index 1b0ad99..54f3292 100644
> --- a/tools/buildman/builder.py
> +++ b/tools/buildman/builder.py
> @@ -664,7 +664,7 @@ class Builder:
> arch = 'unknown'
> str = self.col.Color(color, ' ' + target)
> if not arch in done_arch:
> - str = self.col.Color(color, char) + ' ' + str
> + str = ' %s %s' % (self.col.Color(color, char), str)
> done_arch[arch] = True
> if not arch in arch_list:
> arch_list[arch] = str
> diff --git a/tools/buildman/test.py b/tools/buildman/test.py
> index c0ad5d0..7642d94 100644
> --- a/tools/buildman/test.py
> +++ b/tools/buildman/test.py
> @@ -169,7 +169,7 @@ class TestBuild(unittest.TestCase):
> expected_colour = col.GREEN if ok else col.RED
> expect = '%10s: ' % arch
> # TODO(sjg at chromium.org): If plus is '', we shouldn't need this
> - expect += col.Color(expected_colour, plus)
> + expect += ' ' + col.Color(expected_colour, plus)
> expect += ' '
> for board in boards:
> expect += col.Color(expected_colour, ' %s' % board)
> --
> 2.2.0.rc0.207.ga3a616c
>
Applied u-boot-x86/buildman.
More information about the U-Boot
mailing list