Re: Inconsistency in installation of syscache_info.h

Andres Freund <andres@anarazel.de>

From: Andres Freund <andres@anarazel.de>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers@lists.postgresql.org
Date: 2026-02-16T19:16:28Z
Lists: pgsql-hackers
Hi,

On 2026-02-16 14:01:38 -0500, Tom Lane wrote:
> I wrote:
> > I happened to notice that syscache_info.h is installed into
> > $installdir/include/server/catalog if you use a non-VPATH
> > autoconf build, but not if you use a VPATH build or meson.
> > This is not great IMO.

Indeed. If it were trivial to not install it, I'd have gone that way, but it
looks like it's not...


> > Just to confuse matters even more, meson builds install a copy of
> > src/include/catalog/README, which autoconf builds do not.
> 
> > ISTM we need to get these things in sync.
> 
> Here's a proposed patch for that.  I've confirmed we get identical
> filesets from all three build methods with this.

Nice.


> Although this seems like clearly a bug, I'm not sure anyone would
> thank us for changing the behavior in back branches; particularly
> not packagers, who tend to pay close attention to the set of files
> installed.  So I'm inclined to apply this to master only.

Agreed. Given that syscache_info.h should never be needed, it's not like it'll
bother anyone that it's not there.   And it looks like
src/include/catalog/README is new enough that the installation in meson
happens only on master anyway.

Greetings,

Andres Freund



Commits

  1. Ensure that all three build methods install the same set of files.