fix: Fix missing includes

margelo · May 17, 2024 · e4cd40a · e4cd40a
1 parent 25ce228
commit e4cd40a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/package/android/CMakeLists.txt b/package/android/CMakeLists.txt
@@ -33,11 +33,16 @@ target_include_directories(
 
 # Add Hermes (from com.facebook.react:hermes-android)
 find_package(hermes-engine REQUIRED CONFIG)
+# Add React core libraries (JSI, ...)
+find_package(ReactAndroid REQUIRED CONFIG)
+# Add fbjni (from React core)
+find_package(fbjni REQUIRED CONFIG)
 
 # Link everything together
 target_link_libraries(
         ${PACKAGE_NAME}
         ReactAndroid::jsi               # <-- JSI core
+        fbjni::fbjni                    # <-- fbjni
         react_codegen_RNWorkletsSpec    # <-- Generated Specs from CodeGen
         hermes-engine::libhermes        # <-- Hermes Prefab
 )

diff --git a/package/android/build.gradle b/package/android/build.gradle
@@ -32,6 +32,8 @@ android {
 
     externalNativeBuild {
       cmake {
+        arguments "-DANDROID_STL=c++_shared",
+          "-DANDROID_TOOLCHAIN=clang"
         cppFlags "-O2 -frtti -fexceptions -Wall -fstack-protector-all"
       }
     }

diff --git a/package/cpp/NativeWorkletsModule.h b/package/cpp/NativeWorkletsModule.h
@@ -26,7 +26,7 @@ class NativeWorkletsModule : public NativeWorkletsCxxSpec<NativeWorkletsModule>
   ~NativeWorkletsModule();
 
   jsi::Object createWorkletsApi(jsi::Runtime& runtime);
-  
+
 private:
   std::shared_ptr<CallInvoker> _jsCallInvoker;
 };