From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mail.openembedded.org (Postfix) with ESMTP id 923926A598 for ; Wed, 5 Jun 2013 17:19:54 +0000 (UTC) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP; 05 Jun 2013 10:19:52 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.87,808,1363158000"; d="scan'208";a="348698994" Received: from unknown (HELO [10.255.13.19]) ([10.255.13.19]) by orsmga002.jf.intel.com with ESMTP; 05 Jun 2013 10:19:50 -0700 Message-ID: <51AF7336.8050802@linux.intel.com> Date: Wed, 05 Jun 2013 10:19:50 -0700 From: Saul Wold User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 MIME-Version: 1.0 To: Otavio Salvador References: <1370417067-30166-1-git-send-email-nicolas.dechesne@linaro.org> <51AF67B8.20502@linux.intel.com> <51AF6E2F.8040309@linux.intel.com> In-Reply-To: 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 17:19:55 -0000 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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! > -- > Otavio Salvador O.S. Systems > http://www.ossystems.com.br http://projetos.ossystems.com.br > Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750