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

fix: improve ndk home compatibility #194

Merged
merged 2 commits into from
Feb 20, 2023

Conversation

gengjiawen
Copy link
Member

The preferred environment variable for specifying the location of the Android NDK is ANDROID_NDK_HOME. NDK_HOME is an older variable that was used in earlier versions of the Android NDK, but it is no longer recommended for use.

@cortinico
Copy link
Member

I'm unsure if we need this as NDK_HOME is needed only for Buck compatibility 🤔

@gengjiawen
Copy link
Member Author

I'm unsure if we need this as NDK_HOME is needed only for Buck compatibility 🤔

This in for regular Android NDK developing app without using buck.

@cortinico
Copy link
Member

Yup what I'm saying is that this is not correct:

ENV ANDROID_NDK_HOME=${ANDROID_HOME}/ndk/$NDK_VERSION_BUCK

We need NDK_HOME as we have two NDKs. Buck understands only up to NDK 21 so we set NDK_HOME for it. For "regular Android development" the NDK is another one (the 23). Setting ANDROID_NDK_HOME can have unintended side effects as we want Gradle to pickup NDK 23

@gengjiawen gengjiawen merged commit 2bd98f9 into react-native-community:master Feb 20, 2023
@gengjiawen gengjiawen deleted the fix/ndk branch February 20, 2023 14:16
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