Re: [PATCH] Missing dep on Catalog.pm in meson rules

Поиск
Список
Период
Сортировка
От Tristan Partin
Тема Re: [PATCH] Missing dep on Catalog.pm in meson rules
Дата
Msg-id CT1V90ZD4ZPZ.2J5QU7ZEYSB60@gonk
обсуждение исходный текст
Ответ на Re: [PATCH] Missing dep on Catalog.pm in meson rules  (Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>)
Ответы Re: [PATCH] Missing dep on Catalog.pm in meson rules  (Andres Freund <andres@anarazel.de>)
Список pgsql-hackers
On Thu Jun 1, 2023 at 4:22 PM CDT, Dagfinn Ilmari Mannsåker wrote:
> On Thu, 1 Jun 2023, at 22:16, Tristan Partin wrote:
> > Could you create a variable for the file instead of calling files() 3
> > times?
> >
> >> catalog_pm = files('path/to/Catalog.pm')
>
> Sure, but which meson.build file should it go in? I know nothing about meson variable scoping.

Not a problem. In Meson, variables are globally-scoped. You can use
unset_variable() however to unset it.

In our case, we should add the ^line to src/backend/catalog/meson.build.
I would say just throw the line after the copyright comment. Hopefully
there isn't a problem with the ordering of the Meson file tree traversal
(ie the targets you are changing are configured after we get through
src/backend/catalog/meson.build).

--
Tristan Partin
Neon (https://neon.tech)



В списке pgsql-hackers по дате отправления:

Предыдущее
От: "Fujii.Yuki@df.MitsubishiElectric.co.jp"
Дата:
Сообщение: RE: Partial aggregates pushdown
Следующее
От: Richard Guo
Дата:
Сообщение: Tighten a little bit the computation of potential commutator pairs