-
Notifications
You must be signed in to change notification settings - Fork 616
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
Fix macOS arm64 build #1434
Fix macOS arm64 build #1434
Conversation
Signed-off-by: Julian Amann <julian.amann@tum.de>
c70de4b
to
ea63daf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thank you. It doesn't seem to failing on CI for CMake though, I assume something else might be including arm_neon.h indirectly and not from Bazel.
@meshula It does not fail on CI since GitHub action only run on macOS Intel - not macOS arm64 (e.g. M1 or M2 processors) |
I'm going to merge this into 3.1.8, since it seems likely to be a problem. Thanks. |
Signed-off-by: Julian Amann <julian.amann@tum.de>
FYI, macOS arm64 runners are now available for CI. |
@kmilos I tried |
Only macos-14 is available for public repositories: https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/ |
@kmilos Thanks for the pointer. OpenEXR Bazel build works works already: https://github.com/AcademySoftwareFoundation/openexr/actions/runs/7762614654/job/21173364638?pr=1626 |
Commit 7c40603 introduced with PR #1402 a break of
bazel build //...
on macOS with arm64:This PR fixes the issue (I assume similar behavior with CMake build)