Skip to content

WIP: Add an explicit NDK version to RNTester and ReactAndroid #29987

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.

Sign up for GitHub

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

Closed
gedeagas wants to merge 4 commits into from
Closed

WIP: Add an explicit NDK version to RNTester and ReactAndroid #29987

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
0d42f09
feat: add an explicit NDK version to gradle
gedeagas Sep 19, 2020
a2a1dd4
refactor: findNdkBuildFullPath ndkDir
gedeagas Sep 22, 2020
7c210d0
refactor: ndkDirectory
gedeagas Sep 23, 2020
819452b
refactor: put ndkVersion directly under Android
gedeagas Sep 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions ReactAndroid/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -380,7 +380,7 @@ task extractJNIFiles {

android {
compileSdkVersion 29

ndkVersion ndkVersion
compileOptions {
sourceCompatibility(JavaVersion.VERSION_1_8)
targetCompatibility(JavaVersion.VERSION_1_8)
Expand All @@ -391,7 +391,7 @@ android {
targetSdkVersion(28)
versionCode(1)
versionName("1.0")

ndkVersion "20.1.5948944"
consumerProguardFiles("proguard-rules.pro")

buildConfigField("boolean", "IS_INTERNAL_BUILD", "false")
Expand Down
3 changes: 2 additions & 1 deletion packages/rn-tester/android/app/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ def useIntlJsc = false

android {
compileSdkVersion 29

ndkVersion ndkVersion
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
Expand All @@ -142,6 +142,7 @@ android {
defaultConfig {
applicationId "com.facebook.react.uiapp"
minSdkVersion 19
ndkVersion "20.1.5948944"
targetSdkVersion 29
versionCode 1
versionName "1.0"
Expand Down