From mboxrd@z Thu Jan  1 00:00:00 1970
From: Masahiro Yamada <yamada.masahiro@socionext.com>
Date: Tue, 19 May 2015 12:25:00 +0900
Subject: [U-Boot] [PATCH v4 09/10] moveconfig: Add a switch to enable
 printing errors
In-Reply-To: <1431726052-6519-9-git-send-email-joe.hershberger@ni.com>
References: <1431556137-8426-1-git-send-email-joe.hershberger@ni.com>
 <1431726052-6519-1-git-send-email-joe.hershberger@ni.com>
 <1431726052-6519-9-git-send-email-joe.hershberger@ni.com>
Message-ID: <CAK7LNASOAfZ20LcyZ=8AptEUPGf8c45o43RHiaA3spA2jDj9wA@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

2015-05-16 6:40 GMT+09:00 Joe Hershberger <joe.hershberger@ni.com>:
> In some cases the build for the autoconf breaks. This outputs the errors
> following the status so that action can be taken without building again
> manually.
>
> Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> -New for version 2
>
>  tools/moveconfig.py | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/tools/moveconfig.py b/tools/moveconfig.py
> index f986f55..685b47b 100755
> --- a/tools/moveconfig.py
> +++ b/tools/moveconfig.py
> @@ -153,6 +153,9 @@ Available options
>     Specify the number of threads to run simultaneously.  If not specified,
>     the number of threads is the same as the number of CPU cores.
>
> + -v, --verbose
> +   Show any build errors as boards are built
> +
>  To see the complete list of supported options, run
>
>    $ tools/moveconfig.py -h
> @@ -615,6 +618,9 @@ class Slot:
>                                           COLOR_LIGHT_RED,
>                                           self.defconfig,
>                                           errmsg),
> +            if self.options.verbose:
> +                print >> sys.stderr, color_text(self.options.color,
> +                                                COLOR_LIGHT_CYAN, errout)
>              if self.options.exit_on_error:
>                  sys.exit("Exit on error.")
>              else:
> @@ -882,6 +888,9 @@ def main():
>                        help='only cleanup the headers')
>      parser.add_option('-j', '--jobs', type='int', default=cpu_count,
>                        help='the number of jobs to run simultaneously')
> +    parser.add_option('-v', '--verbose', dest='verbose',
> +                      action='store_true', default=False,
> +                      help='show any build errors as boards are built')

dest='verbose' is redundant.

The destination is chosen based on the long option name.


Apart from that, I think this is a nice improvement.


-- 
Best Regards
Masahiro Yamada