Re: ci: namespace ccache by PostgreSQL major version - Mailing list pgsql-hackers

From Nazir Bilal Yavuz
Subject Re: ci: namespace ccache by PostgreSQL major version
Date
Msg-id CAN55FZ1EO7QJczZxDrBQrU6v32r9TGX53jeZn3-NE+ZPYafVaQ@mail.gmail.com
Whole thread
In response to Re: ci: namespace ccache by PostgreSQL major version  (Andres Freund <andres@anarazel.de>)
Responses Re: ci: namespace ccache by PostgreSQL major version
List pgsql-hackers
Hi,

On Fri, 17 Jul 2026 at 00:27, Andres Freund <andres@anarazel.de> 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 by date:

Previous
From: Matthias van de Meent
Date:
Subject: Re: Init connection time grows quadratically
Next
From: Alexander Lakhin
Date:
Subject: Re: Unexpected behavior after OOM errors