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

help / color / mirror / Atom feed
From: Peter Eisentraut <peter@eisentraut.org>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Thomas Munro <thomas.munro@gmail.com>
Cc: Sam James <sam@gentoo.org>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: Build failure with GCC 15 (defaults to -std=gnu23)
Date: Thu, 21 Nov 2024 08:23:05 +0100
Message-ID: <b906d958-4b37-47f3-9efb-79b575042661@eisentraut.org> (raw)
In-Reply-To: <152145.1732116729@sss.pgh.pa.us>
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>

On 20.11.24 16:32, Tom Lane wrote:
> Peter Eisentraut <peter@eisentraut.org> writes:
>> Note that if we backpatch C23 support, we also need to backpatch at
>> least commits a67a49648d9 and d2b4b4c2259.
> 
> Yeah.  Our normal theory for this kind of thing is "people are
> likely to build our old branches with modern toolchains", so
> we are going to have to back-patch C23 compatibility sooner or
> later.  In fact, we'll have to back-patch to 9.2, or else
> decide that those branches are unbuildable on modern platforms
> and hence out of scope for compatibility testing.

I have checked that with this patch and the two above (well, one is just 
to remove a warning), you can get PG16 and up building cleanly with 
gcc-14 -std=gnu23.

Before that, you get a ton of warnings and errors related to the node 
tree walker routines.  This is presumably related to commit 1c27d16e6e5.

Going further back, the bool patch proposed here assumes that stdbool.h 
exists unconditionally, which is C99, which is not the baseline for 
older branches.  I think for those it's probably best to leave it alone 
and just use gcc-15 -std=gnu89 or whatever.






view thread (42+ messages)  latest in thread

Message-ID: <b906d958-4b37-47f3-9efb-79b575042661@eisentraut.org>
Permalink:  ../b906d958-4b37-47f3-9efb-79b575042661@eisentraut.org/
Also on:    postgresql.org/message-id/b906d958-4b37-47f3-9efb-79b575042661@eisentraut.org

 · 

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: peter@eisentraut.org, tgl@sss.pgh.pa.us, thomas.munro@gmail.com, sam@gentoo.org, pgsql-bugs@lists.postgresql.org
  Subject: Re: Build failure with GCC 15 (defaults to -std=gnu23)
  In-Reply-To: <b906d958-4b37-47f3-9efb-79b575042661@eisentraut.org>

* 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