From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Hershberger Date: Tue, 19 May 2015 12:52:06 -0500 Subject: [U-Boot] [PATCH v4 09/10] moveconfig: Add a switch to enable printing errors In-Reply-To: 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: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Masahiro-san, On Mon, May 18, 2015 at 10:25 PM, Masahiro Yamada wrote: > 2015-05-16 6:40 GMT+09:00 Joe Hershberger : >> 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 >> >> --- >> >> 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