From: | Aleksander Alekseev <aleksander(at)timescale(dot)com> |
---|---|
To: | Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Cc: | Stan Hu <stanhu(at)gmail(dot)com> |
Subject: | Re: PostgreSQL does not compile on macOS SDK 15.0 |
Date: | 2024-06-25 13:49:32 |
Message-ID: | CAJ7c6TOxW4AzDz3WW5dT5j-L_nwaCx63Fz5gCJu24ixpwPfPmw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
> IMO the right way to test PostgreSQL against the recent beta version
> of MacOS SDK would be replacing (via a symlink perhaps) the SDK
> provided by the "Command Line Tools for Xcode" package
> (/Library/Developer/CommandLineTools/SDKs/). Or alternatively finding
> the official way of installing the beta version of this package.
As it turned out there is Command_Line_Tools_for_Xcode_16_beta.dmg
package available. It can be downloaded from
https://developer.apple.com/ after logging it. I installed it and also
did:
```
cd /Library/Developer/CommandLineTools/SDKs
sudo mkdir __ignore__
sudo mv MacOSX14.* __ignore__
```
... to make sure Postgres will not find the older version of SDK (it
did until I made this step).
Now I get the following output from `meson --setup ...`:
```
The Meson build system
Version: 0.61.2
Source dir: /Users/eax/projects/c/postgresql
Build dir: /Users/eax/projects/c/postgresql/build
Build type: native build
Project name: postgresql
Project version: 17beta2
C compiler for the host machine: cc (clang 16.0.0 "Apple clang version
16.0.0 (clang-1600.0.20.10)")
C linker for the host machine: cc ld64 1115.5.3
Host machine cpu family: x86_64
Host machine cpu: x86_64
Run-time dependency threads found: YES
Message: darwin sysroot: /Library/Developer/CommandLineTools/SDKs/MacOSX15.0.sdk
...
```
... and get the error reported by Stan. Also I can confirm that the
proposed workaround fixes it. Attached is the result of `git
format-patch` for convenience.
Personally I'm not extremely happy with this workaround though. An
alternative solution would be adding the "pg_" prefix to our type
declarations.
Another question is whether we should fix this while the SDK is in
beta or only after it is released.
Thoughts?
I added the patch to the nearest commitfest so that it wouldn't be lost [1].
[1]: https://commitfest.postgresql.org/48/5073/
--
Best regards,
Aleksander Alekseev
Attachment | Content-Type | Size |
---|---|---|
v1-0001-Dirty-fix.patch | application/octet-stream | 840 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2024-06-25 13:59:14 | Re: Add new protocol message to change GUCs for usage with future protocol-only GUCs |
Previous Message | Vladimir Sitnikov | 2024-06-25 13:36:58 | Re: Direct SSL connection and ALPN loose ends |