From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mail.openembedded.org (Postfix) with ESMTP id 2EADA65D94 for ; Mon, 14 Apr 2014 08:35:40 +0000 (UTC) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 14 Apr 2014 01:35:39 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,855,1389772800"; d="scan'208";a="520035231" Received: from vpopa-desktop.rb.intel.com (HELO [10.237.105.76]) ([10.237.105.76]) by fmsmga002.fm.intel.com with ESMTP; 14 Apr 2014 01:35:35 -0700 Message-ID: <534B9DDD.4060403@intel.com> Date: Mon, 14 Apr 2014 11:35:41 +0300 From: Valentin Popa User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Robert Yang , Otavio Salvador , Richard Purdie References: <533F8B7C.8010409@windriver.com> <5342BC44.2000708@intel.com> <5344BADD.5090106@windriver.com> <5348B401.20908@windriver.com> In-Reply-To: <5348B401.20908@windriver.com> Cc: "meta-freescale@yoctoproject.org" , Patches and discussions about the oe-core layer Subject: Re: [dora][regression] mesa failed to build 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, 14 Apr 2014 08:35:45 -0000 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 04/12/2014 06:33 AM, Robert Yang wrote: > > > On 04/11/2014 09:56 PM, Otavio Salvador wrote: >> Hello, >> >> On Wed, Apr 9, 2014 at 12:13 AM, Robert Yang >> wrote: >>> On 04/07/2014 10:55 PM, Valentin Popa wrote: >>>> Indeed the build failure was introduced by that patch, which enters a >>>> logical >>>> conflict with the bbappend file for mesa. >>>> To remain compatible with the future releases I suggest to not >>>> remove/add >>>> flags >>>> from/to EXTRA_OECONF explicitly, and make use of what PACKAGECONFIG >>>> contains. >>>> The first part of the __anonymous function from the bbappend file >>>> can be >>>> simply >>>> replaced with: >>>> >>>> PACKAGECONFIG_remove = "egl" >>>> >>>> and the second part with: >>>> >>>> PROVIDES_remove = "virtual/libgles1 virtual/libgles2 virtual/egl" >>>> and >>>> PROVIDES_remove = "virtual/libgl" if mx6 is in SOC_FAMILY. >>>> >>> >>> Hi Otavio, >>> >>> Does Valentin's suggestions work for you, please? >> >> I am traveling and won't be able to test it. >> >> However I am quite surprise this didn't come out /before/ when the >> dora updates were in test in AB since meta-fsl-arm is tested in AB >> too. We need to figure /why/ this broke and for now I think we ought >> to revert this dora patch. >> >> This kind of change needs to be coordinated and raise a build break in >> an Yocto Project is unacceptable IMO. >> >> For me it is clear this didn't run in AB before merging. :( >> > > I'm sorry about that, I had run it on my *local* AB before merging, but > the local AB is less powerful and doesn't include the meta-fsl-arm layer, > I'm fine to revert it, what's RP and valentin's opinion, please ? > > // Robert > > > I'll send a patch to handle this special case for meta-fsl-arm (mutate eglplatform.h only if it exists)