Re: PostgreSQL does not compile on macOS SDK 15.0

From: Andres Freund <andres(at)anarazel(dot)de>
To: Aleksander Alekseev <aleksander(at)timescale(dot)com>
Cc: Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Stan Hu <stanhu(at)gmail(dot)com>
Subject: Re: PostgreSQL does not compile on macOS SDK 15.0
Date: 2024-06-25 14:31:17
Message-ID: 20240625143117.z5yc2rfh37ugyhxb@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2024-06-25 16:49:32 +0300, Aleksander Alekseev wrote:
> ... to make sure Postgres will not find the older version of SDK (it
> did until I made this step).

You should be able to influence that by specifying -Ddarwin_sysroot=...

> ... 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.

Yea, it seems decidedly not great.

> An alternative solution would be adding the "pg_" prefix to our type
> declarations.

A third approach would be to make sure we don't include xlocale.h from
pg_locale.h. IMO pg_locale currently exposes too many implementation details,
neither xlocale.h nor ucol.h should be included in it, that should be in a C
file.

> Another question is whether we should fix this while the SDK is in
> beta or only after it is released.

Yea.

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2024-06-25 14:31:33 Re: Vacuum ERRORs out considering freezing dead tuples from before OldestXmin
Previous Message Tom Lane 2024-06-25 14:26:00 Re: Backporting BackgroundPsql