[U-Boot] [PATCH v4 09/10] moveconfig: Add a switch to enable printing errors

Joe Hershberger joe.hershberger at gmail.com
Tue May 19 19:52:06 CEST 2015


Hi Masahiro-san,

On Mon, May 18, 2015 at 10:25 PM, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> 2015-05-16 6:40 GMT+09:00 Joe Hershberger <joe.hershberger at 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 at 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.

OK. Dropped.

> Apart from that, I think this is a nice improvement.
>
>
> --
> Best Regards
> Masahiro Yamada
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list