From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dan.rpsys.net (5751f4a1.skybroadband.com [87.81.244.161]) by mail.openembedded.org (Postfix) with ESMTP id 6367D7265F for ; Mon, 22 Dec 2014 09:41:52 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by dan.rpsys.net (8.14.4/8.14.4/Debian-4.1ubuntu1) with ESMTP id sBM9fAtC030422; Mon, 22 Dec 2014 09:41:10 GMT Received: from dan.rpsys.net ([127.0.0.1]) by localhost (dan.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id etaHCpm6llRA; Mon, 22 Dec 2014 09:41:10 +0000 (GMT) Received: from [192.168.3.10] ([192.168.3.10]) (authenticated bits=0) by dan.rpsys.net (8.14.4/8.14.4/Debian-4.1ubuntu1) with ESMTP id sBM9eqSd030414 (version=TLSv1/SSLv3 cipher=AES128-GCM-SHA256 bits=128 verify=NOT); Mon, 22 Dec 2014 09:41:04 GMT Message-ID: <1419241289.13316.67.camel@linuxfoundation.org> From: Richard Purdie To: Kai Kang , Koen Kooi Date: Mon, 22 Dec 2014 09:41:29 +0000 In-Reply-To: <38c83309734f903e97c3b8e51b0d6c501d1f2ded.1418885581.git.kai.kang@windriver.com> References: <38c83309734f903e97c3b8e51b0d6c501d1f2ded.1418885581.git.kai.kang@windriver.com> X-Mailer: Evolution 3.12.7-0ubuntu1 Mime-Version: 1.0 Cc: openembedded-core@lists.openembedded.org Subject: Re: [PATCH 08/19] libpng 1.6.13: fix build for aarch64 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: Mon, 22 Dec 2014 09:42:00 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Thu, 2014-12-18 at 16:51 +0800, Kai Kang wrote: > From: Koen Kooi > > The configure override was too restrictive, it needed both 'arm' and > 'neon' to trigger, which breaks on aarch64. Since TUNE_FEATURES is the > only qualifier that matters, drop the 'arm' override. > > Buildtested for 'genericarmv8' and 'qemux86' machines. > > Signed-off-by: Koen Kooi > --- > meta/recipes-multimedia/libpng/libpng_1.6.13.bb | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb > index 0c6fd1f..8798a96 100644 > --- a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb > +++ b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb > @@ -19,7 +19,7 @@ BINCONFIG = "${bindir}/libpng-config ${bindir}/libpng16-config" > inherit autotools binconfig-disabled pkgconfig > > # Work around missing symbols > -EXTRA_OECONF_append_arm = " ${@bb.utils.contains("TUNE_FEATURES", "neon", "--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}" > +EXTRA_OECONF_append = " ${@bb.utils.contains("TUNE_FEATURES", "neon", "--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}" > > PACKAGES =+ "${PN}-tools" > I tried a test build of this series and see: https://autobuilder.yoctoproject.org/main/builders/nightly-arm/builds/136/steps/BuildImages_1/logs/stdio in the output. Could someone look into that please? I'm guessing its related to this patch although I haven't bisected exactly. Cheers, Richard