Re: ci: namespace ccache by PostgreSQL major version

Поиск
Список
Период
Сортировка
Искать
От
Nazir Bilal Yavuz
Тема
Re: ci: namespace ccache by PostgreSQL major version
Дата
в 11:57:18
Msg-id
CAN55FZ1EO7QJczZxDrBQrU6v32r9TGX53jeZn3-NE+ZPYafVaQ@mail.gmail.com
Ответ на
Список
Дерево обсуждения
ci: namespace ccache by PostgreSQL major version Nazir Bilal Yavuz <byavuz81@gmail.com>
Re: ci: namespace ccache by PostgreSQL major version Peter Eisentraut <peter@eisentraut.org>
Re: ci: namespace ccache by PostgreSQL major version Nazir Bilal Yavuz <byavuz81@gmail.com>
Hi,

On Fri, 17 Jul 2026 at 00:27, Andres Freund  wrote:
>
> On 2026-07-07 13:35:34 -0400, Andres Freund wrote:
> >
> > Yea, I don't think that's the right answer. I'm reasonably sure I debugged
> > this issue a while ago:
> >
> > https://postgr.es/m/phsrssp75npoyalqsolcd7fmnmlbzbmquc2p7w7mqjlw7432jk%40bzskz3luyjvb
> >
> > Not a lot has happened on the ccache front, so I suspect we should start
> > adding the -fpch-deps flag I mentioned in that thread.
> >
> > We could make adding -fpch-deps conditional on pch support being enabled, but
> > given that it doesn't do anything when pch is not used, I'd just add it when
> > supported by the compiler.
>
> Attached is the change I propose.

Thanks for the patch! LGTM and I confirm it fixes the problem.

-- 
Regards,
Nazir Bilal Yavuz
Microsoft


В списке pgsql-hackers по дате отправления
От: Matthias van de Meent
Дата:
От: Alexander Lakhin
Дата:
FAQ