<pre style='margin:0'>
Herby Gillot (herbygillot) pushed a commit to branch master
in repository macports-ports.
</pre>
<p><a href="https://github.com/macports/macports-ports/commit/8aa2324a664586bb1a0cf38634daad29572fcca7">https://github.com/macports/macports-ports/commit/8aa2324a664586bb1a0cf38634daad29572fcca7</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 8aa2324a664 duckdb: unbreak 0.10.0 for < 10.7
</span>8aa2324a664 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 8aa2324a664586bb1a0cf38634daad29572fcca7
</span>Author: Sergey Fedorov <vital.had@gmail.com>
AuthorDate: Tue Feb 20 07:08:57 2024 +0800
<span style='display:block; white-space:pre;color:#404040;'> duckdb: unbreak 0.10.0 for < 10.7
</span>---
databases/duckdb/Portfile | 3 +-
.../duckdb/files/fix-libproc-for-older-macOS.patch | 48 ++++++++++++++++++++++
2 files changed, 50 insertions(+), 1 deletion(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/databases/duckdb/Portfile b/databases/duckdb/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index 95e751e879e..eb826ccf4cb 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/databases/duckdb/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/databases/duckdb/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -32,7 +32,8 @@ checksums rmd160 692c7857b78c53486dc06d1327c5a00b9ea44dbb \
</span> size 79037968
patchfiles-append no-ccache.patch \
<span style='display:block; white-space:pre;background:#ffe0e0;'>- no-default-tarball-version.patch
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ no-default-tarball-version.patch \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ fix-libproc-for-older-macOS.patch
</span>
configure.args-append \
-DBUILD_PARQUET_EXTENSION=TRUE \
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/databases/duckdb/files/fix-libproc-for-older-macOS.patch b/databases/duckdb/files/fix-libproc-for-older-macOS.patch
</span>new file mode 100644
<span style='display:block; white-space:pre;color:#808080;'>index 00000000000..0c25186e549
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--- /dev/null
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/databases/duckdb/files/fix-libproc-for-older-macOS.patch
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -0,0 +1,48 @@
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+From 16acae07aebc8259dd21551f19d355f15c61dd99 Mon Sep 17 00:00:00 2001
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+From: Sergey Fedorov <vital.had@gmail.com>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Date: Tue, 20 Feb 2024 06:50:32 +0800
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Subject: [PATCH] local_file_system.cpp: minor fix for macOS libproc code
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+MIME-Version: 1.0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Content-Type: text/plain; charset=UTF-8
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Content-Transfer-Encoding: 8bit
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Recent commit has broken duckdb build on macOS < 10.7: https://github.com/duckdb/duckdb/commit/23112e9aa344905c2d2a0d1cb696c5f73c329534
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Add a fallback to fix it.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+See also get_ppid_of.c in gettext’s libtextstyle.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+---
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ src/common/local_file_system.cpp | 15 +++++++++++++++
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ 1 file changed, 15 insertions(+)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+diff --git src/common/local_file_system.cpp src/common/local_file_system.cpp
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+index bbca902a28..14903303e6 100644
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+--- src/common/local_file_system.cpp
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++++ src/common/local_file_system.cpp
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+@@ -188,6 +188,8 @@ static string AdditionalProcessInfo(FileSystem &fs, pid_t pid) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ string process_name, process_owner;
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++// macOS >= 10.7 has PROC_PIDT_SHORTBSDINFO
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++#if defined PROC_PIDT_SHORTBSDINFO
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ // try to find out more about the process holding the lock
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ struct proc_bsdshortinfo proc;
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if (proc_pidinfo(pid, PROC_PIDT_SHORTBSDINFO, 0, &proc, PROC_PIDT_SHORTBSDINFO_SIZE) ==
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+@@ -199,6 +201,19 @@ static string AdditionalProcessInfo(FileSystem &fs, pid_t pid) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ process_owner = pw->pw_name;
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++#else
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++// Fallback code for older versions
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ // try to find out more about the process holding the lock
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ struct proc_bsdinfo proc;
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ if (proc_pidinfo(pid, PROC_PIDTBSDINFO, 0, &proc, 128) == 128) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ process_name = proc.pbi_comm; // only a short version however, let's take it in case proc_pidpath() below fails
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ // try to get actual name of conflicting process owner
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ auto pw = getpwuid(proc.pbi_uid);
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ if (pw) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ process_owner = pw->pw_name;
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++#endif
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ // try to get a better process name (full path)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ char full_exec_path[PROC_PIDPATHINFO_MAXSIZE];
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if (proc_pidpath(pid, full_exec_path, PROC_PIDPATHINFO_MAXSIZE) > 0) {
</span></pre><pre style='margin:0'>
</pre>