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

Use M1 runner for macOS arm64 wheels #1323

Merged
merged 3 commits into from
Feb 5, 2024

Conversation

keitherskine
Copy link
Collaborator

Now that GitHub has made an ARM64 runner available - hurrah! - let's use it to create the problematic macOS ARM64 wheels.

In the GitHub Action artifacts script, I have separated out the builds for each platform into discrete jobs. They are all slightly different and it seems sensible to have a specific job for each platform.

We were getting a bunch of deprecation warnings about old versions of node, so I have updated the versions of actions/setup-python from v4 to v5, actions/upload-artifact from v3 to v4, and github/codeql-action from v2 to v3. I have also taken the opportunity to pin various actions to specific minor versions so we know exactly what is being run.

Lastly, I have swapped around the Appveyor builds so that Python 3.12 is done first and Python 3.8 is done last. On the free plan, Appveyor jobs are sequential (and slow), and typically it's the Python 3.12 build we are most interested in, so do it first.

Use the new macos-14 (M1) runner for creating ARM64 wheels.  In the Github Action artifacts script, separate out the various platforms for clarity.  Update actions/setup-python to v5 to suppress node deprecation warning.  Update, and pin, other actions accordingly.
Appveyor builds are slow and sequential, so do the most important one first, i.e. Python 3.12.
Version 2 of github/codeql-action is now deprecated.
@keitherskine
Copy link
Collaborator Author

You can see examples of the wheels that are generated here: https://github.com/keitherskine/pyodbc/actions/runs/7775109101
(they can't be built in the main mkleehammer pyodbc repo until this PR is merged into master)

This should hopefully fix #1124 .

@keitherskine
Copy link
Collaborator Author

I have tested the arm64 wheel on an M1 Macbook Pro. If anybody can test it on an M3 that would be appreciated.

@mkleehammer
Copy link
Owner

This is amazing @keitherskine . I was looking into this as soon as I got the email from Github about the new runners, but you are way ahead of me, as usual.

@mkleehammer mkleehammer merged commit 091b45d into mkleehammer:master Feb 5, 2024
9 checks passed
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.

2 participants