From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Sutou Kouhei <kou(at)clear-code(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: meson: catalog/syscache_ids.h isn't installed |
Date: | 2024-02-05 10:17:07 |
Message-ID: | 202402051017.bvrvvbkr3bdr@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2024-Feb-05, Michael Paquier wrote:
> On Mon, Feb 05, 2024 at 03:55:36PM +0900, Masahiko Sawada wrote:
> > Peter, could you check this fix as it seems the recent commits forgot
> > to update the meson.build file?
>
> The patched code is telling us that we're forgetting to install
> syscache_ids.h in the set of headers that should be part of the
> server-side includes installed. In short, Sutou-san is right, this
> is the right fix, and this is a thinko from 9b1a6f50b91d.
What the original rule said was that the file should be installed in
'false', and now we want it installed in include/catalog. So the
oversight is that the original commit seems to have said that we didn't
want the file installed (that's what 'false' means), which is a strange
decision.
The docs here
https://mesonbuild.com/Reference-manual_functions.html#custom_target_install_dir
seem to explain this well.
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
"But static content is just dynamic content that isn't moving!"
http://smylers.hates-software.com/2007/08/15/fe244d0c.html
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2024-02-05 10:17:11 | Improve pg_trigger.tgargs representation |
Previous Message | vignesh C | 2024-02-05 10:13:41 | Re: Race condition in FetchTableStates() breaks synchronization of subscription tables |