Skip to content

Conversation

dulmandakh
Copy link
Contributor

Summary

This PR fixes build failure caused by duplicate libc++_shared.so, because dependencies (like Flipper) might include one in addition to RN.

Changelog

[Android] [Changed] - fix build failure due to duplicate libc++_shared.so

Test Plan

Create a project from master, then build and run. Build will fail without the patch, and succeed with the patch.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 5, 2019
@dulmandakh dulmandakh requested a review from mdvacca December 5, 2019 04:28
@facebook-github-bot facebook-github-bot added the Contributor A React Native contributor. label Dec 5, 2019
@dulmandakh dulmandakh requested a review from grabbou December 5, 2019 04:32
Copy link
Contributor

@mdvacca mdvacca left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mdvacca is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @dulmandakh in 2fd5088.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Dec 5, 2019
@dulmandakh dulmandakh deleted the android-pickfirst branch December 5, 2019 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API: Share CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. Merged This PR has been merged. Platform: Android Android applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants