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

Support building for Android on aarch64 macOS hosts #216

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

fornwall
Copy link
Contributor

@fornwall fornwall commented Oct 9, 2023

The NDK puts universal binaries under the darwin-x86_64 directory.

Verified that it works by patching Cargo.toml to use this branch while building on an M1 macbook.

@fornwall
Copy link
Contributor Author

fornwall commented Oct 9, 2023

The build error is a new clippy lint, fixed by #217

@fornwall fornwall force-pushed the android-build-on-aarch64-apple branch from d87318b to ebde05f Compare October 10, 2023 07:39
Copy link
Member

@tgolsson tgolsson left a comment

Choose a reason for hiding this comment

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

Actually, changelogs. Then OK.

@fornwall fornwall force-pushed the android-build-on-aarch64-apple branch from ebde05f to cecddb6 Compare October 10, 2023 07:51
@fornwall fornwall force-pushed the android-build-on-aarch64-apple branch from cecddb6 to 1444821 Compare October 10, 2023 07:53
The NDK puts universal binaries under the darwin-x86_64 directory.
@fornwall fornwall force-pushed the android-build-on-aarch64-apple branch from 1444821 to 0253a35 Compare October 10, 2023 07:55
@fornwall
Copy link
Contributor Author

@tgolsson Thanks, added a changelog entry now.

@mergify mergify bot merged commit a734a69 into main Oct 10, 2023
10 checks passed
@mergify mergify bot deleted the android-build-on-aarch64-apple branch October 10, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants