Skip to content

Fix OSS native Android build - #33167

Closed
ShikaSD wants to merge 2 commits into
react:mainfrom
ShikaSD:export-D34419236
Closed

ShikaSD wants to merge 2 commits into
react:mainfrom
ShikaSD:export-D34419236

Conversation

@ShikaSD

@ShikaSD ShikaSD commented Feb 23, 2022

Copy link
Copy Markdown
Contributor

Summary:
The build is failing with:

error: undefined reference to 'folly::SharedMutexImpl<false, void, std::__ndk1::atomic, folly::SharedMutexPolicyDefault>::unlock_shared()'

Indicating missing link to folly_futures.

Changelog: [Internal]

Differential Revision: D34419236

Andrei Shikov added 2 commits February 23, 2022 07:33
Differential Revision: D34417682

fbshipit-source-id: 330166739f55f95c1c58f0a7782499c4912efb84
Summary:
The build is failing with:

```
error: undefined reference to 'folly::SharedMutexImpl<false, void, std::__ndk1::atomic, folly::SharedMutexPolicyDefault>::unlock_shared()'
```

Indicating missing link to `folly_futures`.

Changelog: [Internal]

Differential Revision: D34419236

fbshipit-source-id: 86c78933093afa749dcb3be54bae72538eb273e6
@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. fb-exported labels Feb 23, 2022
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D34419236

@analysis-bot

Copy link
Copy Markdown
Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: b1a7793
Branch: main

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @ShikaSD in 78f6cdf.

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 Feb 23, 2022
Saadnajmi pushed a commit to Saadnajmi/react-native-macos that referenced this pull request Jan 15, 2023
Summary:
Pull Request resolved: react#33167

The build is failing with:

```
error: undefined reference to 'folly::SharedMutexImpl<false, void, std::__ndk1::atomic, folly::SharedMutexPolicyDefault>::unlock_shared()'
```

Indicating missing link to `folly_futures`.

Changelog: [Internal]

Reviewed By: cortinico, GijsWeterings

Differential Revision: D34419236

fbshipit-source-id: ca777e0c3a13c2f23791a94b13de18911c93edd5
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. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants