From patchwork Tue Dec 2 00:34:07 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 416651 X-Patchwork-Delegate: sjg@chromium.org Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id A3D68140284 for ; Tue, 2 Dec 2014 11:44:35 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 38D5D4B75A; Tue, 2 Dec 2014 01:44:31 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id oU6OaakP6wve; Tue, 2 Dec 2014 01:44:31 +0100 (CET) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 014AF4B6FA; Tue, 2 Dec 2014 01:44:24 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id CCD894B6F7 for ; Tue, 2 Dec 2014 01:44:13 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id uwHnsa7H3gQ2 for ; Tue, 2 Dec 2014 01:44:13 +0100 (CET) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mail-qc0-f201.google.com (mail-qc0-f201.google.com [209.85.216.201]) by theia.denx.de (Postfix) with ESMTPS id 776024B6F6 for ; Tue, 2 Dec 2014 01:44:09 +0100 (CET) Received: by mail-qc0-f201.google.com with SMTP id m20so620059qcx.0 for ; Mon, 01 Dec 2014 16:44:06 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-type:content-transfer-encoding; bh=B86IMUYXlfXvqb0Tb6xlvQrXsHHjfGZBBFfPpN8+fSg=; b=Y7ET79KrMdfnWNB4qiY5LqNFn4M/V1LatzOGB9M1i9WHlHkHShnr3L8k/Wx9pPQLs2 c77cdUWVMSbZ04JkdR+/8uv7V4ptgxeDlFnfb832LKDxnAwYMgmIIjT3WZp18fMe+GKl nvBIaQPvJ+2bCAwMK6ukfxcDtsWvAgGW5CWewL98SZ+VwvHKDLPeAEtunCmuPPGqZO8f 2oUBqtWWKhKYgTeactaWj50KACIq0SRiYKHVjOxs2bDBa/D13CIWwZhfAsOIeswuufTr aLs5DtN6xq0f9JLPN490t1QISCtXHFJN26j/IsjTQpbictO3v/YswF1nQv0Zx4TemqWU 4E2w== X-Gm-Message-State: ALoCoQmPYtpuubEY/h2ti8gxlo2DrjcRS9Td5DdqOaSsT80nywtYae992vnSp8AXIAGPUdDdInAi X-Received: by 10.236.2.41 with SMTP id 29mr59265767yhe.32.1417481046900; Mon, 01 Dec 2014 16:44:06 -0800 (PST) Received: from corpmail-nozzle1-2.hot.corp.google.com ([100.108.1.103]) by gmr-mx.google.com with ESMTPS id r6si773617yhg.1.2014.12.01.16.44.06 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 01 Dec 2014 16:44:06 -0800 (PST) Received: from kaki.bld.corp.google.com ([172.29.216.32]) by corpmail-nozzle1-2.hot.corp.google.com with ESMTP id mkPBxMy0.1; Mon, 01 Dec 2014 16:44:06 -0800 Received: by kaki.bld.corp.google.com (Postfix, from userid 121222) id 6DCE22210FD; Mon, 1 Dec 2014 17:36:17 -0700 (MST) From: Simon Glass To: U-Boot Mailing List Date: Mon, 1 Dec 2014 17:34:07 -0700 Message-Id: <1417480447-5763-18-git-send-email-sjg@chromium.org> X-Mailer: git-send-email 2.2.0.rc0.207.ga3a616c In-Reply-To: <1417480447-5763-1-git-send-email-sjg@chromium.org> References: <1417480447-5763-1-git-send-email-sjg@chromium.org> MIME-Version: 1.0 Cc: Steve Rae , =?UTF-8?q?Marek=20Va=C5=A1ut?= , Jeroen Hofstee , Tom Rini Subject: [U-Boot] [PATCH 17/17] buildman: Add an option to write the full build output X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.13 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: u-boot-bounces@lists.denx.de Errors-To: u-boot-bounces@lists.denx.de Normally buildman runs with 'make -s' meaning that only errors and warnings appear in the log file. Add a -V option to run make in verbose mode, and with V=1, causing a full build log to be created. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 4 +++- tools/buildman/builderthread.py | 3 ++- tools/buildman/cmdline.py | 2 ++ tools/buildman/control.py | 3 ++- 4 files changed, 9 insertions(+), 3 deletions(-) diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py index 93d048b..1b0ad99 100644 --- a/tools/buildman/builder.py +++ b/tools/buildman/builder.py @@ -175,7 +175,7 @@ class Builder: def __init__(self, toolchains, base_dir, git_dir, num_threads, num_jobs, gnu_make='make', checkout=True, show_unknown=True, step=1, - no_subdirs=False, full_path=False): + no_subdirs=False, full_path=False, verbose_build=False): """Create a new Builder object Args: @@ -193,6 +193,7 @@ class Builder: source for a single board full_path: Return the full path in CROSS_COMPILE and don't set PATH + verbose_build: Run build with V=1 and don't use 'make -s' """ self.toolchains = toolchains self.base_dir = base_dir @@ -220,6 +221,7 @@ class Builder: self._error_lines = 0 self.no_subdirs = no_subdirs self.full_path = full_path + self.verbose_build = verbose_build self.col = terminal.Color() diff --git a/tools/buildman/builderthread.py b/tools/buildman/builderthread.py index a803481..efb62f1 100644 --- a/tools/buildman/builderthread.py +++ b/tools/buildman/builderthread.py @@ -197,7 +197,8 @@ class BuilderThread(threading.Thread): src_dir = os.getcwd() else: args.append('O=build') - args.append('-s') + if not self.builder.verbose_build: + args.append('-s') if self.builder.num_jobs is not None: args.extend(['-j', str(self.builder.num_jobs)]) config_args = ['%s_defconfig' % brd.target] diff --git a/tools/buildman/cmdline.py b/tools/buildman/cmdline.py index e884e19..e8a6dad 100644 --- a/tools/buildman/cmdline.py +++ b/tools/buildman/cmdline.py @@ -82,6 +82,8 @@ def ParseArgs(): default=False, help='Show boards with unknown build result') parser.add_option('-v', '--verbose', action='store_true', default=False, help='Show build results while the build progresses') + parser.add_option('-V', '--verbose-build', action='store_true', + default=False, help='Run make with V=1, showing all output') parser.add_option('-x', '--exclude', dest='exclude', type='string', action='append', help='Specify a list of boards to exclude, separated by comma') diff --git a/tools/buildman/control.py b/tools/buildman/control.py index 6fe53cb..5bbd87d 100644 --- a/tools/buildman/control.py +++ b/tools/buildman/control.py @@ -246,7 +246,8 @@ def DoBuildman(options, args, toolchains=None, make_func=None, boards=None, builder = Builder(toolchains, output_dir, options.git_dir, options.threads, options.jobs, gnu_make=gnu_make, checkout=True, show_unknown=options.show_unknown, step=options.step, - no_subdirs=options.no_subdirs, full_path=options.full_path) + no_subdirs=options.no_subdirs, full_path=options.full_path, + verbose_build=options.verbose_build) builder.force_config_on_failure = not options.quick if make_func: builder.do_make = make_func