Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Apr 2023 15:00:04 +0200
From:      Matthias Andree <matthias.andree@tu-dortmund.de>
To:        Dima Panov <fluffy@FreeBSD.org>
Cc:        Daniel Engberg <diizzy@freebsd.org>, ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org, Alexey Dokuchaev <danfe@freebsd.org>, Guido Falsi <madpilot@freebsd.org>
Subject:   Re: git: 72ad532057c9 - main - archivers/zstd: Add CMake config files and adjust Makefile
Message-ID:  <dc155899-dbb0-428d-0582-d14b976874c2@tu-dortmund.de>
In-Reply-To: <b468b6e6-cbbe-86cf-9eed-8c945717c798@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> <ZC7KBj7lKm8VRtmn@FreeBSD.org> <b468b6e6-cbbe-86cf-9eed-8c945717c798@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Am 06.04.23 um 23:36 schrieb Dima Panov:
> Moin-moin!
> 
> On 06.04.2023 16:32, Alexey Dokuchaev wrote:
>> 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.
> 
> Nope.
> This commit unmasked the real problem of using autocrap tools to build
> zstd -- a broken static library.
> Issue was not raised before 'cause cmake configs was absent and qt6
> (and may be other consumers) hooks up only shared lib.
> 

Can you be more specific as to "autocrap tools" or how tools are held 
responsible for their (inadvertent) abuse by the clueless?

You don't blame the screwdriver if a noob tries to drive a nail with the 
blade you know...





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?dc155899-dbb0-428d-0582-d14b976874c2>