| From: | Sutou Kouhei <kou(at)clear-code(dot)com> |
|---|---|
| To: | peter(at)eisentraut(dot)org |
| Cc: | tristan(at)neon(dot)tech, michael(at)paquier(dot)xyz, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: meson: Specify -Wformat as a common warning flag for extensions |
| Date: | 2024-03-15 09:36:55 |
| Message-ID: | 20240315.183655.616433279210896579.kou@clear-code.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
In <49e97fd0-c17e-4cbc-aeee-80ac51400736(at)eisentraut(dot)org>
"Re: meson: Specify -Wformat as a common warning flag for extensions" on Wed, 13 Mar 2024 08:38:28 +0100,
Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
> I think the fix then is to put -Wall into CFLAGS in
> Makefile.global. Looking at a diff of Makefile.global between an
> autoconf and a meson build, I also see that under meson, CFLAGS
> doesn't get -O2 -g (or similar, depending on settings). This
> presumably has the same underlying issue that meson handles those
> flags internally.
>
> For someone who wants to write a fix for this, the relevant variable
> is var_cflags in our meson scripts. And var_cxxflags as well.
How about the attached v4 patch?
Thanks,
--
kou
| Attachment | Content-Type | Size |
|---|---|---|
| v4-0001-meson-Restore-implicit-warning-debug-optimize-fla.patch | text/x-patch | 3.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Aleksander Alekseev | 2024-03-15 09:47:42 | Re: UUID v7 |
| Previous Message | Daniel Gustafsson | 2024-03-15 09:32:00 | Re: Support json_errdetail in FRONTEND builds |