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