Re: meson: pgxs Makefile.global differences - Mailing list pgsql-hackers

From Tristan Partin
Subject Re: meson: pgxs Makefile.global differences
Date
Msg-id CUV2U1MR5T8E.2T55FGZG23INJ@gonk
Whole thread Raw
In response to meson: pgxs Makefile.global differences  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: meson: pgxs Makefile.global differences
List pgsql-hackers
On Thu Aug 17, 2023 at 2:32 PM CDT, Andrew Dunstan wrote:
> I started digging into a warning I noticed on my FDW builds where
> Postgres is built with meson, e.g.
>
<https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=crake&dt=2023-08-16%2018%3A37%3A25&stg=FileTextArrayFDW-build>

> which has this:
>
> cc1: warning: ‘-Wformat-security’ ignored without ‘-Wformat’
> [-Wformat-security]
>
> I found that the pgxs Makefile.global built under meson is a bit
> different. On debug builds for both this is what I get on HEAD (meson)
> and REL_15_STABLE (autoconf), stripped of the current components:
>
>           HEAD: CFLAGS =-Wshadow=compatible-local
> REL_15_STABLE: CFLAGS =-Wall  -g
>
> The warning is apparently due to the missing -Wall.
>
> Shouldn't we be aiming for pretty much identical settings?

I agree that they should be identical. The meson bild should definitely
be aiming for 100% compatibility for the Makefile.global.

--
Tristan Partin
Neon (https://neon.tech)



pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: meson: pgxs Makefile.global differences
Next
From: Chapman Flack
Date:
Subject: Re: Extract numeric filed in JSONB more effectively