Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify vcpkg caching, update runners to latest images, add macos M1 runner #7

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

timkpaine
Copy link
Member

@timkpaine timkpaine commented Feb 2, 2024

This PR makes minor adjustments to the vcpkg caching. In particular, it seems macOS is ignoring the caching for its cibuildwheel steps. This PR hopes to rectify that.

Docs:

This PR also upgrades from ubuntu-20.04 and macos-11 runners to ubuntu-22.04 and macos-12 runners, the current aliases for ubuntu-latest and macos-latest respectively.

Finally, this PR adds the brand new macos-14 runners which run on M1 arm chips. This is critical because we currently require GNU toolchain, and the homebrew build cannot cross-compile, so we cannot e.g. build arm wheels from x86 chips. With the new M1 runners, we can build native arm wheels for mac.

@timkpaine timkpaine changed the title try different caching strategy for vcpkg Try different caching strategy for vcpkg Feb 2, 2024
@timkpaine timkpaine added the part: github_actions PRs that update GitHub Actions code label Feb 2, 2024
@timkpaine timkpaine force-pushed the tkp/cache branch 28 times, most recently from 6373797 to 14306d2 Compare February 3, 2024 22:23
@timkpaine timkpaine force-pushed the tkp/cache branch 3 times, most recently from 9afbc62 to 38a0f08 Compare February 4, 2024 23:54
@timkpaine timkpaine changed the title Try different caching strategy for vcpkg Modify vcpkg caching, update runners to latest images, add macos M1 runner Feb 5, 2024
@timkpaine timkpaine marked this pull request as ready for review February 5, 2024 14:18
Copy link
Contributor

@ngoldbaum ngoldbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the output from the caching step I see output like: Cache not found for input keys: vcpkg-Linux-X64-7a0833a6925c4ade0c75ec84a7e2d2b43717272adddee7809c9cbf6878b86422, vcpkg-Linux-X64-. I don't immediately see what's wrong with the setup of the cache to trigger that though, or even if it's a problem.

Also one tiny nit in pyproject.toml, ignore if you disagree.

pyproject.toml Outdated Show resolved Hide resolved
…dd mac arm runner

Bump to ubuntu 22.04 and macOS-12 runners
Add new macOS 14 M1 runner for arm wheels

Signed-off-by: Tim Paine <[email protected]>
@timkpaine timkpaine merged commit 03b1f51 into main Feb 6, 2024
10 checks passed
@timkpaine timkpaine deleted the tkp/cache branch February 6, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part: github_actions PRs that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants