[Buildroot] [PATCH v2] pkg-infra: add <pkg>_CONFIG_FIXUP to fix *-config files
Arnout Vandecappelle
arnout at mind.be
Fri Jan 18 15:51:52 UTC 2013
On 18/01/13 13:58, Stefan Fröberg wrote:
> Hi Arnout!
>
> 17.1.2013 10:32, Arnout Vandecappelle kirjoitti:
[snip]
>> A generic solution could be the following: put a script in
>> $(HOST_DIR)/usr/bin (or some other directory) that hands the known
>> arguments to pkg-config and redirects the rest back to the original
>> *-config script. This makes patching of the *-config script
>> unnecessary in most cases.
>>
>
> Sounds good but what about that<pkg>_CONFIG_FIXUP variable that was
> started from Gustavo suggestion (originally from divine-config: fixup
> thread) ?
>
> http://lists.busybox.net/pipermail/buildroot/2013-January/064656.html
>
> Was it now totally waste of time ? :-(
Not at all. First of all, it got the discussion going which may
eventually lead to an acceptable solution. Second, the generic solution
is probably not implementable in the short term. Your patch certainly is,
because it only refactors already-existing fixups into common infrastructure.
I therefore think that your patch should be committed. Will there still
be a v3 or is this it?
>> Something to discuss (again) at the BR developer days?
>>
>>
>>> I noticed that some *-config files have just prefix (and maybe
>>> exec_prefix) but not any includedir or libdir defined inside them
>>> and just give -I/usr/include for --cflags and -L/usr/lib for --libs
>>
>> Even worse! Which one does that?
>>
>
> Well, at least:
>
> giblib-config --cflags gives -I/usr/include
> neon-config --cflags gives -I/usr/include
>
> But Im using older 2012.08 buildroot so maybe they are fixed now?
D'oh, I did a quick check in my allpkgconfig:
for i in staging/usr/bin/*-config; \
do $i --cflags | grep -e '-I/usr' && echo " --- $i"; \
done
24 of the 41 *-config scripts give the wrong cflags (26/40 for --libs)
(roughly 10 scripts need some other argument than --libs/--cflags).
So I'd say that your patch is sorely needed :-)
Regards,
Arnout
[snip]
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list