Skip to contentSkip to conversation

package/libarchive: fix static linking against libpthread

PATCH New Bernd Kuhls 2 participants 2 messages Aug 28, 2016 – Oct 16, 2016

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 package/libarchive: fix static linking against libpthread

Fixes
http://autobuild.buildroot.net/results/e94/e943ee5abc06b1812d7b34976e3a807859958935/

This commit also fixes fwup:
http://autobuild.buildroot.net/results/36b/36b0c6b68209ddd1b0f2db8449392af36876411f/

---
v2: Changed LIBARCHIVE_CONF_ENV from using LDFLAGS to using LIBS, this
way -lpthread is included in libarchive.pc fixing linking problems in other packages using libarchive
This patch follows the advice from Khem Raj:
http://lists.busybox.net/pipermail/buildroot/2016-August/169712.html

 package/libarchive/libarchive.mk | 6 ++++++
 1 file changed, 6 insertions(+)
package/libarchive/libarchive.mk +6 -0
index bcf92f9..d007f49 100644
@@ -110,6 +110,12 @@ else
 LIBARCHIVE_CONF_OPTS += --without-lzma
 endif
 
+ifeq ($(BR2_STATIC_LIBS)$(BR2_TOOLCHAIN_HAS_THREADS),yy)
+# libarchive uses pthreads if available, but forgets to link with it
+# breaking static builds
+LIBARCHIVE_CONF_ENV = LIBS="-lpthread"
+endif
+
 # The only user of host-libarchive needs zlib support
 HOST_LIBARCHIVE_DEPENDENCIES = host-zlib
 HOST_LIBARCHIVE_CONF_OPTS = \
Signature · 1 line
2.9.3
#1 lore raw

PATCH v2 1/1 package/libarchive: fix static linking against libpthread

Hello,

Quoted 6 lines · Bernd Kuhls

Fixes
http://autobuild.buildroot.net/results/e94/e943ee5abc06b1812d7b34976e3a807859958935/

This commit also fixes fwup:
http://autobuild.buildroot.net/results/36b/36b0c6b68209ddd1b0f2db8449392af36876411f/

Quoted 7 lines

---
v2: Changed LIBARCHIVE_CONF_ENV from using LDFLAGS to using LIBS, this
way -lpthread is included in libarchive.pc fixing linking problems in other packages using libarchive
This patch follows the advice from Khem Raj:
http://lists.busybox.net/pipermail/buildroot/2016-August/169712.html

With the recent change in uClibc-ng merging libpthread in libc, this patch is no longer needed, so I've marked it as rejected in patchwork. I tested with fwup, and the problem is indeed fixed.

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com