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

Include boost headers needed by rrc_text and rrc_textinput #43608

Closed
wants to merge 2 commits into from

Conversation

j-piasecki
Copy link
Collaborator

Summary:

Updates PreparePrefabHeadersTask to copy more headers from boost, specifically those required by rrc_text and rrc_textinput.

Changelog:

[ANDROID] [CHANGED] - Copy boost headers needed by rrc_text and rrc_textinput

Test Plan:

@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 Mar 22, 2024
@j-piasecki
Copy link
Collaborator Author

Looks like android doesn't build on main either: 0a12557

…ok/react/tasks/internal/PreparePrefabHeadersTask.kt
@facebook-github-bot
Copy link
Contributor

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

@cortinico
Copy link
Contributor

If you want this in RC5 (which I suppose - cc @tomekzaw also), please file a pick request on https://github.com/reactwg/react-native-releases/issues/new/choose

@facebook-github-bot
Copy link
Contributor

@cortinico merged this pull request in ad9f0b0.

huntie pushed a commit that referenced this pull request Mar 25, 2024
Summary:
Updates `PreparePrefabHeadersTask` to copy more headers from `boost`, specifically those required by `rrc_text` and `rrc_textinput`.

## Changelog:

[ANDROID] [CHANGED] - Copy boost headers needed by `rrc_text` and `rrc_textinput`

Pull Request resolved: #43608

Reviewed By: rshest

Differential Revision: D55241345

Pulled By: cortinico

fbshipit-source-id: e92164676ba78ee15b3678a55c9098b0c6214b69
This was referenced Jun 28, 2024
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.

3 participants