| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Inconsistency in installation of syscache_info.h |
| Date: | 2026-02-16 19:16:28 |
| Message-ID: | uzq74rpl4k5bufbmelesv7cidwkp4m5g37sm5gkaq65otiio2m@lomw3bcjxizd |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| 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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jeff Davis | 2026-02-16 19:36:36 | Re: Expanding HOT updates for expression and partial indexes |
| Previous Message | Tom Lane | 2026-02-16 19:13:45 | Re: generating function default settings from pg_proc.dat |