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

Added pythonforandroid.androidndk.AndroidNDK + some changes needed in order to support build on Apple Silicon macs. #2586

Merged
merged 3 commits into from
Jul 13, 2022

Conversation

misl6
Copy link
Member

@misl6 misl6 commented Apr 22, 2022

It started as a minor change to the codebase in order to support builds on Apple Silicon without the aid of Rosetta.

Unfortunately, while doing changes, a lot of issues come up during CI runs (and locally on my M1 Mac), so this PR started to be something bigger, that fixes broken recipes and introduces the new pythonforandroid.androidndk.AndroidNDK.

This is a WIP PR to track down the current work around python-for-android recipes, and is needed in order to launch tests on our self-hosted Apple Silicon runner.

⚠️ Some changes may land into another PR in order to keep track of specific features and / or fixes.

@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 10 times, most recently from c607cc9 to 61314e9 Compare April 23, 2022 16:31
@misl6
Copy link
Member Author

misl6 commented Apr 23, 2022

✅ All tests are successful (also the ones on device)
⚠️ We should store all the APKs and AABs, so everything could be tested on a real device (ubuntu + macos-latest + apple-silicon-m1) (Targeted via #2588 )
✂️ Will start splicing the PR into multiple subsets.

@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 2 times, most recently from a018304 to 6370886 Compare April 29, 2022 18:33
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 2 times, most recently from f980c39 to 346db77 Compare May 10, 2022 18:50
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 2 times, most recently from c259bac to b0489af Compare May 14, 2022 14:49
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 2 times, most recently from 8994795 to 00ef94c Compare May 21, 2022 08:07
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 2 times, most recently from bd72606 to f91970e Compare May 22, 2022 09:50
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch from f91970e to 23ceb2f Compare May 25, 2022 18:48
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 2 times, most recently from 288008a to 41b946e Compare May 28, 2022 09:24
@misl6 misl6 force-pushed the fix/build-platform-on-m1 branch 6 times, most recently from d4ddb31 to 5fdde17 Compare July 8, 2022 21:56
@misl6 misl6 changed the title [WIP] Added pythonforandroid.androidndk.AndroidNDK + some changes needed in order to support build on Apple Silicon macs. Added pythonforandroid.androidndk.AndroidNDK + some changes needed in order to support build on Apple Silicon macs. Jul 9, 2022
@misl6 misl6 requested a review from AndreMiras July 9, 2022 06:23
AndreMiras
AndreMiras previously approved these changes Jul 11, 2022
Copy link
Member

@AndreMiras AndreMiras left a comment

Choose a reason for hiding this comment

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

Absolutely awesome you got the Apple Silicon support fully working 👏
Thanks a mil!

pythonforandroid/androidndk.py Outdated Show resolved Hide resolved
tests/test_androidndk.py Outdated Show resolved Hide resolved
Co-authored-by: Andre Miras <AndreMiras@users.noreply.github.com>
Copy link
Member

@AndreMiras AndreMiras left a comment

Choose a reason for hiding this comment

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

Nice! I can't wait to have this merged

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