Date: Fri, 14 Jun 2024 10:22:01 -0500
From: Justin Pryzby <pryzby@telsasoft.com>
To: Nazir Bilal Yavuz <byavuz81@gmail.com>
Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>,
	vignesh C <vignesh21@gmail.com>,
	Michael Paquier <michael@paquier.xyz>,
	Andres Freund <andres@anarazel.de>,
	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
Message-ID: <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>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAN55FZ2Jof7xqHbUQFKsGekWVGL6Fjhqg=pE2jNGoLyADOEa=Q@mail.gmail.com>
Archived-At: <https://www.postgresql.org/message-id/ZmxgGSuMzPQu9smA%40pryzbyj2023>
Precedence: bulk

On Fri, Jun 14, 2024 at 05:36:54PM +0300, Nazir Bilal Yavuz wrote:
> Hi,
> 
> On Thu, 13 Jun 2024 at 14:56, Justin Pryzby <pryzby@telsasoft.com> wrote:
> >
> > ccache should be installed in the image rather than re-installed on each
> > invocation.
> 
> ccache is installed in the Windows VM images now [1]. It can be used
> as 'set CC=ccache.exe cl.exe' in the Windows CI task.
> 
> [1] https://github.com/anarazel/pg-vm-images/commit/03a9225ac962fb30b5c0722c702941e2d7c1e81e

Thanks.  I think that works by using a "shim" created by choco in
C:\ProgramData\chocolatey\bin.

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.

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.

-- 
Justin