[Buildroot] [PATCH] bandwidthd: fix static build
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sun Oct 5 12:40:43 UTC 2014
Baruch, Nathaniel,
On Sun, 5 Oct 2014 07:50:37 +0300, Baruch Siach wrote:
> > Hi Baruch, I didn't end up using pcap-config, but could you check git
> > version 4b07a0b3d3a280cdde582060cb29f3333ba4bf6e for me? It seems to
> > work in the config you pasted above and I haven't had any issues in my
> > other tests.
>
> I can confirm that upgrading bandwidthd to
> 4b07a0b3d3a280cdde582060cb29f3333ba4bf6e fixes the build of the config above.
> I still think that holding the complete knowledge of all your indirect
> dependencies, mandatory and optional, is not robust. A better future proof
> solution IMO is to use tools like pkg-config (or pcap-config in the case of
> libpcap) to list all dependencies that are in actual use for this specific
> build. But this is your call as upstream.
I definitely agree with Baruch here. Using pkg-config is much more
robust, as it figures out the indirect dependencies automatically,
without hardcoding them in bandwidthd and/or Buildroot.
However, Baruch patch doesn't seem to be fully correct: it does both a
PKG_CHECK_MODULES on libpng, and an AC_CHECK_LIB, which seems a bit
redundant.
Therefore, Nathaniel, since you're now the upstream developer, could
you make the necessary changes in configure.ac to use pkg-config
instead, and then send a patch updating Buildroot to a new version of
bandwidthd ?
In the mean time, I'll mark:
http://patchwork.ozlabs.org/patch/395809/
http://patchwork.ozlabs.org/patch/381899/
as 'Changes requested'.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list