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

Wheels: Fix macOS arm64 (M1) builds #1233

Merged
merged 24 commits into from
May 24, 2022
Merged

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Mar 18, 2022

We only built x64_64 targets so far for arm64 builds for macOS.

Setting now CMAKE_OSX_ARCHITECTURES to change that.
https://cmake.org/cmake/help/latest/envvar/CMAKE_OSX_ARCHITECTURES.html

Disable macOS universal2 builds because it's too cumbersome to build ADIOS1 and HDF5 twice and then combine them via lipo if we have two individual wheels anyway for now.

Fix macOS arm64 builds, we can cross-compile HDF5 from macOS x86-64:

Disable ADIOS1 on macOS arm64, due to missing autotools support.

Fix #1232

  • pypi: mark previous universal2 and arm64 macOS binaries as broken

We only built x64_64 targets so far for `arm64` and `universal2`
builds for macOS.

Setting now `CMAKE_OSX_ARCHITECTURES` to change that.
https://cmake.org/cmake/help/latest/envvar/CMAKE_OSX_ARCHITECTURES.html?highlight=cmake_osx_architectures#envvar:CMAKE_OSX_ARCHITECTURES
.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
@ax3l ax3l force-pushed the fix-macos-arm-builds branch 5 times, most recently from 81bb03a to 4ca8853 Compare May 24, 2022 05:24
@ax3l ax3l changed the title Wheels: Disable macOS arm64 (M1) builds Wheels: Fix macOS arm64 (M1) builds May 24, 2022
@ax3l ax3l force-pushed the fix-macos-arm-builds branch 11 times, most recently from bc458d7 to c68dee3 Compare May 24, 2022 07:39
@ax3l ax3l merged commit dc013f0 into openPMD:wheels May 24, 2022
@ax3l ax3l deleted the fix-macos-arm-builds branch May 24, 2022 18:56

cmake -S dep-hdf5/hdf5-1.12.0 -B build-hdf5 ^
cmake -S dep-hdf5/hdf5-1.12.2 -B build-hdf5 ^
Copy link
Member Author

@ax3l ax3l Jul 26, 2022

Choose a reason for hiding this comment

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

Forgot to update CIBW_ENVIRONMENT_WINDOWS which sets CMAKE_PREFIX_PATH in build.yml.
Fix in #1305

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants