Skip to content

Commit

Permalink
fix: native build config for Android (#639)
Browse files Browse the repository at this point in the history
* fix: cmake config for android

* fix: remove buildTools requirement from build.gradle

* chore: add changeset
  • Loading branch information
jbroma committed Jun 7, 2024
1 parent 2988683 commit 13db9ed
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 3 deletions.
5 changes: 5 additions & 0 deletions .changeset/proud-shrimps-share.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@callstack/repack": patch
---

Fix android native module configuration
4 changes: 3 additions & 1 deletion packages/repack/android/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@ set(CMAKE_CXX_STANDARD 17)
add_library(
${PACKAGE_NAME}
SHARED
src/main/cpp
src/main/cpp/OnLoad.cpp
src/main/cpp/NativeScriptLoader.cpp
)

# Configure C++ 17
Expand All @@ -29,6 +30,7 @@ find_package(ReactAndroid REQUIRED CONFIG)
target_include_directories(
${PACKAGE_NAME}
PRIVATE
"src/main/cpp"
"${REACT_NATIVE_DIR}/ReactCommon"
"${REACT_NATIVE_DIR}/ReactCommon/callinvoker"
"${REACT_NATIVE_DIR}/ReactAndroid/src/main/jni/react/turbomodule"
Expand Down
1 change: 0 additions & 1 deletion packages/repack/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ android {
}

compileSdkVersion getExtOrIntegerDefault('compileSdkVersion')
buildToolsVersion getExtOrDefault('buildToolsVersion')

if (rootProject.hasProperty("ndkPath")) {
ndkPath rootProject.ext.ndkPath
Expand Down
1 change: 0 additions & 1 deletion packages/repack/android/gradle.properties
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
RePack_kotlinVersion=1.7.0
RePack_compileSdkVersion=29
RePack_buildToolsVersion=29.0.2
RePack_targetSdkVersion=29
RePack_minSdkVersion=21

0 comments on commit 13db9ed

Please sign in to comment.