Re: [PATCH dylan, master] Fix meta-toolchain-qte SDK build for x11-less DISTRO

[Denys Dmytriyenko <denis@denix.org>]

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 <sgw@linux.intel.com> wrote:
> >
> >> On 06/05/2013 10:12 AM, Otavio Salvador wrote:
> >>
> >>>
> >>>
> >>>
> >>> On Wed, Jun 5, 2013 at 1:58 PM, Saul Wold <sgw@linux.intel.com
> >>> <mailto:sgw@linux.intel.com>> 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
> >>>         <mailto:sgw@linux.intel.com>
> >>>         <mailto:sgw@linux.intel.com <mailto: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...

-- 
Denys