[Buildroot] [PATCH 1/4] libpcap: update static handling

Peter Korsgaard jacmet at uclibc.org
Mon Jan 24 16:25:25 UTC 2011


>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at free-electrons.com> writes:

 Thomas> Hello,
 Thomas> Completely unrelated to Mike's change, but still spotted while looking
 Thomas> at the patch context:

 Thomas> On Wed, 19 Jan 2011 22:10:29 -0500
 Thomas> Mike Frysinger <vapier at gentoo.org> wrote:

 >> LIBPCAP_CONF_ENV:=ac_cv_linux_vers=$(firstword $(subst .,$(space),$(firstword $(call qstrip,$(BR2_DEFAULT_KERNEL_HEADERS))))) \

 Thomas> Both libpcap and tcpdump use this and I never noticed, but this
 Thomas> is completely broken with external toolchains and the
 Thomas> Crosstool-NG backend, since BR2_DEFAULT_KERNEL_HEADERS is only
 Thomas> set with internal toolchains.

 Thomas> I haven't thought of what the right fix would be, but it's definitely
 Thomas> not that one.

True. The question is why they need this info? Perhaps we can just lie
and simply say 2.6.x?

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list