[Buildroot] [PATCH] qt5: Disable passing of -isystem flag in CXXFLAGS

Arnout Vandecappelle arnout at mind.be
Thu Sep 29 21:34:17 UTC 2016



On 29-09-16 23:05, Arnout Vandecappelle wrote:
> 
> 
> On 28-09-16 10:54, Alexey Brodkin wrote:
>> So it all boils down to:
>> 1) "pkg-config" case:
>> --------------------->8--------------------
>> PKG_CONFIG_LIBDIR=${PKG_CONFIG_LIBDIR:-/XXX/output/host/usr/arc-buildroot-linux-uclibc/sysroot/usr/lib/pkgconfig}
>> PKG_CONFIG_SYSROOT_DIR=${PKG_CONFIG_SYSROOT_DIR:-/XXX/output/host/usr/arc-buildroot-linux-uclibc/sysroot}
>> ./output/host/usr/bin/pkgconf --libs icu-i18n
>> -licui18n -L/XXX/output/host/usr/arc-buildroot-linux-uclibc/sysroot/usr/lib -licuuc -licudata
>> --------------------->8--------------------
>>
>> 2. Pure "pkgconf" case:
>> --------------------->8--------------------
>> ./output/host/usr/bin/pkgconf --libs icu-i18n
>> -licui18n -licuuc -licudata
>> --------------------->8--------------------
> 
>  Your pure pkgconf case is using the host icu-i18n, that's why the -L flag is
> missing. The host icu-i18n doesn't have a -L bit in its Libs: stanza.

 Hm, actually, icu-uc.pc *does* have a -L bit in its Libs: stanza. But for the
host, I guess it recognizes that ${libdir} == /usr/lib so it can be eliminated.
So perhaps that can be extended to cover a sysroot as well.

 BTW, I did a quick test with pkgconf 1.0.1 and it still behaves the same.

 Regards,
 Arnout

> 
>  You should compare:
> 
> 1) pkgconf case
> PKG_CONFIG_LIBDIR=$STAGING_DIR/usr/lib/pkgconfig
> PKG_CONFIG_SYSROOT_DIR=$STAGING_DIR/sysroot output/host/usr/bin/pkgconf --libs
> icu-i18n
> -licui18n -L$STAGING_DIR/usr/lib -licuuc -licudata
> 
> 2) Real pkg-config case:
> PKG_CONFIG_LIBDIR=$STAGING_DIR/usr/lib/pkgconfig
> PKG_CONFIG_SYSROOT_DIR=$STAGING_DIR/sysroot /usr/bin/pkg-config --libs icu-i18n
> -licui18n -licuuc -licudata
> 
>  We switched from pkg-config to pkgconf because it builds a whole lot faster
> (pkg-config has a whole lot of dependencies). Obviously we give up features with
> that, but this is the first one where I think it is an important feature that
> we're missing.
> 
>  Regards,
>  Arnout
> 
>>
>>>>  So ideally, we should patch pkgconf (and send upstream) to remove the default
>>>> search paths.
>> Given my comments above I'm not really sure it's pkgconf who's guilty.
> 

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF


More information about the buildroot mailing list