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

Add Apple Silicon/ARM wheels #1672

Closed
reinecke opened this issue Oct 24, 2023 · 2 comments · Fixed by #1689 or #1739 · May be fixed by #1673
Closed

Add Apple Silicon/ARM wheels #1672

reinecke opened this issue Oct 24, 2023 · 2 comments · Fixed by #1689 or #1739 · May be fixed by #1673
Labels
build issues building OTIO wheels

Comments

@reinecke
Copy link
Collaborator

reinecke commented Oct 24, 2023

Feature Request

We'd like to distribute Apple Silicon/ARM wheels.

Description

Right now, we're still building only classic x86_64 wheels for macOS and Apple Silicon users are forced to build from the source tarball. Ideally, we'd offer a range of prebuilt wheels that includes Apple Silicon where applicable.

Context

We've resisted this in the past because we didn't have a mechanism to test Apple Silicon artifacts to make sure they worked correctly, this looks like it may be solved now.

Checking the 0.17.0.dev1 release, we're already cross-compiling a universal binary for the python bindings:

% lipo -archs _otio.cpython-37m-darwin.so
x86_64 arm64

We also may look at reshuffling py_build_test and package_wheels - we could consider building all the wheels in one step (using x86_64 to cross-build an Apple Silicon wheel). And have a separate action that pulls down the wheels and runs the test suite against them in a matrix fashion. This way, only the test suite would run on an expensive Apple Silicon runner and the rest can happen on free runners.

References:

@JeanChristopheMorinPerso
Copy link
Member

Cross linking #1027

@JeanChristopheMorinPerso
Copy link
Member

@JeanChristopheMorinPerso JeanChristopheMorinPerso linked a pull request Apr 14, 2024 that will close this issue
@reinecke reinecke added this to the Public Beta 17 milestone Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment