<pre style='margin:0'>
Perry E. Metzger (pmetzger) pushed a commit to branch master
in repository macports-ports.

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/22a6c44c2bfa21cddb25feb9b749a7b634877c5e">https://github.com/macports/macports-ports/commit/22a6c44c2bfa21cddb25feb9b749a7b634877c5e</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 22a6c44c2bf tiledb: cleanup and migrate from legacy dependency management mode
</span>22a6c44c2bf is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 22a6c44c2bfa21cddb25feb9b749a7b634877c5e
</span>Author: Theodore Tsirpanis <theodore.tsirpanis@tiledb.com>
AuthorDate: Fri May 31 23:47:52 2024 +0300

<span style='display:block; white-space:pre;color:#404040;'>    tiledb: cleanup and migrate from legacy dependency management mode
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    * Enable the modern mechanism of acquiring dependencies, and do not
</span><span style='display:block; white-space:pre;color:#404040;'>      download vcpkg.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>      The name "TILEDB_VCPKG" is misleading. Vcpkg is not actually required
</span><span style='display:block; white-space:pre;color:#404040;'>      but merely downloaded by default, by disabling it, TileDB will simply
</span><span style='display:block; white-space:pre;color:#404040;'>      call `find_package` to find the dependencies.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Disable the superbuild. TileDB will be configured as a single CMake
</span><span style='display:block; white-space:pre;color:#404040;'>      project.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Move libpng dependency to test-only.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Explicitly disable GCS and HDFS.
</span><span style='display:block; white-space:pre;color:#404040;'>      Other features are likewise disabled as well.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Add `file` dependency to get libmagic.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Remove blosc and curl dependencies.
</span><span style='display:block; white-space:pre;color:#404040;'>      Blosc is vendored and curl is used only with TILEDB_SERIALIZATION.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Add patch to manually find lz4.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Use zstd subport with CMake config.
</span><span style='display:block; white-space:pre;color:#404040;'>    * Bump revision.
</span>---
 databases/tiledb/Portfile                          | 31 ++++++++++++----------
 databases/tiledb/files/lz4_fix.diff                | 26 ++++++++++++++++++
 .../tiledb/files/patch-do-not-force-VCPKG.diff     | 22 ---------------
 3 files changed, 43 insertions(+), 36 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/databases/tiledb/Portfile b/databases/tiledb/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index 6d5c1513714..79890c3e55d 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/databases/tiledb/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/databases/tiledb/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -15,7 +15,7 @@ legacysupport.use_mp_libcxx yes
</span> 
 github.setup        TileDB-Inc TileDB 2.23.0
 name                tiledb
<span style='display:block; white-space:pre;background:#ffe0e0;'>-revision            1
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+revision            2
</span> categories          databases
 license             MIT
 maintainers         {@barracuda156 gmail.com:vital.had} openmaintainer
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -39,15 +39,13 @@ set libfmt_ver      10
</span> cmake.module_path-append \
                     ${prefix}/lib/libfmt${libfmt_ver}/cmake
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-depends_lib-append  port:blosc \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    port:bzip2 \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    port:curl \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+depends_lib-append  port:bzip2 \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    port:file \
</span>                     port:libfmt${libfmt_ver} \
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                    port:libpng \
</span>                     port:lz4 \
                     port:spdlog \
                     port:zlib \
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                    port:zstd
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    port:zstdConfig.cmake
</span> 
 depends_build-append \
                     port:gmake
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -76,10 +74,10 @@ if {[string match *gcc* ${configure.compiler}] && ${configure.build_arch} in [li
</span>                     patch-libatomic-32-bit.diff
 }
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# Upstream introduced an undesirable forcing of VCPKG.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# https://github.com/TileDB-Inc/TileDB/issues/4960
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# We do not want that, it does not work correctly and totally unneeded anyway.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-patchfiles-append   patch-do-not-force-VCPKG.diff
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Patch TileDB to find lz4 manually, because the lz4 package does not provide CMake configs.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Updating the lz4 port is blocked on lz4 releasing a new version, with some official
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# linkage-agnostic targets.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+patchfiles-append   lz4_fix.diff
</span> 
 compiler.cxx_standard       2020
 # https://github.com/TileDB-Inc/TileDB/issues/4566
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -93,14 +91,20 @@ configure.args-append \
</span>                     -DTILEDB_AZURE=OFF \
                     -DTILEDB_CPP_API=ON \
                     -DTILEDB_EXPERIMENTAL_FEATURES=OFF \
<span style='display:block; white-space:pre;background:#e0ffe0;'>+                    -DTILEDB_GCS=OFF \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    -DTILEDB_HDFS=OFF \
</span>                     -DTILEDB_SERIALIZATION=OFF \
<span style='display:block; white-space:pre;background:#e0ffe0;'>+                    -DTILEDB_SUPERBUILD=OFF \
</span>                     -DTILEDB_TESTS=OFF \
                     -DTILEDB_TOOLS=OFF \
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                    -DTILEDB_VCPKG=OFF \
</span>                     -DTILEDB_VERBOSE=ON \
                     -DTILEDB_WEBP=OFF \
                     -DTILEDB_WERROR=OFF
 
<span style='display:block; white-space:pre;background:#e0ffe0;'>+# Disable auto-downloading of vcpkg. All dependencies will be provided by MacPorts.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+configure.env-append \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    TILEDB_DISABLE_AUTO_VCPKG=1
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span> configure.cflags-append \
                     -DSPDLOG_FMT_EXTERNAL=1
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -119,8 +123,6 @@ if {${build_arch} eq "i386" && [string match *clang* ${configure.compiler}]} {
</span>                     -Wno-c++11-narrowing
 }
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-destroot.target     install-tiledb
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span> # FIXME: ppc32 tests build, but fail to pass. Possibly size of bool issue?
 # ppc64 tests are much better, though not clean. See above.
 variant tests description "Enable testing" {
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -136,7 +138,8 @@ variant tests description "Enable testing" {
</span>     # https://github.com/TileDB-Inc/TileDB/issues/4590
     # https://github.com/TileDB-Inc/TileDB/issues/4095
     depends_build-append \
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                    port:catch2
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    port:catch2 \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    port:libpng
</span>     # As of 2.19.1, bundled one is broken:
     # :info:build Undefined symbols:
     # :info:build   "__ZN5Catch29formatReconstructedExpressionERSoRKSsNS_9StringRefES2_", referenced from:
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/databases/tiledb/files/lz4_fix.diff b/databases/tiledb/files/lz4_fix.diff
</span>new file mode 100644
<span style='display:block; white-space:pre;color:#808080;'>index 00000000000..dbaf294aca5
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--- /dev/null
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/databases/tiledb/files/lz4_fix.diff
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -0,0 +1,26 @@
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+From e290ae2306b19e065b6580514eed7a53519d062b Mon Sep 17 00:00:00 2001
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+From: Theodore Tsirpanis <theodore.tsirpanis@tiledb.com>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Date: Fri, 31 May 2024 23:06:58 +0300
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Subject: [PATCH] Skip finding the lz4 package.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ 
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Downstream packagers like Conda and MacPorts need to find lz4 manually, because the packages for lz4 in these environments do not provide CMake configs.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+---
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ cmake/Modules/FindLZ4_EP.cmake | 5 -----
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ 1 file changed, 5 deletions(-)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ 
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+diff --git cmake/Modules/FindLZ4_EP.cmake cmake/Modules/FindLZ4_EP.cmake
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+index b044992998e..b2263217519 100644
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+--- cmake/Modules/FindLZ4_EP.cmake
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++++ cmake/Modules/FindLZ4_EP.cmake
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+@@ -34,11 +34,6 @@
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # Include some common helper functions.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ include(TileDBCommon)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ 
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-if(TILEDB_VCPKG)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-  find_package(lz4 REQUIRED)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-  return()
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-endif()
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # First check for a static version in the EP prefix.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ find_library(LZ4_LIBRARIES
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+   NAMES
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/databases/tiledb/files/patch-do-not-force-VCPKG.diff b/databases/tiledb/files/patch-do-not-force-VCPKG.diff
</span>deleted file mode 100644
<span style='display:block; white-space:pre;color:#808080;'>index 24d235d8110..00000000000
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/databases/tiledb/files/patch-do-not-force-VCPKG.diff
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ /dev/null
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1,22 +0,0 @@
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>---- cmake/Options/BuildOptions.cmake       2024-05-03 00:58:42.000000000 +0800
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-+++ cmake/Options/BuildOptions.cmake       2024-05-10 03:13:43.000000000 +0800
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-@@ -8,7 +8,7 @@
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- include(CMakeDependentOption)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- 
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- option(TILEDB_SUPERBUILD "If true, perform a superbuild (builds all missing dependencies)." ON)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--option(TILEDB_VCPKG "If true, use vcpkg to download and build dependencies." ON)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-+option(TILEDB_VCPKG "If true, use vcpkg to download and build dependencies." OFF)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- cmake_dependent_option(TILEDB_FORCE_ALL_DEPS "If true, force superbuild to download and build all dependencies, even those installed on the system." OFF "NOT TILEDB_VCPKG" OFF)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- option(TILEDB_SANITIZER "Sets the sanitizers to use. Only address is currently supported." "")
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- option(TILEDB_VCPKG_BASE_TRIPLET "Sets the base vcpkg triplet when building with sanitizers." "")
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-@@ -48,10 +48,6 @@
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-   endif()
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- endif()
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- 
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--if (NOT TILEDB_VCPKG)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--  message(FATAL_ERROR "Disabling TILEDB_VCPKG is not supported.")
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--endif()
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- # enable assertions by default for debug builds
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- if (CMAKE_BUILD_TYPE STREQUAL "Debug")
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-   set(TILEDB_ASSERTIONS TRUE)
</span></pre><pre style='margin:0'>

</pre>