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: Fix android build script for support [email protected](without new architecture) #3325

Closed
wants to merge 4 commits into from
Closed

fix: Fix android build script for support [email protected](without new architecture) #3325

wants to merge 4 commits into from

Conversation

YangJonghun
Copy link

@YangJonghun YangJonghun commented Jun 26, 2022

Description

this PR written for RNReanimated v2 support React Native 0.69 (without new architecture) so, should be include v2.x.x

as I know v3 is currently release candidate but, this PR created for some users use reanimated v1 api with RN v0.69

(I don't know which base branch is correct, please make any branch for release 2.8.x or 2.9.0)

fixes #3317
fixes #3320
fixes #3323
fixes #3310

Changes

Test code and steps to reproduce

Checklist

  • Included code example that can be used to test this change
  • Updated TS types
  • Added TS types tests
  • Added unit / integration tests
  • Updated documentation
  • Ensured that CI passes

@jsreynaud
Copy link

On file CMakeLists.txt:194 you should change ${FOLLY_JSON_LIB} to ${FOLLY_LIB} also

@YangJonghun
Copy link
Author

YangJonghun commented Jun 28, 2022

On file CMakeLists.txt:194 you should change ${FOLLY_JSON_LIB} to ${FOLLY_LIB} also

@jsreynaud
My bad. I missed case when hermes disabled, thank you for your comment

@jsreynaud
Copy link

@YangJonghun You should also add "**/libfolly_runtime.so", around android/build.gradle:209 to avoid double loading...

@piaskowyk piaskowyk mentioned this pull request Jun 30, 2022
@piaskowyk
Copy link
Member

Hey, thanks for your PR! I prepared another PR based on your code, available here - #3344 I will merge it to branch Reanimated2, because the current main contains Reaniamted3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment