On Wed, Jun 5, 2013 at 5:07 PM, Denys Dmytriyenko wrote: > 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 < > sgw@linux.intel.com > > >>> > > >>> >> > > >>> 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/< > 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... > In this case the patch can be merged 'as is'. -- 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