pgsql: Remove unneeded defines from pg_config.h.in

Started by Michael Paquier7 months ago6 messagescomitters
Jump to latest
#1Michael Paquier
michael@paquier.xyz

Remove unneeded defines from pg_config.h.in

This commit removes HAVE_ATOMIC_H and HAVE_MBARRIER_H from
pg_config.h.in, cleanup that could have been done in 25f36066dd2a.

Author: Alexander Lakhin <exclusion@gmail.com>
Discussion: /messages/by-id/b2c0d0b7-3944-487d-a03d-d155851958ff@gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/7d854bdc5b72b30ae58041d88c2be509950b99d4

Modified Files
--------------
src/include/pg_config.h.in | 6 ------
1 file changed, 6 deletions(-)

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Michael Paquier (#1)
Re: pgsql: Remove unneeded defines from pg_config.h.in

Michael Paquier <michael@paquier.xyz> writes:

Remove unneeded defines from pg_config.h.in
This commit removes HAVE_ATOMIC_H and HAVE_MBARRIER_H from
pg_config.h.in, cleanup that could have been done in 25f36066dd2a.

This patch is not okay. pg_config.h.in is a generated file and
hand modifications in it will not do.

If we don't need these symbols anymore, the right fix is to remove
the probes for these headers in the autoconf and meson scripts.

regards, tom lane

#3Michael Paquier
michael@paquier.xyz
In reply to: Tom Lane (#2)
Re: pgsql: Remove unneeded defines from pg_config.h.in

On Sun, Jan 04, 2026 at 07:41:23PM -0500, Tom Lane wrote:

This patch is not okay. pg_config.h.in is a generated file and
hand modifications in it will not do.

If we don't need these symbols anymore, the right fix is to remove
the probes for these headers in the autoconf and meson scripts.

Oops, sorry for the brain fart. How about the attached? autoreconf
and autoheader seem to be OK with it.
--
Michael

Attachments:

fix_autoconf.patchtext/x-diff; charset=us-asciiDownload+1-5
#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Michael Paquier (#3)
Re: pgsql: Remove unneeded defines from pg_config.h.in

Michael Paquier <michael@paquier.xyz> writes:

On Sun, Jan 04, 2026 at 07:41:23PM -0500, Tom Lane wrote:

If we don't need these symbols anymore, the right fix is to remove
the probes for these headers in the autoconf and meson scripts.

Oops, sorry for the brain fart. How about the attached? autoreconf
and autoheader seem to be OK with it.

Looks about right to me.

regards, tom lane

#5Michael Paquier
michael@paquier.xyz
In reply to: Tom Lane (#4)
Re: pgsql: Remove unneeded defines from pg_config.h.in

On Sun, Jan 04, 2026 at 08:09:40PM -0500, Tom Lane wrote:

Looks about right to me.

Thanks, applied now. And sorry about the noise.
--
Michael

#6Tom Lane
tgl@sss.pgh.pa.us
In reply to: Michael Paquier (#5)
Re: pgsql: Remove unneeded defines from pg_config.h.in

Michael Paquier <michael@paquier.xyz> writes:

Thanks, applied now. And sorry about the noise.

Thanks for fixing.

regards, tom lane