Skip to content

Conversation

@ADKaster
Copy link
Member

This is a workaround for the deprecation of the cache v1 REST API that was replaced with a new protobuf RPC based API this month. vcpkg was using the private cache backend API without the knowledge of the GitHub actions team, and was thus broken by the deprecation.

While we wait for Microsoft to talk to Microsoft to get this feature restored, we can use the raw actions/cache step to get almost the same cache behavior. The only difference is that the cache will be less fine grained than the per-package cache that VCPKG_BINARY_SOURCES of x-gha was giving us before.

re: #4391
upstream: microsoft/vcpkg#45073

This is a workaround for the deprecation of the cache v1 REST API that
was replaced with a new protobuf RPC based API this month. vcpkg was
using the private cache backend API without the knowledge of the GitHub
actions team, and was thus broken by the deprecation.

While we wait for Microsoft to talk to Microsoft to get this feature
restored, we can use the raw actions/cache step to get almost the same
cache behavior. The only difference is that the cache will be less fine
grained than the per-package cache that VCPKG_BINARY_SOURCES of x-gha
was giving us before.
@ADKaster
Copy link
Member Author

as far as I can tell, the cache parameters for the test262 job are completely bogus. We don't even have a ccache for that job. Presumably because it's on a private runner? So I didn't touch it here.

@ADKaster ADKaster requested a review from trflynn89 April 24, 2025 21:06
@ADKaster ADKaster enabled auto-merge (rebase) April 24, 2025 21:15
@ADKaster ADKaster merged commit 6eb0e9a into LadybirdBrowser:master Apr 24, 2025
10 of 11 checks passed
@SNMetamorph
Copy link

SNMetamorph commented Apr 26, 2025

While we wait for Microsoft to talk to Microsoft to get this feature restored, we can use the raw actions/cache step to get almost the same cache behavior. The only difference is that the cache will be less fine grained than the per-package cache that VCPKG_BINARY_SOURCES of x-gha was giving us before.

It is unlikely that they will restore it, and this fact makes this PR especially useful.
изображение

@ADKaster
Copy link
Member Author

Thanks. It looks like this position of the vcpkg team is more formalized in this vcpkg-tool PR microsoft/vcpkg-tool#1662

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants