Skip to content

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Dec 1, 2025

Overview: What does this pull request change?

Test on Apple Silicon ARM64

Motivation and Explanation: Why and how do your changes improve the library?

Ensure that our CI tests pass on modern Macs.

The currently available GitHub Actions macOS runners are:

macOS Version runner.arch
macos-14 ARM64
macos-15 ARM64
macos-15-intel X64
macos-26 ARM64
macos-latest ARM64

Links to added or changed documentation pages

Further Information and Comments

Reviewer Checklist

  • The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
  • If applicable: newly added functions and classes are tested

@github-project-automation github-project-automation bot moved this to 🆕 New in Dev Board Dec 1, 2025
@cclauss cclauss force-pushed the test-on-apple-silicon branch 6 times, most recently from f133c00 to d535f81 Compare December 1, 2025 15:05
@behackl
Copy link
Member

behackl commented Dec 2, 2025

I wonder if we should drop testing for macos-15-intel altogether and only use macos-latest -- AFAIR there were some build / linker issues with cairo when @chopan050 tried that recently. Does the combination of macos-latest with Python 3.13 just play nice(r)?

@cclauss
Copy link
Contributor Author

cclauss commented Dec 2, 2025

I think those Cairo issues are more about the macOS cache logic at

key: ${{ runner.os }}-dependencies-cairo-${{ hashFiles('.github/scripts/ci_build_cairo.py') }}

Perhaps it also needs to capture runner.arch so that Intel and ARM caches do not overwrite each other.

@cclauss cclauss force-pushed the test-on-apple-silicon branch 8 times, most recently from 7933e19 to e7379df Compare December 2, 2025 04:28
@cclauss
Copy link
Contributor Author

cclauss commented Dec 2, 2025

OK! macos-latest passes on 4 Pythons. macos-15-intel running on Py3.13 (because it is Required).

@cclauss cclauss force-pushed the test-on-apple-silicon branch from e470f93 to c63bce1 Compare December 2, 2025 04:46
@cclauss cclauss force-pushed the test-on-apple-silicon branch from 9458f0e to 7fe060f Compare December 2, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🆕 New

Development

Successfully merging this pull request may close these issues.

2 participants