Re: cpluspluscheck/headerscheck require build in REL_16_STABLE

Поиск
Список
Период
Сортировка
От Marina Polyakova
Тема Re: cpluspluscheck/headerscheck require build in REL_16_STABLE
Дата
Msg-id db4607d98a3a62d7777071dd2b60e209@postgrespro.ru
обсуждение исходный текст
Ответ на Re: cpluspluscheck/headerscheck require build in REL_16_STABLE  (John Naylor <johncnaylorls@gmail.com>)
Ответы Re: cpluspluscheck/headerscheck require build in REL_16_STABLE  (John Naylor <johncnaylorls@gmail.com>)
Список pgsql-hackers
On 2024-04-13 08:40, John Naylor wrote:
> On Fri, Apr 12, 2024 at 11:51 PM Marina Polyakova
> <m.polyakova@postgrespro.ru> wrote:
...
>> In the other branches everything is fine: these problems begin with
>> commits [2] (jsonpath_gram.h) and [3] (gram.h) and in the master 
>> branch
>> there're no such problems after commit [4]. The attached diff.patch
>> fixes this issue for me. (IIUC internal headers generated by Bison are
>> usually excluded from such checks so I also excluded gramparse.h and
>> jsonpath_internal.h...)
> 
> I'm not in favor of this patch because these files build fine on
> master, so there is no good reason to exclude them. We should arrange
> so that they build fine on PG16 as well. The problem is, not all the
> required headers are generated when invoking `make headerscheck`. The
> attached patch brings in some Makefile rules from master to make this
> work. Does this fix it for you?

Everything seems to work with this patch, thank you!

-- 
Marina Polyakova
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company



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

Предыдущее
От: Bertrand Drouvot
Дата:
Сообщение: Re: promotion related handling in pg_sync_replication_slots()
Следующее
От: Robert Haas
Дата:
Сообщение: Re: Add SPLIT PARTITION/MERGE PARTITIONS commands