Re: Build warning with meson and dtrace on Fedora 43

From: Andrey Rachitskiy <pl0h0yp1(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Build warning with meson and dtrace on Fedora 43
Date: 2026-07-30 05:32:00
Message-ID: CAB8bMisA1rHTSPmtGkLBYZcTteUJ91BxrZjbFRxdu2qO8KvAuQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> Pushed with minor additional comment-smithing. I also double-checked
> that macOS's current behavior applies at least as far back as
> macOS 14 (Sonoma), so we should be good on any version that anyone
> is likely to install PG 20 on.
>
> [...]
>
> BTW, I realized that we had no buildfarm coverage of --enable-dtrace
> on macOS, so I've enabled that on indri.

Thanks for pushing, and for the extra comment polish.
Having --enable-dtrace enabled on indri should help catch macOS-specific
issues sooner.

чт, 30 июл. 2026 г. в 05:33, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:

> Andrey Rachitskiy <pl0h0yp1(at)gmail(dot)com> writes:
> > I've
> > rewritten the probes.d note along those lines and kept ssize_t/size_t
> > for the smgr probes (system-supplied, matching md.c).
>
> > Updated patch attached.
>
> Pushed with minor additional comment-smithing. I also double-checked
> that macOS's current behavior applies at least as far back as
> macOS 14 (Sonoma), so we should be good on any version that anyone
> is likely to install PG 20 on.
>
> I noticed that we had
>
> #define bool unsigned char
>
> in there, and tried to get rid of that, reasoning that now that we
> use <stdbool.h> "bool" does not mean "unsigned char". However, macOS
> rejects "bool", so I desisted. Thinking twice, it looks like we
> could instead do
>
> #define bool _Bool
>
> but that seems like a matter for a separate patch, perhaps.
>
> BTW, I realized that we had no buildfarm coverage of --enable-dtrace
> on macOS, so I've enabled that on indri.
>
> regards, tom lane
>

--
Regards,
Rachitskiy Andrey

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Chao Li 2026-07-30 05:45:49 Re: A new C function `get_partition_root`.
Previous Message Andrey Rachitskiy 2026-07-30 05:30:56 Fix BUG #19586: money division overflow for INT64_MIN / -1