From nobody Sat Jan 8 20:25:49 2022 X-Original-To: dev-commits-ports-branches@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 3AE5A194666C; Sat, 8 Jan 2022 20:25:50 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (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 (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4JWWn95Y5Tz3tVy; Sat, 8 Jan 2022 20:25:49 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (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 did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 9F2D325275; Sat, 8 Jan 2022 20:25:49 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 208KPnBa002316; Sat, 8 Jan 2022 20:25:49 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 208KPnfH002315; Sat, 8 Jan 2022 20:25:49 GMT (envelope-from git) Date: Sat, 8 Jan 2022 20:25:49 GMT Message-Id: <202201082025.208KPnfH002315@gitrepo.freebsd.org> To: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-branches@FreeBSD.org From: Jan Beich Subject: git: 700d924a605c - 2022Q1 - graphics/libjxl: switch to system highway List-Id: Commits to the quarterly branches of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-branches List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-branches@freebsd.org X-BeenThere: dev-commits-ports-branches@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: jbeich X-Git-Repository: ports X-Git-Refname: refs/heads/2022Q1 X-Git-Reftype: branch X-Git-Commit: 700d924a605cc558e3ee86033ac8b76d8d12383c Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1641673549; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=wKEnFHvAo3P6HXnWqsOG7zCm3eeG89EMAQotM6egAlg=; b=Mv+TMpv2RiwlD67fIJNW41MrR2qXpm4MTEriF04pJnc1dxZqshuAfRIYyx5yygHQ9vP5a9 nAxqOxsQPdz7hAp1yx4+YrGy3k+hG5mKGYUNVHdrWlK+Ni7ciunsdv/9rpM8TZvvX2XPYt ZV5m4l88YQUEGxvwfiLUjblw2ks+KfXMTocbGZWhOx+3ocP+wfublPq9fwgGtlsdAldcwE 2xrH4Y+zr+TtcRVr461MCd7TrHmyTb17BRC5kBH9RA3gr/c2GwNJD0/3E3hkW25OrJcx1A dKYPMbFADoQ0PXT0QV5ncR0XJ6x90W5FISSy2rwrDIK6RgFiQ2TPEftnjw96uQ== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1641673549; a=rsa-sha256; cv=none; b=PO0JsvOhLeT2Ccz8mShIOwcjh6nx11Ec7kdoYuCyi5ujRgM+aq9F9V3wlUWC+uGC1hDZpS IzuKRRp33fcNZtPF+0JQQqJ4KX8SPUVp0jq16PdY5hshR4S8Z8887S7STt+8VYSyYDOM2v JFq0Ds3s6baPC/p4L+JEjpe21H8bqGbiyl3ALTmb3nHH+PwYbpegEaN4HUbCB1O1DdI5M6 gvdT10JYqNDX+05YSfq0hGGzW8cYn9XWcH6gIrxxxjlEUbrC4uCuhshpjUle33i7OG33WY 4c0oqoPR9VF/IpGVBwoCET+/BCWdI0Cu5Icx/0AfMlYQ947e71HRQr7zWma08w== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch 2022Q1 has been updated by jbeich: URL: https://cgit.FreeBSD.org/ports/commit/?id=700d924a605cc558e3ee86033ac8b76d8d12383c commit 700d924a605cc558e3ee86033ac8b76d8d12383c Author: Jan Beich AuthorDate: 2022-01-08 20:17:09 +0000 Commit: Jan Beich CommitDate: 2022-01-08 20:25:19 +0000 graphics/libjxl: switch to system highway PR: 261003 (cherry picked from commit fdd0ceeceefafad3ce9bda41029ca4973e5fb71d) --- graphics/libjxl/Makefile | 4 +- graphics/libjxl/distinfo | 2 - graphics/libjxl/files/patch-private-libhwy | 22 +++++++++++ graphics/libjxl/files/patch-private-libs | 45 ---------------------- ...patch-third__party_highway_hwy_nanobenchmark.cc | 40 ------------------- 5 files changed, 24 insertions(+), 89 deletions(-) diff --git a/graphics/libjxl/Makefile b/graphics/libjxl/Makefile index 2bc9513c9222..1b3bd1c71576 100644 --- a/graphics/libjxl/Makefile +++ b/graphics/libjxl/Makefile @@ -1,7 +1,7 @@ PORTNAME= libjxl DISTVERSIONPREFIX= v DISTVERSION= 0.6.1 -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= graphics PATCH_SITES= https://github.com/${GH_ACCOUNT}/${GH_PROJECT}/commit/ @@ -16,6 +16,7 @@ LICENSE_COMB= multi LICENSE_FILE_BSD3CLAUSE=${WRKSRC}/LICENSE LICENSE_FILE_ZLIB= ${WRKSRC}/third_party/lodepng/LICENSE +BUILD_DEPENDS= highway>0:devel/highway LIB_DEPENDS= libbrotlienc.so:archivers/brotli TEST_DEPENDS= googletest>0:devel/googletest @@ -24,7 +25,6 @@ CPE_VENDOR= ${PORTNAME}_project USE_GITHUB= yes USE_LDCONFIG= yes GH_TUPLE= barmco:skcms:6437475:skcms/third_party/skcms \ - google:highway:0.12.2-4-ge239774:highway/third_party/highway \ lvandeve:lodepng:48e5364:lodepng/third_party/lodepng \ webmproject:sjpeg:868ab55:sjpeg/third_party/sjpeg \ ${NULL} diff --git a/graphics/libjxl/distinfo b/graphics/libjxl/distinfo index e8589da7cf24..dc9bc3ff7b74 100644 --- a/graphics/libjxl/distinfo +++ b/graphics/libjxl/distinfo @@ -3,8 +3,6 @@ SHA256 (libjxl-libjxl-v0.6.1_GH0.tar.gz) = ccbd5a729d730152303be399f033b905e6083 SIZE (libjxl-libjxl-v0.6.1_GH0.tar.gz) = 16953185 SHA256 (barmco-skcms-6437475_GH0.tar.gz) = 6818ea7aa053a964d1795d6d1d3c5e2aad5d30dff9817c8e604e324ef643ec01 SIZE (barmco-skcms-6437475_GH0.tar.gz) = 9200034 -SHA256 (google-highway-0.12.2-4-ge239774_GH0.tar.gz) = 27ae8a40f82874cb30bd62ef8f5567cc27838ee258ed0af47745685a15b23a8e -SIZE (google-highway-0.12.2-4-ge239774_GH0.tar.gz) = 1488138 SHA256 (lvandeve-lodepng-48e5364_GH0.tar.gz) = 41e0bcc686e31ee2d51a6c87c0089045f4805e4053d39026ad02e8aa90c782c1 SIZE (lvandeve-lodepng-48e5364_GH0.tar.gz) = 239988 SHA256 (webmproject-sjpeg-868ab55_GH0.tar.gz) = ffe757b376fed2fd94473f8625c25f82a3a23aa317fb1bbf1ac8159b6e0a7726 diff --git a/graphics/libjxl/files/patch-private-libhwy b/graphics/libjxl/files/patch-private-libhwy new file mode 100644 index 000000000000..c9b5add67ac0 --- /dev/null +++ b/graphics/libjxl/files/patch-private-libhwy @@ -0,0 +1,22 @@ +Avoid depending on static libhwy for shared libjxl consumers + +--- lib/jxl.cmake.orig 2021-10-04 18:02:44 UTC ++++ lib/jxl.cmake +@@ -557,7 +557,7 @@ endif() # TARGET_SUPPORTS_SHARED_LIBS AND NOT JPEGXL_ + + # Add a pkg-config file for libjxl. + set(JPEGXL_LIBRARY_REQUIRES +- "libhwy libbrotlicommon libbrotlienc libbrotlidec") ++ "libbrotlicommon libbrotlienc libbrotlidec") + if(NOT JPEGXL_ENABLE_SKCMS) + set(JPEGXL_LIBRARY_REQUIRES "${JPEGXL_LIBRARY_REQUIRES} lcms2") + endif() +--- lib/jxl/libjxl.pc.in.orig 2021-10-04 18:02:44 UTC ++++ lib/jxl/libjxl.pc.in +@@ -8,5 +8,5 @@ Description: Loads and saves JPEG XL files + Version: @JPEGXL_LIBRARY_VERSION@ + Requires.private: @JPEGXL_LIBRARY_REQUIRES@ + Libs: -L${libdir} -ljxl +-Libs.private: -lm ++Libs.private: -lhwy -lm + Cflags: -I${includedir} diff --git a/graphics/libjxl/files/patch-private-libs b/graphics/libjxl/files/patch-private-libs deleted file mode 100644 index b1b13b911f85..000000000000 --- a/graphics/libjxl/files/patch-private-libs +++ /dev/null @@ -1,45 +0,0 @@ -highway is untagged and have no other consumers. Keep it private to -libjxl package for now. - ---- lib/jxl.cmake.orig 2021-10-04 18:02:44 UTC -+++ lib/jxl.cmake -@@ -557,7 +557,7 @@ endif() # TARGET_SUPPORTS_SHARED_LIBS AND NOT JPEGXL_ - - # Add a pkg-config file for libjxl. - set(JPEGXL_LIBRARY_REQUIRES -- "libhwy libbrotlicommon libbrotlienc libbrotlidec") -+ "libbrotlicommon libbrotlienc libbrotlidec") - if(NOT JPEGXL_ENABLE_SKCMS) - set(JPEGXL_LIBRARY_REQUIRES "${JPEGXL_LIBRARY_REQUIRES} lcms2") - endif() ---- lib/jxl/libjxl.pc.in.orig 2021-10-04 18:02:44 UTC -+++ lib/jxl/libjxl.pc.in -@@ -8,5 +8,5 @@ Description: Loads and saves JPEG XL files - Version: @JPEGXL_LIBRARY_VERSION@ - Requires.private: @JPEGXL_LIBRARY_REQUIRES@ - Libs: -L${libdir} -ljxl --Libs.private: -lm -+Libs.private: -lhwy -lm - Cflags: -I${includedir} ---- third_party/CMakeLists.txt.orig 2020-11-14 00:52:03 UTC -+++ third_party/CMakeLists.txt -@@ -83,7 +83,7 @@ endif() # BUILD_TESTING - # Highway - if (EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/highway/CMakeLists.txt" AND - NOT JPEGXL_FORCE_SYSTEM_HWY) -- add_subdirectory(highway) -+ add_subdirectory(highway EXCLUDE_FROM_ALL) - configure_file("${CMAKE_CURRENT_SOURCE_DIR}/highway/LICENSE" - ${PROJECT_BINARY_DIR}/LICENSE.highway COPYONLY) - else() ---- third_party/highway/CMakeLists.txt.orig 2020-12-02 19:39:56 UTC -+++ third_party/highway/CMakeLists.txt -@@ -230,7 +230,7 @@ set_target_properties(hwy_benchmark - - include(CTest) - --if(BUILD_TESTING) -+if(HWY_BUILD_TESTING) - enable_testing() - include(GoogleTest) - diff --git a/graphics/libjxl/files/patch-third__party_highway_hwy_nanobenchmark.cc b/graphics/libjxl/files/patch-third__party_highway_hwy_nanobenchmark.cc deleted file mode 100644 index e6bcd6711d0f..000000000000 --- a/graphics/libjxl/files/patch-third__party_highway_hwy_nanobenchmark.cc +++ /dev/null @@ -1,40 +0,0 @@ -https://github.com/google/highway/commit/3769cb2c464c - ---- third_party/highway/hwy/nanobenchmark.cc.orig 2021-06-09 08:56:32 UTC -+++ third_party/highway/hwy/nanobenchmark.cc -@@ -46,7 +46,7 @@ - #endif - - #include "hwy/base.h" --#if HWY_ARCH_PPC -+#if HWY_ARCH_PPC && defined(__GLIBC__) - #include // NOLINT __ppc_get_timebase_freq - #elif HWY_ARCH_X86 - -@@ -119,7 +119,7 @@ using Ticks = uint64_t; - // divide by InvariantTicksPerSecond. - inline Ticks Start() { - Ticks t; --#if HWY_ARCH_PPC -+#if HWY_ARCH_PPC && defined(__GLIBC__) - asm volatile("mfspr %0, %1" : "=r"(t) : "i"(268)); - #elif HWY_ARCH_X86 && HWY_COMPILER_MSVC - _ReadWriteBarrier(); -@@ -161,7 +161,7 @@ inline Ticks Start() { - - inline Ticks Stop() { - uint64_t t; --#if HWY_ARCH_PPC -+#if HWY_ARCH_PPC && defined(__GLIBC__) - asm volatile("mfspr %0, %1" : "=r"(t) : "i"(268)); - #elif HWY_ARCH_X86 && HWY_COMPILER_MSVC - _ReadWriteBarrier(); -@@ -399,7 +399,7 @@ double NominalClockRate() { - } // namespace - - double InvariantTicksPerSecond() { --#if HWY_ARCH_PPC -+#if HWY_ARCH_PPC && defined(__GLIBC__) - return __ppc_get_timebase_freq(); - #elif HWY_ARCH_X86 - // We assume the TSC is invariant; it is on all recent Intel/AMD CPUs.