Cache the Go installer in AppVeyor CI (PR #26132)
Maarten
There were some issues downloading the Go installer in AppVeyor CI, see #26116 (comment)
Try to cache it.
You can view, comment on, or merge this pull request online at:
https://github.com/wxWidgets/wxWidgets/pull/26132
Commit Summary
- 2b46d8c Cache the Go installer in AppVeyor CI
File Changes
(1 file)
- M appveyor.yml (12)
Patch Links:
- https://github.com/wxWidgets/wxWidgets/pull/26132.patch
- https://github.com/wxWidgets/wxWidgets/pull/26132.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
@MaartenBent pushed 1 commit.
- bb68c1f test with quotes
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#26132)Thanks a lot! I am not sure if the cache is shared between jobs or not? If not, it might still not work because this seems to be failing all the time now (I think there is really something wrong in this image, see the issue I opened), but maybe it could still work at least once...
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
@MaartenBent pushed 1 commit.
- 8fa2f52 use existing path
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
MaartenBent left a comment (wxWidgets/wxWidgets#26132)I am not sure if the cache is shared between jobs or not?
I think it is. The cache variable is a global entry in the file, not specific to a job in the matrix. Once it works we can see if it redownloads it in the 2nd and 3rd job.
It is not always failing, there have been some successful builds the last days https://ci.appveyor.com/project/wxWidgets/wxwidgets/history
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
@MaartenBent pushed 1 commit.
- 62de689 single line
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
MaartenBent left a comment (wxWidgets/wxWidgets#26132)https://www.appveyor.com/docs/build-cache/
Note: By default, saving cache is disabled in Pull Request builds.
Seems a PR doesn't create the cache. Maybe merge and see what happens?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#26132)OK, will do in a moment.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
MaartenBent left a comment (wxWidgets/wxWidgets#26132)It seems to work. Though all 3 jobs cache it separately. Their logs contain:
set go_exe=go%GOVERSION%.windows-amd64.msi
if not exist "%go_exe%" appveyor DownloadFile https://go.dev/dl/%go_exe%
Downloading go1.22.1.windows-amd64.msi (63,164,416 bytes)...100%
msiexec /i %go_exe% INSTALLDIR="%GOROOT%" /q
go version
go version go1.22.1 windows/amd64
...
Updating build cache...
Cache 'C:\Users\appveyor\go1.22.1.windows-amd64.msi' - Updated
Build success
The latest AppVeyor build that just started does not download the installer:
set go_exe=go%GOVERSION%.windows-amd64.msi
if not exist "%go_exe%" appveyor DownloadFile https://go.dev/dl/%go_exe%
msiexec /i %go_exe% INSTALLDIR="%GOROOT%" /q
go version
go version go1.22.1 windows/amd64
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#26132)Great, thanks a lot once again!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.