pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Peter Eisentraut <peter@eisentraut.org>
Cc: Sam James <sam@gentoo.org>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: Build failure with GCC 15 (defaults to -std=gnu23)
Date: Mon, 25 Nov 2024 15:07:03 -0500
Message-ID: <1447160.1732565223@sss.pgh.pa.us> (raw)
In-Reply-To: <CA+hUKGL0LWK7hVO9r2eb3hFDBptZByOkAKGAjXKJ5mqd9hs_fA@mail.gmail.com>
References: <87o72eo9iu.fsf@gentoo.org>
	<3073930.1731867112@sss.pgh.pa.us>
	<CA+hUKGJsuBiBcYnmJhGUNEg6Ud3n+C3K8c6em6RTOgu1E-dbwg@mail.gmail.com>
	<3160019.1731875201@sss.pgh.pa.us>
	<CA+hUKGLJnhpJBrsutH2aK91ARMS+XCez5P8XrR-kwdfSyLXGiQ@mail.gmail.com>
	<3169079.1731880169@sss.pgh.pa.us>
	<CA+hUKGJEnmapE7YYEfdJYZ-yGmcDpVGRrQ_eEVdcux26ZNJq_g@mail.gmail.com>
	<3175762.1731883847@sss.pgh.pa.us>
	<CA+hUKGJBNYAAYE51Ef-reKo1cUDvs3GgnChyh4ETqiSVQ0MR5w@mail.gmail.com>
	<dec2bd5b-66e3-4750-83ef-e597d539c2f4@eisentraut.org>
	<152145.1732116729@sss.pgh.pa.us>
	<b906d958-4b37-47f3-9efb-79b575042661@eisentraut.org>
	<CA+hUKGJma08NBfwv8kiLAW08wFaz-WxDuXWbENi92XdE46N6ZA@mail.gmail.com>
	<485f5982-18c5-4210-9c7d-7c05453f7033@eisentraut.org>
	<CA+hUKGL0LWK7hVO9r2eb3hFDBptZByOkAKGAjXKJ5mqd9hs_fA@mail.gmail.com>

Thomas Munro <thomas.munro@gmail.com> writes:
> Oh.  Yeah.  1c27d16e6e5 was not back-patchable.  And what f9a56e72 did
> in 15 and older doesn't seem to have any equivalent in C23, at least
> without going way overboard.  -Wdeprecated-non-prototype was
> recognising a category of function type that no longer exists, so the
> code now falls into the more general case of
> -Wincompatible-pointer-types in C23, which you certainly wouldn't want
> to suppress.  So perhaps we actually can't make any branch older than
> PostgreSQL 16 into a valid C23 program, and if that's true, I needn't
> have back-patched the <stdbool.h> change any further back than 16.
> Perhaps we should reconsider that, then.  And if it can't be all the
> back-branches, we could even decide to focus just on master.  Where do
> we want our C23 support to begin?

Unless somebody has a better idea than 1c27d16e6e5, it would seem
reasonable to say that we'll support C23 in v16 and later, but
to build an older branch you have to back off to an older C version.

I don't feel a need to revert those <stdbool.h> changes.
If nothing else, that saved a bit of configure runtime.

If we leave it like this, alligator will need some configuration
adjustments, and so will other BF animals when they migrate to new
gcc, and so will individual hackers when they're trying to build
old branches.  A possible compromise to reduce the manual pain
level could be to adjust configure to add "-std=gnu99" or so to
CFLAGS in the pre-v16 branches, if the compiler accepts that.
(OTOH, maybe that'd cause pain for some extensions?)

			regards, tom lane





view thread (42+ messages)  latest in thread

Message-ID: <1447160.1732565223@sss.pgh.pa.us>
Permalink:  ../1447160.1732565223@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/1447160.1732565223@sss.pgh.pa.us

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: tgl@sss.pgh.pa.us, thomas.munro@gmail.com, peter@eisentraut.org, sam@gentoo.org, pgsql-bugs@lists.postgresql.org
  Subject: Re: Build failure with GCC 15 (defaults to -std=gnu23)
  In-Reply-To: <1447160.1732565223@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox