[Buildroot] [PATCH 1/2] package/lldpd: disable use of libbsd
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Thu Aug 18 21:24:14 UTC 2016
Hello,
On Thu, 18 Aug 2016 23:01:21 +0200, Yann E. MORIN wrote:
> libbsd installs at least one .pc file that contains a -isystem include
> path (-isystem /usr.include/bsd), but -isystem is not munged by
> pkg-config, so we end up using /usr/include/bsd which is not suitable
> for the target. And breaks big time (see failures, below).
>
> But using libbsd is completely optional. In fact, our lldpd.mk did not
> even express the dependency on libbsd, so we may well have had builds
> without libbsd.
>
> The functionality brought in by using libbsd is very minor (just setting
> the neighbour name in /proc/self/smdline), so we just make lldpd not use
> libbsd at all. There is no --disable-libbsd or such, so we just patch it
> out of configure.ac.
I don't like this fix, for two reasons:
* The patch is not upstreamable.
* It works around the problem, living a potentially problematic
libbsd.pc installed which can be used by other packages than lldpd.
Isn't the right solution to add support in pkg-config to mungle
-isystem flags?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list