Skip to content

Expose jsctooling via prefab#46430

Closed
tomekzaw wants to merge 1 commit into
facebook:mainfrom
tomekzaw:@tomekzaw/expose-jsctooling-via-prefab
Closed

Expose jsctooling via prefab#46430
tomekzaw wants to merge 1 commit into
facebook:mainfrom
tomekzaw:@tomekzaw/expose-jsctooling-via-prefab

Conversation

@tomekzaw
Copy link
Copy Markdown
Contributor

@tomekzaw tomekzaw commented Sep 11, 2024

Summary:

This PR exposes jsctooling prefab that contains facebook::jsc::makeJSCRuntime used by Reanimated and other third-party libraries previously accessed via libjscexecutor.so.

Based on #46423.

Changelog:

[Android] [Changed] - Expose jsctooling via prefab

Test Plan:

Tested on Reanimated paper-example app built from source on RN 0.76.0-rc.0 with JSC enabled

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Software Mansion Partner: Software Mansion Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Sep 11, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cortinico has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 11, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cortinico merged this pull request in e91690d.

@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @tomekzaw in e91690d

When will my fix make it into a release? | How to file a pick request?

cipolleschi pushed a commit that referenced this pull request Sep 16, 2024
Summary:
This PR exposes `jsctooling` prefab that contains `facebook::jsc::makeJSCRuntime` used by Reanimated and other third-party libraries previously accessed via `libjscexecutor.so`.

Based on #46423.

## Changelog:

[Android] [Changed] - Expose jsctooling via prefab

Pull Request resolved: #46430

Test Plan: Tested on Reanimated paper-example app built from source on RN 0.76.0-rc.0 with JSC enabled

Reviewed By: cipolleschi

Differential Revision: D62492763

Pulled By: cortinico

fbshipit-source-id: 53b6c0d9bb88559c40b5b8796bf6a1513bd388d9
@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @tomekzaw in b0db081

When will my fix make it into a release? | How to file a pick request?

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Software Mansion Partner: Software Mansion Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants