Date: Mon, 3 Dec 2018 07:09:39 +0100 From: Niclas Zeising <zeising@freebsd.org> To: Justin Hibbits <jrh29@alumni.cwru.edu>, Martin Matuska <mm@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers <src-committers@freebsd.org> Subject: Re: svn commit: r340997 - head/lib/libarchive Message-ID: <a394288a-b69e-66ff-dfd8-006b384c31fd@freebsd.org> In-Reply-To: <CAHSQbTB8Aue=HU1nY4H9vQLps9NwxM=Jk2K40mNedxoOMG4Egg@mail.gmail.com> References: <201811262145.wAQLjRwO048495@repo.freebsd.org> <CAHSQbTB8Aue=HU1nY4H9vQLps9NwxM=Jk2K40mNedxoOMG4Egg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 12/2/18 4:03 AM, Justin Hibbits wrote: > On Mon, Nov 26, 2018 at 3:45 PM Martin Matuska <mm@freebsd.org> wrote: >> >> Author: mm >> Date: Mon Nov 26 21:45:27 2018 >> New Revision: 340997 >> URL: https://svnweb.freebsd.org/changeset/base/340997 >> >> Log: >> libarchive configuration changes >> - move HAVE_BZLIB_H, HAVE_LIBLZMA and HAVE_LZMA_H to config_freebsd.h >> - activate support for multi-threaded lzma encoding [1] >> >> PR: 233543 [1] >> Reported by: cem >> MFC after: 1 week >> >> Modified: >> head/lib/libarchive/Makefile >> head/lib/libarchive/config_freebsd.h >> >> Modified: head/lib/libarchive/Makefile >> ============================================================================== >> --- head/lib/libarchive/Makefile Mon Nov 26 20:56:05 2018 (r340996) >> +++ head/lib/libarchive/Makefile Mon Nov 26 21:45:27 2018 (r340997) >> @@ -7,7 +7,6 @@ _LIBARCHIVEDIR= ${SRCTOP}/contrib/libarchive >> LIB= archive >> >> LIBADD= z bz2 lzma bsdxml >> -CFLAGS+= -DHAVE_BZLIB_H=1 -DHAVE_LIBLZMA=1 -DHAVE_LZMA_H=1 >> >> # FreeBSD SHLIB_MAJOR value is managed as part of the FreeBSD system. >> # It has no real relation to the libarchive version number. >> >> Modified: head/lib/libarchive/config_freebsd.h >> ============================================================================== >> --- head/lib/libarchive/config_freebsd.h Mon Nov 26 20:56:05 2018 (r340996) >> +++ head/lib/libarchive/config_freebsd.h Mon Nov 26 21:45:27 2018 (r340997) >> @@ -133,14 +133,17 @@ >> #define HAVE_LCHFLAGS 1 >> #define HAVE_LCHMOD 1 >> #define HAVE_LCHOWN 1 >> +#define HAVE_LIBLZMA 1 >> #define HAVE_LIBZ 1 >> #define HAVE_LIMITS_H 1 >> #define HAVE_LINK 1 >> +#define HAVE_LZMA_H 1 >> #define HAVE_LOCALE_H 1 >> #define HAVE_LOCALTIME_R 1 >> #define HAVE_LONG_LONG_INT 1 >> #define HAVE_LSTAT 1 >> #define HAVE_LUTIMES 1 >> +#define HAVE_LZMA_STREAM_ENCODER_MT 1 >> #define HAVE_MBRTOWC 1 >> #define HAVE_MEMMOVE 1 >> #define HAVE_MEMORY_H 1 >> > > This breaks ports-mgmt/pkg now, with the following failure log: > > --- pkg-static --- > /usr/lib/liblzma.a(stream_encoder_mt.o): In function `mythread_cond_init': > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:230: > undefined reference to `pthread_condattr_init' > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:233: > undefined reference to `pthread_condattr_setclock' > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:237: > undefined reference to `pthread_condattr_destroy' > /usr/lib/liblzma.a(stream_encoder_mt.o): In function `get_thread': > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:237: > undefined reference to `pthread_condattr_destroy' > /usr/lib/liblzma.a(stream_encoder_mt.o): In function `mythread_cond_init': > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:233: > undefined reference to `pthread_condattr_setclock' > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:237: > undefined reference to `pthread_condattr_destroy' > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:230: > undefined reference to `pthread_condattr_init' > /usr/local/poudriere/jails/ppc64/usr/src/contrib/xz/src/common/mythread.h:237: > undefined reference to `pthread_condattr_destroy' > *** [pkg-static] Error code 1 > I'm seeing the same issue when builing i386 packages on amd64 using poudriere. For some reason the amd64 build is fine. This needs to be reverted or fixed asap. In the diff above it also looks like the -DHAVE_BZLIB_H=1 flag was forgottgen when moving to config_freebsd.h, but perhaps this was intentional. Regards -- Niclas Zeising
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?a394288a-b69e-66ff-dfd8-006b384c31fd>