Feedback-ID: id4a34324:Fastmail
Date: Fri, 14 Jun 2024 08:34:37 -0700
From: Andres Freund <andres@anarazel.de>
To: Justin Pryzby <pryzby@telsasoft.com>, Nazir Bilal Yavuz <byavuz81@gmail.com>
CC: Alvaro Herrera <alvherre@alvh.no-ip.org>, vignesh C <vignesh21@gmail.com>,
 Michael Paquier <michael@paquier.xyz>, Andrew Dunstan <andrew@dunslane.net>,
 Thomas Munro <thomas.munro@gmail.com>, pgsql-hackers@lists.postgresql.org,
 Noah Misch <noah@leadboat.com>,
 Anastasia Lubennikova <lubennikovaav@gmail.com>,
 Tom Lane <tgl@sss.pgh.pa.us>, Robert Haas <robertmhaas@gmail.com>,
 Melanie Plageman <melanieplageman@gmail.com>,
 Peter Eisentraut <peter.eisentraut@enterprisedb.com>,
 Daniel Gustafsson <daniel@yesql.se>, samay sharma <smilingsamay@gmail.com>
Subject: Re: CI and test improvements
User-Agent: K-9 Mail for Android
In-Reply-To: <ZmxgGSuMzPQu9smA@pryzbyj2023>
References: <CALDaNm1zVZFUoxRSUn3iHb-cDQx_EwZ=WTb-mCuptGqRWyy2XQ@mail.gmail.com> <202401311059.7w5bl2ftqxaj@alvherre.pgsql> <Zcu-nD7ybe7P0Hjg@pryzbyj2023> <ZmmeP8Tq36l7lp6q@pryzbyj2023> <CAN55FZ2eV+Q_9EzV=n0VEb8hz_LzAOpivttsNvffO3kgA7SyOQ@mail.gmail.com> <ZmreZDMM6MSzu2EZ@pryzbyj2023> <CAN55FZ2Jof7xqHbUQFKsGekWVGL6Fjhqg=pE2jNGoLyADOEa=Q@mail.gmail.com> <ZmxgGSuMzPQu9smA@pryzbyj2023>
Message-ID: <F47E58F9-17FD-483D-9755-23A262E0B245@anarazel.de>
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://www.postgresql.org/message-id/F47E58F9-17FD-483D-9755-23A262E0B245%40anarazel.de>
Precedence: bulk

Hi,=20

On June 14, 2024 8:22:01 AM PDT, Justin Pryzby <pryzby@telsasoft=2Ecom> wr=
ote:
>On Fri, Jun 14, 2024 at 05:36:54PM +0300, Nazir Bilal Yavuz wrote:
>> Hi,
>>=20
>> On Thu, 13 Jun 2024 at 14:56, Justin Pryzby <pryzby@telsasoft=2Ecom> wr=
ote:
>> >
>> > ccache should be installed in the image rather than re-installed on e=
ach
>> > invocation=2E
>>=20
>> ccache is installed in the Windows VM images now [1]=2E It can be used
>> as 'set CC=3Dccache=2Eexe cl=2Eexe' in the Windows CI task=2E
>>=20
>> [1] https://github=2Ecom/anarazel/pg-vm-images/commit/03a9225ac962fb30b=
5c0722c702941e2d7c1e81e
>
>Thanks=2E  I think that works by using a "shim" created by choco in
>C:\ProgramData\chocolatey\bin=2E
>
>But going through that indirection seems to incur an extra 15sec of
>compilation time; I think we'll want to do something to avoid that=2E
>
>I'm not sure what the options are, like maybe installing ccache into a
>fixed path like c:\ccache or installing a custom link, to a "pinned"
>version of ccache=2E


Hm=2E There actually already is the mingw ccache installed=2E The images f=
or mingw and msvc used to be separate (for startup performance when using c=
ontainers), but we just merged them=2E  So it might be easiest to just expl=
icitly use the ccache from there (also an explicit path might be faster)=2E=
 Could you check if that's fast? If not, we can just install the binaries d=
istributed by the project, it's just more work to keep up2date that way=2E=
=20

Andres=20
--=20
Sent from my Android device with K-9 Mail=2E Please excuse my brevity=2E