Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Apr 2023 13:32:54 +0000
From:      Alexey Dokuchaev <danfe@freebsd.org>
To:        Guido Falsi <madpilot@freebsd.org>
Cc:        Daniel Engberg <diizzy@freebsd.org>, ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org
Subject:   Re: git: 72ad532057c9 - main - archivers/zstd: Add CMake config files and adjust Makefile
Message-ID:  <ZC7KBj7lKm8VRtmn@FreeBSD.org>
In-Reply-To: <e77963c0-fdd0-5fc5-049d-aa7853ef5c51@FreeBSD.org>
References:  <202304050615.3356FKNq062154@gitrepo.freebsd.org> <7accd096-7871-29ad-0824-c035504d01b0@FreeBSD.org> <ZC7GUhyVa%2Bjbpe1S@FreeBSD.org> <e77963c0-fdd0-5fc5-049d-aa7853ef5c51@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Apr 06, 2023 at 03:19:55PM +0200, Guido Falsi wrote:
> On 06/04/23 15:17, Alexey Dokuchaev wrote:
> > On Wed, Apr 05, 2023 at 02:20:21PM +0200, Guido Falsi wrote:
> >> On 05/04/23 08:15, Daniel Engberg wrote:
> >>> commit 72ad532057c9f9fb6fa370c80f663c005e9dd021
> >>>
> >>>       archivers/zstd: Add CMake config files and adjust Makefile
> >>>       
> >>>       - Use upstream's short test for "make test"
> >>>       - Use verbose output by default during build
> >>>       * Tweak Makefile to follow Porters Handbook more closely
> >>>       * Install CMake files (taken from CMake build framework)
> >>>       
> >>>       PR:             267652
> >>>       Reported by:    fluffy
> >>>       Approved by:    portmgr (maintainer timeout, 1+ month)
> >>
> >> Looks like this, un[luckily], causes a failure with devel/qt6-base
> >> (abridged error, full log if needed):
> >>
> >> ld: error: relocation R_X86_64_32 cannot be used against local symbol;
> >> recompile with -fPIC
> >>   >>> defined in /usr/local/lib/libzstd.a(zstd_common.o)
> >>   >>> referenced by zstd_common.c
> >>   >>>               zstd_common.o:(ZSTD_versionString) in archive
> > 
> > Yeah, I'm seeing this as well.  Admittedly, the whole commit looks
> > dubious to me, with all these out-of-blue new files and gratuitous
> > changes.  I guess sunpoet@ should revert it and merge only needed
> > parts thereof properly.
> 
> I have only reported the issue as an early warning, as I said.
> 
> Regarding qt6-base a fix has already been proposed here by diizzy:
> 
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=270664

Ah, good to know it's being taken care of, thanks for sharing!

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ZC7KBj7lKm8VRtmn>