From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from comal.ext.ti.com (comal.ext.ti.com [198.47.26.152]) by arago-project.org (Postfix) with ESMTPS id 3932D52A04 for ; Thu, 4 Sep 2014 19:03:07 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id s84J36jG023240 for ; Thu, 4 Sep 2014 14:03:06 -0500 Received: from DLEE71.ent.ti.com (dlee71.ent.ti.com [157.170.170.114]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id s84J36m4003279 for ; Thu, 4 Sep 2014 14:03:06 -0500 Received: from dlep32.itg.ti.com (157.170.170.100) by DLEE71.ent.ti.com (157.170.170.114) with Microsoft SMTP Server id 14.3.174.1; Thu, 4 Sep 2014 14:03:06 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dlep32.itg.ti.com (8.14.3/8.13.8) with ESMTP id s84J36UI021101; Thu, 4 Sep 2014 14:03:06 -0500 Date: Thu, 4 Sep 2014 15:03:05 -0400 From: Denys Dmytriyenko To: "Stiffler, Jacob" Message-ID: <20140904190305.GR18620@edge> References: <1409851868-7389-1-git-send-email-j-stiffler@ti.com> <1409851868-7389-3-git-send-email-j-stiffler@ti.com> <20140904173842.GL18620@edge> <4B26FB5A64EBA14C87266869E037F958140B10AE@DFLE09.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <4B26FB5A64EBA14C87266869E037F958140B10AE@DFLE09.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" Subject: Re: [tisdk-build-scripts][PATCH 3/4] bitbake-test-sdk-image: Add support to control verbosity of build logging. X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Sep 2014 19:03:07 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Thu, Sep 04, 2014 at 02:05:01PM -0400, Stiffler, Jacob wrote: > This change is to capture stderr and stdout to a file. I could not find > anywhere that the stderr and stdout of the bitbake command was logged. From > what I see, results.html only shows the configuration parameters. In the log > directory, the $m-build-log.txt only contains a single line: " executing > bitbake command arago-amsdk-image for machine am57xx-evm". Yes, you are right, it's captured by Jenkins, not results.html... I've been meaning to add the per-machine logs to the output and email, but never had time. This basically makes it possible - thanks! > Without this, the only place I can find the complete log is the Jenkins > console in a browser window. Here is an example of what I would like to > enable with this change: > http://158.218.103.230/nightly_builds/mcsdk-8.0.x/75-2014-09-03_13-16-59/artifacts/logs/log-files/am57xx-evm-build-log.txt > > > > -----Original Message----- > From: Dmytriyenko, Denys > Sent: Thursday, September 04, 2014 1:39 PM > To: Stiffler, Jacob > Cc: meta-arago@arago-project.org > Subject: Re: [meta-arago] [tisdk-build-scripts][PATCH 3/4] bitbake-test-sdk-image: Add support to control verbosity of build logging. > > Can you please elaborate on this change? You don't seem to be increasing Bitbake's verbosity level, just enabling capturing stdout and stderr into a file, which gets captured by results.html anyway. What's the purpose? > > > On Thu, Sep 04, 2014 at 01:31:07PM -0400, Jacob Stiffler wrote: > > * Setting BUILD_LOG_VERBOSE="true" (case sensitive) will enable logging stdout > > and stderr for the bitbake command to the build log. > > > > Signed-off-by: Jacob Stiffler > > --- > > lib/oesdk/bitbake-test-sdk-image | 8 ++++++-- > > 1 file changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/lib/oesdk/bitbake-test-sdk-image > > b/lib/oesdk/bitbake-test-sdk-image > > index cff86e4..7d8d093 100644 > > --- a/lib/oesdk/bitbake-test-sdk-image > > +++ b/lib/oesdk/bitbake-test-sdk-image > > @@ -375,12 +375,16 @@ bitbake_sdk_images(){ > > for c in "${bitbake_command_list[@]}" > > do > > log $machine_build_log echo "executing bitbake command $c for machine $m" > > - MACHINE=$m EXTRA_TISDK_FILES="${INSTALLER_EXTRAS_ROOT}/${MACHINE}" TISDK_VERSION="${TISDK_VERSION}" bitbake $c > > + if [ "${BUILD_LOG_VERBOSE}" = "true" ]; then > > + MACHINE=$m EXTRA_TISDK_FILES="${INSTALLER_EXTRAS_ROOT}/${MACHINE}" TISDK_VERSION="${TISDK_VERSION}" bitbake $c 2>&1 | eval tee -a $machine_build_log > > + else > > + MACHINE=$m EXTRA_TISDK_FILES="${INSTALLER_EXTRAS_ROOT}/${MACHINE}" TISDK_VERSION="${TISDK_VERSION}" bitbake $c > > + fi > > > > # Check the return status of the build process. If the build returned > > # a non-zero exit value then log the failure and continue (without > > # testing). A lack of a test file will indicate a test skip > > - if [ "$?" != "0" ] > > + if [ "${PIPESTATUS[0]}" != "0" ] > > then > > echo "FAILED" > $LOG_DIR/$m-build-result.txt > > build_status="fail" > > -- > > 1.7.9.5 > > > > _______________________________________________ > > meta-arago mailing list > > meta-arago@arago-project.org > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago