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 better support for cross compilation for OSX #2354

Merged

Conversation

Shelnutt2
Copy link
Member

This adjusts the super build in order to support cross compilation for OSX. This has been tested for compiling x86_64 from arm64. Only GCS is not supported and this is due to abseil/abseil-cpp#970 blocking the build by using the host instead of target for setting flags. Once abseil/abseil-cpp#970 is solved GCS should work based on the patches included in this PR.

Enable cross compilation using the CMAKE_OSX_ARCHITECTURES flag. Currently we do not support a universal lib.
-DCMAKE_OSX_ARCHITECTURES="arm64" or -DCMAKE_OSX_ARCHITECTURES="x86_64"

I plan to squash on merge, this was left in separate commits for ease of review.


TYPE: IMPROVEMENT
DESC: Add support for cross compilation on OSX in superbuild

@shortcut-integration
Copy link

This pull request has been linked to Clubhouse Story #8069: Test cross compilation for osx arm64.

Copy link
Member

@ihnorton ihnorton left a comment

Choose a reason for hiding this comment

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

🎉

@Shelnutt2 Shelnutt2 force-pushed the sethshelnutt/ch8069/test-cross-compilation-for-osx-arm64 branch from 0e155dc to 4b57793 Compare June 15, 2021 13:54
@Shelnutt2 Shelnutt2 merged commit 2d6186c into dev Jun 15, 2021
@Shelnutt2 Shelnutt2 deleted the sethshelnutt/ch8069/test-cross-compilation-for-osx-arm64 branch June 15, 2021 20:47
@Shelnutt2 Shelnutt2 restored the sethshelnutt/ch8069/test-cross-compilation-for-osx-arm64 branch June 15, 2021 20:47
@Shelnutt2 Shelnutt2 deleted the sethshelnutt/ch8069/test-cross-compilation-for-osx-arm64 branch June 15, 2021 20:47
github-actions bot pushed a commit that referenced this pull request Jun 15, 2021
* Add support CMAKE_OSX_ARCHITECTURES for superbuild

* Update openssl configure command for OSX cross compilation

* Adjust bzip2 to support MacOS cross compilation

* Support CURL cross compilation for MacOS

* GCS MacOS cross-compilation tweaks

* AWSSDK needs CMAKE_OSX_SYSROOT and CMAKE_OSX_DEPLOYMENT_TARGET for cross compilation
Shelnutt2 added a commit that referenced this pull request Jun 15, 2021
* Add support CMAKE_OSX_ARCHITECTURES for superbuild

* Update openssl configure command for OSX cross compilation

* Adjust bzip2 to support MacOS cross compilation

* Support CURL cross compilation for MacOS

* GCS MacOS cross-compilation tweaks

* AWSSDK needs CMAKE_OSX_SYSROOT and CMAKE_OSX_DEPLOYMENT_TARGET for cross compilation

Co-authored-by: Seth Shelnutt <seth@tiledb.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants