-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update android ndk version to latest LTS(r23c) #2832
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon. Please see the contribution instructions for more information. |
Let's give a try, @bors r+ |
📌 Commit 501ce18 has been approved by |
Update android ndk version to latest LTS(r23c) The latest LTS ndk version: https://developer.android.com/ndk/downloads And the latest stable API version of Android is Android 12 (api=31)
💔 Test failed - checks-actions |
Seems they removed the arch suffix from the URL. |
The URL should be correct now. |
@bors r+ |
📌 Commit 00f6fd9 has been approved by |
Update android ndk version to latest LTS(r23c) The latest LTS ndk version: https://developer.android.com/ndk/downloads And the latest stable API version of Android is Android 12 (api=31)
💔 Test failed - checks-actions |
ndk23 remove *-ar, we should use llvm-ar instead: bbqsrc/cargo-ndk#38 |
on ndk23, there are two changes:
|
@bors r+ |
📌 Commit b205ce8 has been approved by |
I checked it on https://www.shellcheck.net/, should be fine now |
@bors r+ |
📌 Commit 4191221 has been approved by |
Update android ndk version to latest LTS(r23c) The latest LTS ndk version: https://developer.android.com/ndk/downloads And the latest stable API version of Android is Android 12 (api=31)
💔 Test failed - checks-actions |
Sorry....the ndk directory is incorrect :) |
@bors r+ |
📌 Commit 4f172fb has been approved by |
Update android ndk version to latest LTS(r23c) The latest LTS ndk version: https://developer.android.com/ndk/downloads And the latest stable API version of Android is Android 12 (api=31)
💔 Test failed - checks-actions |
Some items aren't matched between C and Rust on arm-linux-androideabi:
Is this expected? |
Could you tell me how do i check this, any C header or the test code would be helpful. |
You could compare the actual definitions on C headers with the declarations on Rust. The errors say that they aren't matched. |
Closing as inactive, feel free to reopen/resubmit if you're still interested in this change. Thank you! |
The latest LTS ndk version: https://developer.android.com/ndk/downloads
And the latest stable API version of Android is Android 12 (api=31)