-
Notifications
You must be signed in to change notification settings - Fork 69
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
Build fails on Android with Reanimated 3.14.0 #208
Comments
Investigated a bit. Build fails because of this https://github.com/software-mansion/react-native-reanimated/blob/183add4d329660f6064e4195a4ef6559da097c9a/packages/react-native-reanimated/android/CMakeLists.txt#L131 Now application will compile, but at any reanimated encounter will crash.
|
Made minimal app that shows the bug https://github.com/reinismu/ReanimatedV8Deadlock I'm not too sure but it seems that reanimated calls v8 engine from js thread in a way that deadlocks |
…error Using v8 javascript engine was causing react-native-reanimated gradle build to fail: software-mansion#3570 Changing the `V8_SO_DIR` according to the recommendation from @reinismu fixes this: Kudo/react-native-v8#208 (comment)
…xt (#6603) Using v8 javascript engine was causing react-native-reanimated gradle build to fail: #3570 Changing the `V8_SO_DIR` according to the recommendation from @reinismu fixes this: Kudo/react-native-v8#208 (comment) <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please follow the template so that the reviewers can easily understand what the code changes affect. --> ## Summary Fixes #3570 ## Test plan I'm not sure how the current CI is passing (#3570 (comment)), but that would be the way to test it. --------- Co-authored-by: Tomasz Żelawski <40713406+tjzel@users.noreply.github.com>
…xt (#6603) Using v8 javascript engine was causing react-native-reanimated gradle build to fail: #3570 Changing the `V8_SO_DIR` according to the recommendation from @reinismu fixes this: Kudo/react-native-v8#208 (comment) <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please follow the template so that the reviewers can easily understand what the code changes affect. --> ## Summary Fixes #3570 ## Test plan I'm not sure how the current CI is passing (#3570 (comment)), but that would be the way to test it. --------- Co-authored-by: Tomasz Żelawski <40713406+tjzel@users.noreply.github.com>
…xt (#6603) Using v8 javascript engine was causing react-native-reanimated gradle build to fail: #3570 Changing the `V8_SO_DIR` according to the recommendation from @reinismu fixes this: Kudo/react-native-v8#208 (comment) <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please follow the template so that the reviewers can easily understand what the code changes affect. --> ## Summary Fixes #3570 ## Test plan I'm not sure how the current CI is passing (#3570 (comment)), but that would be the way to test it. --------- Co-authored-by: Tomasz Żelawski <40713406+tjzel@users.noreply.github.com>
…xt (#6603) Using v8 javascript engine was causing react-native-reanimated gradle build to fail: #3570 Changing the `V8_SO_DIR` according to the recommendation from @reinismu fixes this: Kudo/react-native-v8#208 (comment) <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please follow the template so that the reviewers can easily understand what the code changes affect. --> ## Summary Fixes #3570 ## Test plan I'm not sure how the current CI is passing (#3570 (comment)), but that would be the way to test it. --------- Co-authored-by: Tomasz Żelawski <40713406+tjzel@users.noreply.github.com>
…xt (#6603) Using v8 javascript engine was causing react-native-reanimated gradle build to fail: #3570 Changing the `V8_SO_DIR` according to the recommendation from @reinismu fixes this: Kudo/react-native-v8#208 (comment) <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please follow the template so that the reviewers can easily understand what the code changes affect. --> ## Summary Fixes #3570 ## Test plan I'm not sure how the current CI is passing (#3570 (comment)), but that would be the way to test it. --------- Co-authored-by: Tomasz Żelawski <40713406+tjzel@users.noreply.github.com>
Hello!
Build fails if using Reanimated library.
I tried to build the app without these packages, then it builds but the app has quite a laggy feeling - at this point, it's hard to tell if it's because of an old Android phone (for testing purposes) or the problem is somewhere else.
Packages:
Here is the error that I got:
I will appreciate for looking into this issue.
The text was updated successfully, but these errors were encountered: