From: "Aníbal Limón" <anibal.limon@linux.intel.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org,
benjamin.esquivel@intel.com,
"Aníbal Limón" <limon.anibal@gmail.com>
Subject: Re: [PATCH 15/20] classes/populate_sdk_ext: Add OE_SDK_EXT_SILENT env variable
Date: Tue, 2 Feb 2016 15:23:54 -0600 [thread overview]
Message-ID: <56B11E6A.6080005@linux.intel.com> (raw)
In-Reply-To: <35476879.65FcALFN4F@peggleto-mobl.ger.corp.intel.com>
[-- Attachment #1: Type: text/plain, Size: 2110 bytes --]
Because this is disabled when SDK tests (compatibility) ran using eSDK
so sometimes needs to SILENT it when automatic process is working
another times don't like when user uses it.
In this case the SDK tests fails because it looks to stdout and found
devtool msg.
alimon
On 02/02/2016 03:19 PM, Paul Eggleton wrote:
> On Tue, 02 Feb 2016 09:14:18 Aníbal Limón wrote:
>> From: Aníbal Limón <limon.anibal@gmail.com>
>>
>> Sometimes we need to load environment without show help information
>> only useful when user is interacting with the SDK.
>>
>> For example: For test extensible SDK.
>>
>> Signed-off-by: Aníbal Limón <limon.anibal@gmail.com>
>> ---
>> meta/classes/populate_sdk_ext.bbclass | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/classes/populate_sdk_ext.bbclass
>> b/meta/classes/populate_sdk_ext.bbclass index 76308ec..fc96a4b 100644
>> --- a/meta/classes/populate_sdk_ext.bbclass
>> +++ b/meta/classes/populate_sdk_ext.bbclass
>> @@ -291,7 +291,7 @@ sdk_ext_postinst() {
>> # so put it at the end of $PATH.
>> echo "export
>> PATH=\$PATH:$target_sdk_dir/sysroots/${SDK_SYS}/${bindir_nativesdk}" >>
>> $env_setup_script
>>
>> - echo "printf 'SDK environment now set up; additionally you may now run
>> devtool to perform development tasks.\nRun devtool --help for further
>> details.\n'" >> $env_setup_script + echo "[ -z \$OE_SDK_EXT_SILENT ] &&
>> printf 'SDK environment now set up; additionally you may now run devtool to
>> perform development tasks.\nRun devtool --help for further details.\n'" >>
>> $env_setup_script
>>
>> # Warn if trying to use external bitbake and the ext SDK together
>> echo "(which bitbake > /dev/null 2>&1 && echo 'WARNING: attempting to use
>> the extensible SDK in an environment set up to run bitbake - this may lead
>> to unexpected results. Please source this script in a new shell session
>> instead.') || true" >> $env_setup_script
>
> Rather than adding another variable for this why not just redirect the output
> to /dev/null ?
>
> Cheers,
> Paul
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2016-02-02 21:22 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-02 15:14 [PATCH 00/20] Add Extensible SDK test suite Aníbal Limón
2016-02-02 15:14 ` [PATCH 01/20] testimage: Modularize helper functions for get test lists Aníbal Limón
2016-02-02 15:14 ` [PATCH 02/20] classes/testsdk: Add new class testsdk Aníbal Limón
2016-02-02 15:14 ` [PATCH 03/20] classes/testimage: Add defeault inherit for testsdk Aníbal Limón
2016-02-02 15:14 ` [PATCH 04/20] populate_sdk_ext: Set TOOLCHAINEXT_OUTPUTNAME Aníbal Limón
2016-02-02 15:14 ` [PATCH 05/20] get_test_suites: Add sdkext type for load test suites Aníbal Limón
2016-02-02 15:14 ` [PATCH 06/20] classes/testsdk: Add testsdkext task only install Aníbal Limón
2016-02-02 15:14 ` [PATCH 07/20] bb/fetch2: Move export_proxies function from wget to utils Aníbal Limón
2016-02-02 15:14 ` [PATCH 08/20] classes/testsdk: Add call to export_proxies on testsdkext Aníbal Limón
2016-02-02 15:14 ` [PATCH 09/20] toolchain-shar-extract.sh: Add proxy variable to new env Aníbal Limón
2016-02-02 15:14 ` [PATCH 10/20] testimage/testsdk: Modularize TestContext Aníbal Limón
2016-02-02 15:14 ` [PATCH 11/20] testimage/testsdk: Move get test suites routine inside TestContext Aníbal Limón
2016-02-02 15:14 ` [PATCH 12/20] oetest.py/TestContext: Move loadTests and runTests inside it Aníbal Limón
2016-02-02 15:14 ` [PATCH 13/20] oeqa/oetest.py: Fix missing oeqa.runtime import Aníbal Limón
2016-02-02 21:25 ` Paul Eggleton
2016-02-02 21:31 ` Aníbal Limón
2016-02-02 21:32 ` Paul Eggleton
2016-02-02 15:14 ` [PATCH 14/20] classes/testsdk: Add function run_test_context Aníbal Limón
2016-02-02 15:14 ` [PATCH 15/20] classes/populate_sdk_ext: Add OE_SDK_EXT_SILENT env variable Aníbal Limón
2016-02-02 21:19 ` Paul Eggleton
2016-02-02 21:23 ` Aníbal Limón [this message]
2016-02-02 21:25 ` Paul Eggleton
2016-02-02 21:30 ` Aníbal Limón
2016-02-02 21:31 ` Paul Eggleton
2016-02-02 21:38 ` Aníbal Limón
2016-02-02 21:40 ` Paul Eggleton
2016-02-02 21:47 ` Aníbal Limón
2016-02-02 21:49 ` Paul Eggleton
2016-02-02 15:14 ` [PATCH 16/20] classes/testsdk: Add compatibility SDK testsuite to eSDK Aníbal Limón
2016-02-02 15:14 ` [PATCH 17/20] testsdkext: Add skeleton for support Extensible SDK tests Aníbal Limón
2016-02-02 15:14 ` [PATCH 18/20] classes/populate_sdk_ext: Add SDK_EXT_TARGET_MANIFEST and SDK_EXT_HOST_MANIFEST Aníbal Limón
2016-02-02 21:52 ` Paul Eggleton
2016-02-02 22:05 ` Aníbal Limón
2016-02-02 22:06 ` Paul Eggleton
2016-02-02 22:13 ` Aníbal Limón
2016-02-02 22:15 ` Paul Eggleton
2016-02-02 15:14 ` [PATCH 19/20] oeqa/sdkext: Add devtool basic tests for eSDK Aníbal Limón
2016-02-02 22:03 ` Paul Eggleton
2016-02-02 22:14 ` Aníbal Limón
2016-02-02 15:14 ` [PATCH 20/20] classes/testsdk: Add help information on how to run tests Aníbal Limón
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56B11E6A.6080005@linux.intel.com \
--to=anibal.limon@linux.intel.com \
--cc=benjamin.esquivel@intel.com \
--cc=limon.anibal@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=paul.eggleton@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox