[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