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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude arm64 for macos #53

Merged
merged 3 commits into from
Dec 7, 2020
Merged

Exclude arm64 for macos #53

merged 3 commits into from
Dec 7, 2020

Conversation

surpher
Copy link
Owner

@surpher surpher commented Dec 7, 2020

馃摑 Summary of Changes

arm64 and arm64e architectures for macOS are not yet fully (and painlessly) supported by all the tools and dependencies when dealing with PactSwift and specifically the Rust libpact_mock_server.a. Building the libpact_mock_server.a binary with aarch64-apple-darwin triple continues to fail. As arm64 is still scarce in the wild, I decided to exclude from valid architectures for macOS target for now.

鈿狅笍 Items of Note

  • Relates to $52

馃馃棐 Reviewer Notes

I'm keeping a close eye on the progress of aarch64-apple-darwin availability. As soon as it is available in the stable toolchain and building the dependencies succeeds, this will be reverted.

馃敤 How To Test

  • CI passes

`arm64` is not fully supported by all of the Rust dependencies. Until #52 is open and building Rust dependency with triple `aarm64-apple-darwin` continues to fail `arm64` will not be supported.
@surpher surpher added breaking change Something that would break the existing callers tech Something technical that doesn't fit a feature or enhancement labels Dec 7, 2020
@surpher surpher merged commit c2d062f into master Dec 7, 2020
@surpher surpher deleted the tech/exclude-macos-arm64 branch December 7, 2020 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Something that would break the existing callers tech Something technical that doesn't fit a feature or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant