From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from vms173015pub.verizon.net (vms173015pub.verizon.net [206.46.173.15]) by mail.openembedded.org (Postfix) with ESMTP id 7735C6065C for ; Wed, 5 Jun 2013 20:07:52 +0000 (UTC) Received: from gandalf.denix.org ([unknown] [108.45.150.102]) by vms173015.mailsrvcs.net (Sun Java(tm) System Messaging Server 7u2-7.02 32bit (built Apr 16 2009)) with ESMTPA id <0MNX001CRRWS1OX0@vms173015.mailsrvcs.net> for openembedded-core@lists.openembedded.org; Wed, 05 Jun 2013 15:07:42 -0500 (CDT) Received: by gandalf.denix.org (Postfix, from userid 1000) id 546F2200E2; Wed, 05 Jun 2013 16:07:40 -0400 (EDT) Date: Wed, 05 Jun 2013 16:07:40 -0400 From: Denys Dmytriyenko To: Otavio Salvador Message-id: <20130605200740.GF10298@denix.org> References: <1370417067-30166-1-git-send-email-nicolas.dechesne@linaro.org> <51AF67B8.20502@linux.intel.com> <51AF6E2F.8040309@linux.intel.com> <51AF7336.8050802@linux.intel.com> MIME-version: 1.0 In-reply-to: User-Agent: Mutt/1.5.20 (2009-06-14) Cc: Patches and discussions about the oe-core layer Subject: Re: [PATCH dylan, master] Fix meta-toolchain-qte SDK build for x11-less DISTRO 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: Wed, 05 Jun 2013 20:07:53 -0000 Content-type: text/plain; charset=us-ascii Content-disposition: inline On Wed, Jun 05, 2013 at 02:43:07PM -0300, Otavio Salvador wrote: > On Wed, Jun 5, 2013 at 2:23 PM, Nicolas Dechesne < > nicolas.dechesne@linaro.org> wrote: > > > > > > > > > On Wed, Jun 5, 2013 at 7:19 PM, Saul Wold wrote: > > > >> On 06/05/2013 10:12 AM, Otavio Salvador wrote: > >> > >>> > >>> > >>> > >>> On Wed, Jun 5, 2013 at 1:58 PM, Saul Wold >>> > wrote: > >>> > >>> On 06/05/2013 09:32 AM, Nicolas Dechesne wrote: > >>> > >>> > >>> On Wed, Jun 5, 2013 at 6:30 PM, Saul Wold >>> > >>> >> > >>> wrote: > >>> > >>> You could just directly put the nativesdk-libx11 in place > >>> of the > >>> variable, no need to have the variable there. > >>> > >>> > >>> yes, that's what I had initially, but found it was less easy to > >>> read... > >>> with X11DEPENDS it's more 'obvious' that there is something > >>> special.. > >>> that said, i can make the change if that's really needed. > >>> > >>> We do use the X11DEPENDS elsewhere when there are multiple > >>> dependencies, but I also found cases where we just include the > >>> dependency directly in the test. I was trying pick a direction: > >>> single entry no X11DEPENDS, multiple entries use X11DEPENDS. > >>> > >>> Comments, flames, ... > >>> > >>> > >>> Yes; I sent this patch in Febuary: > >>> http://patchwork.openembedded.**org/patch/44759/ > >>> > >>> Please use this one instead of the recent one. > >>> > >>> > >> Well reading back on that, it looks like I was waiting for an > >> EXTRA_OECONF or related change to the autoconf scripts. > >> > >> Sau! > > > > > > hmm. ok, sorry Otavio, i missed the other patch. I will check on my side > > too about EXTRA_OECONF. > > > > Nicolas, don't worry. It is normal to end redoing some stuff. > > Last time I checked it had no support in Qt build system; I am not sure if > it uses or not the host headers (in case they exist) but it needs testing > to be sure. I can confirm that it does not link against host X11 when built w/o that dependency and nativesdk has no x11 libs/headers. As I previously mentioned, we've been using this fix for over 6 months on several releases built on different machines w/o problems... -- Denys