From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.windriver.com (mail.windriver.com [147.11.1.11]) by mail.openembedded.org (Postfix) with ESMTP id 01DC960667 for ; Fri, 2 Aug 2013 02:08:26 +0000 (UTC) Received: from ALA-HCA.corp.ad.wrs.com (ala-hca.corp.ad.wrs.com [147.11.189.40]) by mail.windriver.com (8.14.5/8.14.3) with ESMTP id r7228QYU014415 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Thu, 1 Aug 2013 19:08:26 -0700 (PDT) Received: from [128.224.162.233] (128.224.162.233) by ALA-HCA.corp.ad.wrs.com (147.11.189.50) with Microsoft SMTP Server (TLS) id 14.2.342.3; Thu, 1 Aug 2013 19:08:26 -0700 Message-ID: <51FB14A3.9040706@windriver.com> Date: Fri, 2 Aug 2013 10:08:35 +0800 From: ChenQi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Saul Wold References: <51FAA515.6050302@linux.intel.com> In-Reply-To: <51FAA515.6050302@linux.intel.com> X-Originating-IP: [128.224.162.233] Cc: Zhangle.Yang@windriver.com, openembedded-core@lists.openembedded.org Subject: Re: [PATCH 1/1] buildtools-tarball: unset OECORE_NATIVE_SYSROOT X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Aug 2013 02:08:27 -0000 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit On 08/02/2013 02:12 AM, Saul Wold wrote: > On 07/29/2013 06:50 PM, Qi.Chen@windriver.com wrote: >> From: Chen Qi >> >> When building a qemu image inside the environment created by the >> buildtools-tarball, the qemu image cannot be started, as the runqemu >> script uses the tunctl binary which cannot be found inside the sysroot >> directory of the buildtools-tarball. >> >> The buildtools-tarball is inherently a tool set instead of a fully >> functional SDK, so leaving the OECORE_NATIVE_SYSROOT variable in the >> environment will mess things up. >> >> However, we do need a line of 'OECORE_NATIVE_SYSROOT=xxx' in the >> environment >> setup script so that the SDK can be extracted and relocated correctly. >> > Where is this being processed, can could be done as a since comment in > the the environment-setup script instead of exporting and unseting it? > Ah.... I got what you mean. It's something like: --- a/meta/recipes-core/meta/buildtools-tarball.bb +++ b/meta/recipes-core/meta/buildtools-tarball.bb @@ -59,7 +59,7 @@ create_sdk_files_append () { script=${1:-${SDK_OUTPUT}/${SDKPATH}/environment-setup-${SDK_SYS}} touch $script echo 'export PATH=${SDKPATHNATIVE}${bindir_nativesdk}:$PATH' >> $script - echo 'export OECORE_NATIVE_SYSROOT="${SDKPATHNATIVE}"' >> $script + echo '#OECORE_NATIVE_SYSROOT="${SDKPATHNATIVE}"' >> $script toolchain_create_sdk_version ${SDK_OUTPUT}/${SDKPATH}/version-${SDK_SYS} } I'll test this and send out a V2 if it works. Thanks, Chen Qi > Sau! >> That's why this patch unsets the variable instead of removing it from >> the >> environment setup script. >> >> [YOCTO #4939] >> >> Signed-off-by: Chen Qi >> --- >> meta/recipes-core/meta/buildtools-tarball.bb | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/meta/recipes-core/meta/buildtools-tarball.bb >> b/meta/recipes-core/meta/buildtools-tarball.bb >> index 9771497..1f8f142 100644 >> --- a/meta/recipes-core/meta/buildtools-tarball.bb >> +++ b/meta/recipes-core/meta/buildtools-tarball.bb >> @@ -59,7 +59,11 @@ create_sdk_files_append () { >> script=${1:-${SDK_OUTPUT}/${SDKPATH}/environment-setup-${SDK_SYS}} >> touch $script >> echo 'export PATH=${SDKPATHNATIVE}${bindir_nativesdk}:$PATH' >> >> $script >> + # OECORE_NATIVE_SYSROOT variable needs to be in $script so that the >> + # relocate script can find the ld-linux.so. >> echo 'export OECORE_NATIVE_SYSROOT="${SDKPATHNATIVE}"' >> $script >> - >> + # buildtools-tarball is inherently a tool set instead of a fully >> functional SDK. >> + # Leaving OECORE_NATIVE_SYSROOT in environment will mess things up. >> + echo 'unset OECORE_NATIVE_SYSROOT' >> $script >> toolchain_create_sdk_version >> ${SDK_OUTPUT}/${SDKPATH}/version-${SDK_SYS} >> } >> > >