[Buildroot] [PATCH 1/2] pkgconf: new package

Gustavo Zacarias gustavo at zacarias.com.ar
Tue Sep 25 21:00:54 UTC 2012


On 09/25/12 17:53, Arnout Vandecappelle wrote:

>  Since we anyway need a wrapper, wouldn't it be more consistent
> to set PKG_CONFIG_PATH there instead?

Yes, it's possible, i can't think of any gain by doing so, but then i
can't think of any loss either.

>  As long as pkg-config exists, I would wrap this in a
> ifeq ($(BR2_PACKAGE_PKG_CONFIG),)

See next point...

>  If for whatever reason, host-pkg-config is also built, the script
> will be overwritten.  So to be safe, I'd remove host-pkg-config as well
> (as a third patch).

Yes, the idea is to remove pkg-config, but i don't think this is a final
patch yet, it's more like a "let people test it and see if nothing
breaks" (things didn't break for a couple of projects i've run with the
patchset, but that may not be the case for everyone else).

>  Although this is just copy-pasted from pkg-config.mk, that's no reason
> to break the buildroot conventions :-)
> 
> PKG_CONFIG_HOST_BINARY = $(HOST_DIR)/usr/bin/pkg-config

Yeah, it was a copy, again, not a final patchset :)
Regards.



More information about the buildroot mailing list