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 Example app with JSC on Android #3572

Merged
merged 4 commits into from
Sep 29, 2022
Merged

Conversation

Kwasow
Copy link
Contributor

@Kwasow Kwasow commented Sep 15, 2022

Description

The Example app crashes on Android if compiled with JSC instead of Hermes. Error message:

2022-09-15 12:34:45.701 15509-15509/com.swmansion.reanimated.example W/System.err: java.lang.UnsatisfiedLinkError: dlopen failed: library "libjsc.so" not found: needed by /data/app/~~YZvjm8gAMg_2TYxNnYOCDA==/com.swmansion.reanimated.example--9SU_U_zrfiGBnf1eSq-kA==/lib/arm64/libjscexecutor.so in namespace classloader-namespace

Cause of crash

The build.gradle files in the Example app were outdated, which caused multiple issues. Some of the changes are:

  • Added missing 64-bit architectures in some places
  • Fixed enabled build abi resolution
  • Fixed incorrect maven repositories definition in top-level build.gradle

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

@billouboq
Copy link

Can we use reanimated 2 without Hermes without this patch, or is this allowing it ?

@Kwasow
Copy link
Contributor Author

Kwasow commented Sep 21, 2022

Hi @billouboq, we have just discovered this issue and haven't had time to investigate it so at this time I cannot answer your question, but feel free to try running Reanimated 2 without Hermes and let us know if it works!

@Kwasow
Copy link
Contributor Author

Kwasow commented Sep 22, 2022

I've investigated the issue @billouboq, and it seems that it is only an issue with our Example app, so you should be fine using other runtimes than Hermes with Reanimated.

@Kwasow Kwasow marked this pull request as ready for review September 22, 2022 08:45
@billouboq
Copy link

Thanks you very much, will try it too !

@tomekzaw

This comment was marked as outdated.

Copy link
Member

@tomekzaw tomekzaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've cloned the repository again and now it works ✅

Thanks a lot for fixing this! 👏

@piaskowyk piaskowyk merged commit 1fcf2e2 into main Sep 29, 2022
@piaskowyk piaskowyk deleted the @kwasow/fixExampleAppJSC branch September 29, 2022 10:38
piaskowyk pushed a commit that referenced this pull request Oct 6, 2022
## Description

The Example app crashes on Android if compiled with JSC instead of
Hermes. Error message:

```
2022-09-15 12:34:45.701 15509-15509/com.swmansion.reanimated.example W/System.err: java.lang.UnsatisfiedLinkError: dlopen failed: library "libjsc.so" not found: needed by /data/app/~~YZvjm8gAMg_2TYxNnYOCDA==/com.swmansion.reanimated.example--9SU_U_zrfiGBnf1eSq-kA==/lib/arm64/libjscexecutor.so in namespace classloader-namespace
```

## Cause of crash

The `build.gradle` files in the Example app were outdated, which caused
multiple issues. Some of the changes are:

- Added missing 64-bit architectures in some places
- Fixed enabled build abi resolution
- Fixed incorrect maven repositories definition in top-level
`build.gradle`

## 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
- [x] Ensured that CI passes
fluiddot pushed a commit to wordpress-mobile/react-native-reanimated that referenced this pull request Jun 5, 2023
## Description

The Example app crashes on Android if compiled with JSC instead of
Hermes. Error message:

```
2022-09-15 12:34:45.701 15509-15509/com.swmansion.reanimated.example W/System.err: java.lang.UnsatisfiedLinkError: dlopen failed: library "libjsc.so" not found: needed by /data/app/~~YZvjm8gAMg_2TYxNnYOCDA==/com.swmansion.reanimated.example--9SU_U_zrfiGBnf1eSq-kA==/lib/arm64/libjscexecutor.so in namespace classloader-namespace
```

## Cause of crash

The `build.gradle` files in the Example app were outdated, which caused
multiple issues. Some of the changes are:

- Added missing 64-bit architectures in some places
- Fixed enabled build abi resolution
- Fixed incorrect maven repositories definition in top-level
`build.gradle`

## 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
- [x] Ensured that CI passes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants