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

bump NDK to latest LTS or 25 #177

Closed
wants to merge 1 commit into from

Conversation

dulmandakh
Copy link
Contributor

No description provided.

@@ -26,7 +26,7 @@ ENV DEBIAN_FRONTEND=noninteractive
ARG SDK_VERSION=commandlinetools-linux-8512546_latest.zip
ARG ANDROID_BUILD_VERSION=31
ARG ANDROID_TOOLS_VERSION=31.0.0
ARG NDK_VERSION=21.4.7075529
ARG NDK_VERSION=25.0.8775105
Copy link
Member

Choose a reason for hiding this comment

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

This should be aligned with a change on React Native's end (especially the template)

Copy link
Member

Choose a reason for hiding this comment

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

CI also looks failed:

 /opt/android/ndk/25.0.8775105/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64 doesn't exist.

Copy link
Contributor

@leotm leotm Aug 4, 2022

Choose a reason for hiding this comment

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

it'll just be updating android/build.gradle right

NDK 25 been looking pretty solid for me past few months

think only extra thing had to do was bump CMake from 3.22.1 -> 3.23.1 on Ubuntu 20.04

Copy link
Member

Choose a reason for hiding this comment

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

Yup I think we should be fine by updating everything to NDK 25 for RN 0.71

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the comments. I'll try to finish the PR very soon.

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.

None yet

4 participants