From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [143.182.124.37]) by mx1.pokylinux.org (Postfix) with ESMTP id 26EDF4C802F2 for ; Wed, 6 Apr 2011 06:16:31 -0500 (CDT) Received: from azsmga001.ch.intel.com ([10.2.17.19]) by azsmga102.ch.intel.com with ESMTP; 06 Apr 2011 04:16:31 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.63,309,1299484800"; d="scan'208";a="414703682" Received: from unknown (HELO [10.255.12.213]) ([10.255.12.213]) by azsmga001.ch.intel.com with ESMTP; 06 Apr 2011 04:16:30 -0700 From: Joshua Lock To: Richard Purdie Date: Wed, 06 Apr 2011 12:16:29 +0100 In-Reply-To: <1301957183.24596.414.camel@rex> References: <8d834327e2d84b8d4ad72b9c617d261ab8b206e2.1301664697.git.josh@linux.intel.com> <1301957183.24596.414.camel@rex> X-Mailer: Evolution 2.91.92 (2.91.92-2.fc15) Message-ID: <1302088591.2355.7.camel@scimitar> Mime-Version: 1.0 Cc: poky@yoctoproject.org Subject: Re: [PATCH 1/7] elfutils: fix builds with gcc 4.6 X-BeenThere: poky@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Poky build system developer discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Apr 2011 11:16:32 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Mon, 2011-04-04 at 23:46 +0100, Richard Purdie wrote: > On Mon, 2011-04-04 at 23:37 +0200, Martin Jansa wrote: > > On Fri, Apr 1, 2011 at 3:40 PM, Joshua Lock wrote: > > > From: Joshua Lock > > > > > > gcc 4.6 (as used in Fedora 15) adds some extra warnings which are > > > included with Werror. The new unused-but-set variable warning causes > > > an error in libasm of elfutils. Work around this by removing > > > unused-but-set from Werror. > > > > > > Signed-off-by: Joshua Lock > > > --- > > > meta/recipes-devtools/elfutils/elfutils_0.148.bb | 4 ++++ > > > 1 files changed, 4 insertions(+), 0 deletions(-) > > > > > > diff --git a/meta/recipes-devtools/elfutils/elfutils_0.148.bb b/meta/recipes-devtools/elfutils/elfutils_0.148.bb > > > index b2f700e..c395be8 100644 > > > --- a/meta/recipes-devtools/elfutils/elfutils_0.148.bb > > > +++ b/meta/recipes-devtools/elfutils/elfutils_0.148.bb > > > @@ -40,6 +40,10 @@ SRC_URI += "\ > > > > > > inherit autotools > > > > > > +# GCC 4.6.0 raises an unused-but-set warning in libasm, for now remove > > > +# this warning from Werror > > > +CFLAGS_virtclass-native += "-Wno-error=unused-but-set-variable" > > > + > > > EXTRA_OECONF = "--program-prefix=eu-" > > > > this breaks all other distros not yet using gcc-4.6.. > > > > > > configure:2892: checking whether the C compiler works > > configure:2914: gcc -Wno-error=unused-but-set-variable > > -isystem/OE/shr-core/tmp/sysroots/x86_64-linux/usr/include > > -L/OE/shr-core/tmp/sysroots/x86_64-linux/usr/lib > > -Wl,-rpath-link,/OE/shr-core/tmp/sysroots/x86_64-linux/usr/lib > > -Wl,-rpath,/OE/shr-core/tmp/sysroots/x86_64-linux/usr/lib -Wl,-O1 > > conftest.c >&5 > > cc1: error: -Werror=unused-but-set-variable: No option -Wunused-but-set-variable > > configure:2918: $? = 1 > > configure:2956: result: no > > configure: failed program was: > > | /* confdefs.h */ > > | #define PACKAGE_NAME "Red Hat elfutils" > > | #define PACKAGE_TARNAME "elfutils" > > | #define PACKAGE_VERSION "0.148" > > | #define PACKAGE_STRING "Red Hat elfutils 0.148" > > | #define PACKAGE_BUGREPORT "http://bugzilla.redhat.com/bugzilla/" > > | #define PACKAGE_URL "" > > | #define PACKAGE "elfutils" > > | #define VERSION "0.148" > > | #define MODVERSION "Build on jama 2011-04-04T23:34:25+0200" > > | /* end confdefs.h. */ > > | > > | int > > | main () > > | { > > | > > | ; > > | return 0; > > | } > > configure:2961: error: in > > `/OE/shr-core/tmp/work/x86_64-linux/elfutils-native-0.148-r1/elfutils-0.148': > > configure:2965: error: C compiler cannot create executables > > See `config.log' for more details. Sorry about this Martin. Thanks for catching and reporting. > > Ouch, I'm going to revert this, we'll need an alternative fix. Thanks > for reporting! I've worked up an alternative fix and sent a pull request to oe-core. I took the approach of consolidating several patches from the upstream source repository which remove the unused variables. I consolidated them into a single patch so that they are easier to drop once we update elfutils to 0.152 or higher. Regards, Joshua -- Joshua Lock Yocto Build System Monkey Intel Open Source Technology Centre