Re: ci: namespace ccache by PostgreSQL major version
От
Andres Freund
Тема
Re: ci: namespace ccache by PostgreSQL major version
Дата
Msg-id
ybeudonc63cejz55fjlfzsqnylt6svipl6pgxaturlvqylq5xv@y7aokc35oak6
Ответ на
Re: ci: namespace ccache by PostgreSQL major version (Nazir Bilal Yavuz)
Список
Дерево обсуждения
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 2026-07-17 11:57:18 +0300, Nazir Bilal Yavuz wrote: > 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. Thanks for reviewing. Pushed. Greetings, Andres
В списке pgsql-hackers по дате отправления
От: Paul Kim
Дата: