From nobody Sun Mar 31 21:06:31 2024 X-Original-To: dev-commits-ports-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4V76By2TxNz5GQwc for ; Sun, 31 Mar 2024 21:06:34 +0000 (UTC) (envelope-from daniel.engberg.lists@pyret.net) Received: from smtp-bc0d.mail.infomaniak.ch (smtp-bc0d.mail.infomaniak.ch [IPv6:2001:1600:7:10::bc0d]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "relay.mail.infomaniak.ch", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4V76Bx71gvz4k22 for ; Sun, 31 Mar 2024 21:06:33 +0000 (UTC) (envelope-from daniel.engberg.lists@pyret.net) Authentication-Results: mx1.freebsd.org; none Received: from smtp-3-0000.mail.infomaniak.ch (smtp-3-0000.mail.infomaniak.ch [10.4.36.107]) by smtp-4-3000.mail.infomaniak.ch (Postfix) with ESMTPS id 4V76Bv4xP6zTbR; Sun, 31 Mar 2024 23:06:31 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=pyret.net; s=20231006; t=1711919191; bh=IT+24J+qKdK5G6WT1jFZz+N77TehEPuPPah4FQT07sw=; h=Date:Subject:From:Reply-To:To:Cc:References:In-Reply-To:From; b=RS9cpqWXex+vyX4wBhvXVZXWLwSoSwYnUhjMxMM02f6CM3FYZX5GrvPK3iIPJUtdk ToUR801N/YtI59kpstC5HsfKKfH3dnBOZWg7TxVhSm2npQn7YDEZW63svBldouB0Go 0HfUGohHvcNi0Z2gHi88W8yJgiPPUZ0Pnxkl5Qw+KIghlutE9EVQk7yZ1LPg9ucXcW Uj6zIJYRvA06itDYXI+iRNifJkuzJJELSAz8Q29bKxDG6Vf/o2kp83kYK0gayLe0W+ ov22Cz4sWY7ookJRvE7Ucf+uL6IqPrDSKeO6mZfAZ6YRv7rFVjNgc2B9bBaHk+vxER ku+4c2/UTcQFw== Received: from unknown by smtp-3-0000.mail.infomaniak.ch (Postfix) with ESMTPA id 4V76Bv27VPz3X; Sun, 31 Mar 2024 23:06:31 +0200 (CEST) Message-ID: <826d811c6ad3e17b8ae7d743403fb228@mail.infomaniak.com> Date: Sun, 31 Mar 2024 23:06:31 +0200 Subject: Re: git: b594c775ed2d - main - archivers/zstd: Update to 1.5.6 From: Daniel Engberg Reply-To: Daniel Engberg To: Po-Chuan Hsieh Cc: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-WS-User-Origin: eyJpdiI6ImFTQ3VRQ3cxZW0rSHlISXc5U3RISmc9PSIsInZhbHVlIjoia0pLUFNnSHhzYWVaNXVNU3BvWnBpQT09IiwibWFjIjoiZDU3NjE4YmE3OTM2NDU0YzFmNzU3MTg1Y2I3MTdjMjVmMGYyYmE0MGQ0MWI4MDc4ZWU2NDMwZDdjNjY0MmYyMyIsInRhZyI6IiJ9 X-WS-User-Mbox: eyJpdiI6IklJVWg4ODdjbU1MaXNSSFhiUzhORXc9PSIsInZhbHVlIjoiY2s2c25ycW5QU0Rwc0tDMWN6dFdRZz09IiwibWFjIjoiZjY4NTk3N2ZkYzhmNTMwZjEwZDQzMzkxM2Q5NmM2MmQ5MjEzYTI2YzFhNjkwZjIyZTI5ZDEwMTFlZTRmYmM5YSIsInRhZyI6IiJ9 X-WS-Location: eJxzKUpMKykGAAfpAmU- X-Mailer: Infomaniak Workspace (1.3.662) References: <202403311856.42VIuAtn044147@gitrepo.freebsd.org> In-Reply-To: <202403311856.42VIuAtn044147@gitrepo.freebsd.org> X-Infomaniak-Routing: alpha X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:29222, ipnet:2001:1600::/32, country:CH] X-Rspamd-Queue-Id: 4V76Bx71gvz4k22 On 2024-03-31T20:56:10.000+02:00, Po-Chuan Hsieh wrot= e: > The branch main has been updated by sunpoet: >=20 > URL: https://cgit.FreeBSD.org/ports/commit/?id=3Db594c775ed2dbae25e509d19= 8c462940f0dc9d5c >=20 > commit b594c775ed2dbae25e509d198c462940f0dc9d5c > Author: Po-Chuan Hsieh > AuthorDate: 2024-03-31 18:50:47 +0000 > Commit: Po-Chuan Hsieh > CommitDate: 2024-03-31 18:50:47 +0000 >=20 > archivers/zstd: Update to 1.5.6 > =20 > Changes: https://github.com/facebook/zstd/releases > --- > archivers/zstd/Makefile | 15 ++-------- > archivers/zstd/distinfo | 6 ++-- > archivers/zstd/files/zstdConfig.cmake | 34 ++++++++++++++++= ++++++ > archivers/zstd/files/zstdConfigVersion.cmake.in | 5 ---- > .../zstd/files/zstdTargets-BUILD_TYPE.cmake.in | 2 +- > archivers/zstd/files/zstdTargets.cmake | 16 +++++++--- > archivers/zstd/pkg-plist | 1 - > 7 files changed, 53 insertions(+), 26 deletions(-) >=20 > diff --git a/archivers/zstd/Makefile b/archivers/zstd/Makefile > index 386af72d0ca8..00fefd673750 100644 > --- a/archivers/zstd/Makefile > +++ b/archivers/zstd/Makefile > @@ -1,6 +1,5 @@ > PORTNAME=3D=09zstd > -PORTVERSION=3D=091.5.5 > -PORTREVISION=3D=091 > +PORTVERSION=3D=091.5.6 > CATEGORIES=3D=09archivers > MASTER_SITES=3D=09https://github.com/facebook/zstd/releases/download/v${= PORTVERSION}/ > =20 > @@ -34,23 +33,15 @@ PLIST_SUB=3D=09BUILD_TYPE=3D${BUILD_TYPE:tl} PORTVERS= ION=3D${PORTVERSION} > SUB_FILES=3D=09zstdConfigVersion.cmake zstdTargets-BUILD_TYPE.cmake > SUB_LIST=3D=09BUILD_TYPE_CAPS=3D${BUILD_TYPE:tu} BUILD_TYPE_MIXED=3D${BU= ILD_TYPE} PORTVERSION=3D${PORTVERSION} > =20 > -.if defined(WITH_DEBUG) > -BUILD_TYPE=3D Debug > -.else > -BUILD_TYPE=3D Release > -.endif > +BUILD_TYPE=3D=09${defined(WITH_DEBUG) :?Debug :Release} > =20 > OPTIONS_DEFINE=3D=09OPTIMIZED_CFLAGS > =20 > OPTIMIZED_CFLAGS_VARS=3D=09CFLAGS+=3D-O3 > =20 > -post-build: > -=09cd ${WRKSRC}/programs && ${DO_MAKE_BUILD} zstd-frugal > - > post-install: > =09${MKDIR} ${STAGEDIR}${PREFIX}/lib/cmake/zstd > -=09${INSTALL_PROGRAM} ${WRKSRC}/programs/zstd-frugal ${STAGEDIR}${PREFIX= }/bin/zstd-frugal > -=09${INSTALL_DATA} ${WRKSRC}/build/cmake/zstdConfig.cmake ${STAGEDIR}${P= REFIX}/lib/cmake/zstd/zstdConfig.cmake > +=09${INSTALL_DATA} ${FILESDIR}/zstdConfig.cmake ${STAGEDIR}${PREFIX}/lib= /cmake/zstd/zstdConfig.cmake > =09${INSTALL_DATA} ${WRKDIR}/zstdConfigVersion.cmake ${STAGEDIR}${PREFIX= }/lib/cmake/zstd/zstdConfigVersion.cmake > =09${INSTALL_DATA} ${WRKDIR}/zstdTargets-BUILD_TYPE.cmake ${STAGEDIR}${P= REFIX}/lib/cmake/zstd/zstdTargets-${BUILD_TYPE:tl}.cmake > =09${INSTALL_DATA} ${FILESDIR}/zstdTargets.cmake ${STAGEDIR}${PREFIX}/li= b/cmake/zstd/zstdTargets.cmake > diff --git a/archivers/zstd/distinfo b/archivers/zstd/distinfo > index d89bac34fc53..46a55033cd34 100644 > --- a/archivers/zstd/distinfo > +++ b/archivers/zstd/distinfo > @@ -1,3 +1,3 @@ > -TIMESTAMP =3D 1684104304 > -SHA256 (zstd-1.5.5.tar.zst) =3D ce264bca60eb2f0e99e4508cffd0d4d19dd362e8= 4244d7fc941e79fa69ccf673 > -SIZE (zstd-1.5.5.tar.zst) =3D 1817544 > +TIMESTAMP =3D 1711540146 > +SHA256 (zstd-1.5.6.tar.zst) =3D 4aa8dd1c1115c0fd6b6b66c35c7f6ce7bd58cc1d= fd3e4f175b45b39e84b14352 > +SIZE (zstd-1.5.6.tar.zst) =3D 1845516 > diff --git a/archivers/zstd/files/zstdConfig.cmake b/archivers/zstd/files= /zstdConfig.cmake > new file mode 100644 > index 000000000000..7cc9666dbaed > --- /dev/null > +++ b/archivers/zstd/files/zstdConfig.cmake > @@ -0,0 +1,34 @@ > + > +####### Expanded from @PACKAGE_INIT@ by configure_package_config_file() = ####### > +####### Any changes to this file will be overwritten by the next CMake r= un #### > +####### The input file was zstdConfig.cmake.in = ######## > + > +get_filename_component(PACKAGE_PREFIX_DIR "${CMAKE_CURRENT_LIST_DIR}/../= ../../" ABSOLUTE) > + > +macro(set_and_check _var _file) > + set(${_var} "${_file}") > + if(NOT EXISTS "${_file}") > + message(FATAL_ERROR "File or directory ${_file} referenced by variab= le ${_var} does not exist !") > + endif() > +endmacro() > + > +macro(check_required_components _NAME) > + foreach(comp ${${_NAME}_FIND_COMPONENTS}) > + if(NOT ${_NAME}_${comp}_FOUND) > + if(${_NAME}_FIND_REQUIRED_${comp}) > + set(${_NAME}_FOUND FALSE) > + endif() > + endif() > + endforeach() > +endmacro() > + > +########################################################################= ############ > + > +include(CMakeFindDependencyMacro) > +if(ON AND "1") > + find_dependency(Threads) > +endif() > + > +include("${CMAKE_CURRENT_LIST_DIR}/zstdTargets.cmake") > + > +check_required_components("zstd") > diff --git a/archivers/zstd/files/zstdConfigVersion.cmake.in b/archivers/= zstd/files/zstdConfigVersion.cmake.in > index 74c5c7103a97..4528f77accac 100644 > --- a/archivers/zstd/files/zstdConfigVersion.cmake.in > +++ b/archivers/zstd/files/zstdConfigVersion.cmake.in > @@ -52,11 +52,6 @@ else() > endif() > =20 > =20 > -# if the installed project requested no architecture check, don't perfor= m the check > -if("FALSE") > - return() > -endif() > - > # if the installed or the using project don't have CMAKE_SIZEOF_VOID_P s= et, ignore it: > if("${CMAKE_SIZEOF_VOID_P}" STREQUAL "" OR "8" STREQUAL "") > return() > diff --git a/archivers/zstd/files/zstdTargets-BUILD_TYPE.cmake.in b/archi= vers/zstd/files/zstdTargets-BUILD_TYPE.cmake.in > index 72d7b1ba1e73..8dd7b7b22952 100644 > --- a/archivers/zstd/files/zstdTargets-BUILD_TYPE.cmake.in > +++ b/archivers/zstd/files/zstdTargets-BUILD_TYPE.cmake.in > @@ -18,7 +18,7 @@ list(APPEND _cmake_import_check_files_for_zstd::libzstd= _shared "${_IMPORT_PREFIX > # Import target "zstd::libzstd_static" for configuration "%%BUILD_TYPE_M= IXED%%" > set_property(TARGET zstd::libzstd_static APPEND PROPERTY IMPORTED_CONFIG= URATIONS %%BUILD_TYPE_CAPS%%) > set_target_properties(zstd::libzstd_static PROPERTIES > - IMPORTED_LINK_INTERFACE_LANGUAGES_%%BUILD_TYPE_CAPS%% "C" > + IMPORTED_LINK_INTERFACE_LANGUAGES_%%BUILD_TYPE_CAPS%% "ASM;C" > IMPORTED_LOCATION_%%BUILD_TYPE_CAPS%% "${_IMPORT_PREFIX}/lib/libzstd.a= " > ) > =20 > diff --git a/archivers/zstd/files/zstdTargets.cmake b/archivers/zstd/file= s/zstdTargets.cmake > index 60da8aa603e0..537317b1e571 100644 > --- a/archivers/zstd/files/zstdTargets.cmake > +++ b/archivers/zstd/files/zstdTargets.cmake > @@ -7,7 +7,7 @@ if(CMAKE_VERSION VERSION_LESS "2.8.3") > message(FATAL_ERROR "CMake >=3D 2.8.3 required") > endif() > cmake_policy(PUSH) > -cmake_policy(VERSION 2.8.3...3.23) > +cmake_policy(VERSION 2.8.3...3.26) > #---------------------------------------------------------------- > # Generated CMake target import file. > #---------------------------------------------------------------- > @@ -19,7 +19,7 @@ set(CMAKE_IMPORT_FILE_VERSION 1) > set(_cmake_targets_defined "") > set(_cmake_targets_not_defined "") > set(_cmake_expected_targets "") > -foreach(_cmake_expected_target IN ITEMS zstd::libzstd_shared zstd::libzs= td_static) > +foreach(_cmake_expected_target IN ITEMS zstd::libzstd_shared zstd::libzs= td_static zstd::libzstd) > list(APPEND _cmake_expected_targets "${_cmake_expected_target}") > if(TARGET "${_cmake_expected_target}") > list(APPEND _cmake_targets_defined "${_cmake_expected_target}") > @@ -71,8 +71,16 @@ set_target_properties(zstd::libzstd_static PROPERTIES > INTERFACE_LINK_LIBRARIES "-pthread" > ) > =20 > -if(CMAKE_VERSION VERSION_LESS 2.8.12) > - message(FATAL_ERROR "This file relies on consumers using CMake 2.8.12 = or greater.") > +# Create imported target zstd::libzstd > +add_library(zstd::libzstd INTERFACE IMPORTED) > + > +set_target_properties(zstd::libzstd PROPERTIES > + INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include" > + INTERFACE_LINK_LIBRARIES "zstd::libzstd_static" > +) > + > +if(CMAKE_VERSION VERSION_LESS 3.0.0) > + message(FATAL_ERROR "This file relies on consumers using CMake 3.0.0 o= r greater.") > endif() > =20 > # Load information for each installed configuration. > diff --git a/archivers/zstd/pkg-plist b/archivers/zstd/pkg-plist > index 2f26988884ce..5183e20b2d6c 100644 > --- a/archivers/zstd/pkg-plist > +++ b/archivers/zstd/pkg-plist > @@ -1,6 +1,5 @@ > bin/unzstd > bin/zstd > -bin/zstd-frugal > bin/zstdcat > bin/zstdgrep > bin/zstdless Hi, Is there a reason why we can't adopt the CMake build instead which removes = local patching, cleans up the Makefile and also builds quite a bit faster? = It's already adoped by several other distros such as Arch Linux, Debian, Ho= mebrew and OpenMandriva. Is there also any reason behind why zstd-frugal wa= s removed? PR 278008 is also related Best regards, Daniel