Skip to content

Use public umbrellas in text headers (#58571) - #58571

Closed
coado wants to merge 2 commits into
react:mainfrom
coado:export-D120529435
Closed

coado wants to merge 2 commits into
react:mainfrom
coado:export-D120529435

Conversation

@coado

@coado coado commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435

@meta-cla meta-cla 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 Sep 17, 2026
@meta-codesync

meta-codesync Bot commented Sep 17, 2026

Copy link
Copy Markdown

@coado has exported this pull request. If you are a Meta employee, you can view the originating Diff in D120529435.

@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Sep 17, 2026
coado added a commit to coado/react-native that referenced this pull request Sep 17, 2026
Summary:
Pull Request resolved: react#58571

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Differential Revision: D120529435
@meta-codesync meta-codesync Bot changed the title Use public umbrellas in text headers Use public umbrellas in text headers (#58571) Sep 17, 2026
coado added a commit to coado/react-native that referenced this pull request Sep 18, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 18, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Differential Revision: D120529435

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:
Pull Request resolved: react#58571

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:
Pull Request resolved: react#58571

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 22, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
Summary:

Source-built CocoaPods frameworks do not automatically expose the directories behind stable <React/...> umbrella includes. Exported and generated headers consume umbrellas owned by sibling frameworks, while consuming application and pod targets do not automatically inherit the header roots for those frameworks.

Add source-only aggregate paths for the umbrella frameworks used by FabricComponents and React-perflogger, configure the React-timing dependency of React-perflogger with its framework header root, and expose React-bridging to generated module targets. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Reviewed By: cipolleschi

Differential Revision: D120992289
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
coado added a commit to coado/react-native that referenced this pull request Sep 22, 2026
Summary:

Route public C++ dependencies through their supported umbrella entry points so framework consumers can use the text module with strict API enforcement enabled.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D120529435
@meta-codesync meta-codesync Bot closed this in d381a99 Sep 22, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 22, 2026
@meta-codesync

meta-codesync Bot commented Sep 22, 2026

Copy link
Copy Markdown

This pull request has been merged in d381a99.

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. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants