Re: meson: catalog/syscache_ids.h isn't installed - Mailing list pgsql-hackers

From Masahiko Sawada
Subject Re: meson: catalog/syscache_ids.h isn't installed
Date
Msg-id CAD21AoCnMgr=OzQSVfwC4=gR86MmND4S1p5=Gug3BoEZVaAuZg@mail.gmail.com
Whole thread Raw
In response to meson: catalog/syscache_ids.h isn't installed  (Sutou Kouhei <kou@clear-code.com>)
Responses Re: meson: catalog/syscache_ids.h isn't installed
List pgsql-hackers
On Mon, Feb 5, 2024 at 10:29 AM Sutou Kouhei <kou@clear-code.com> wrote:
>
> Hi,
>
> catalog/syscache_ids.h is referred by utils/syscache.h but
> it's not installed with Meson.
>
> FYI:
> * 9b1a6f50b91dca6610932650c8c81a3c924259f9
>   It uses catalog/syscache_ids.h in utils/syscache.h but
>   catalog/syscache_ids.h isn't installed.
> * 6eb6086faa3842c2a38a1ee2f97bf9a42ce27610
>   It changes a Makefile to install catalog/syscache_ids.h but
>   it doesn't change meson.build.
>
> ----
> diff --git a/src/include/catalog/meson.build b/src/include/catalog/meson.build
> index 6be76dca1d..0bf6e112d5 100644
> --- a/src/include/catalog/meson.build
> +++ b/src/include/catalog/meson.build
> @@ -114,7 +114,7 @@ output_install = [
>    dir_data,
>    dir_data,
>    dir_include_server / 'catalog',
> -  false,
> +  dir_include_server / 'catalog',
>    dir_include_server / 'catalog',
>    dir_include_server / 'catalog',
>  ]
> ----
>

Thank you for reporting the issue and the patch.

I've confirmed this patch fixes the issue. But I don't have enough
knowledge of meson to assess this fix.

Peter, could you check this fix as it seems the recent commits forgot
to update the meson.build file?

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Is this a problem in GenericXLogFinish()?
Next
From: Richard Guo
Date:
Subject: cfbot does not list patches in 'Current commitfest'