Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Fix Frame Processors not building because of hasWorklets flag #2137

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

mrousavy
Copy link
Owner

@mrousavy mrousavy commented Nov 7, 2023

What

Apparently passing a boolean (true or false) from Groovy (build.gradle) to Cmake (CMakeLists.txt) turns it into a string ("true" or "false").

Instead, we now pass the value ON and OFF to CMake, which is their understanding of a boolean. God knows why.

Changes

Tested on

Related issues

Copy link

vercel bot commented Nov 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
react-native-vision-camera ✅ Ready (Inspect) Visit Preview Nov 7, 2023 11:59am

@mrousavy mrousavy merged commit 24ddca3 into main Nov 9, 2023
@mrousavy mrousavy deleted the fix/cmake-worklets branch November 9, 2023 10:56
isaaccolson pushed a commit to isaaccolson/deliveries-mobile that referenced this pull request Oct 30, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 ld: error: undefined symbol: typeinfo for RNWorklet::JsiHostObject
1 participant