[Buildroot] [PATCH] dnsmasq: fix static linking
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue Mar 4 18:29:27 UTC 2014
Dear Vicente Olivert Riera,
On Tue, 4 Mar 2014 15:14:34 +0000, Vicente Olivert Riera wrote:
> Append -pthread to LIBS to fix a static linking error like this one:
>
> /home/test/test/2/output/host/usr/mipsel-buildroot-linux-uclibc/sysroot/usr/lib/libdbus-1.a(libdbus_1_la-dbus-sysdeps-pthread.o):
> In function `_dbus_platform_cmutex_new':
> dbus-sysdeps-pthread.c:(.text+0x30): undefined reference to
> `pthread_mutex_init'
>
> Fixes:
> http://autobuild.buildroot.net/results/161/161446dde7e8e774773eb2b34fd555f5ac22dd02/
>
> Reviewed-by: Markos Chandras <Markos.Chandras at imgtec.com>
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
Thanks, but I don't think it's the best fix for the problem.
> ifeq ($(BR2_PACKAGE_DBUS),y)
> DNSMASQ_DEPENDENCIES += host-pkgconf dbus
> + DNSMASQ_LIBS += $(if $(BR2_PREFER_STATIC_LIB),-pthread)
> endif
dnsmasq properly uses pkg-config to find which library to link against,
and the dbus-1.pc properly lists -lrt and -lpthread in Libs.private
(i.e libraries needed in the link when linking statically).
The problem is just that the dnsmasq Makefile does not call pkg-config
with the --static option when being built statically. Compare the
output of:
$ ./output/host/usr/bin/pkg-config --libs dbus-1
-L/home/thomas/projets/buildroot/output/host/usr/mipsel-buildroot-linux-uclibc/sysroot/usr/lib -ldbus-1
against the output of:
$ ./output/host/usr/bin/pkg-config --static --libs dbus-1
-L/home/thomas/projets/buildroot/output/host/usr/mipsel-buildroot-linux-uclibc/sysroot/usr/lib -ldbus-1 -lpthread -lrt
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list