Skip to content

Fix framework umbrella search paths (#58622) - #58622

Closed
coado wants to merge 1 commit into
react:mainfrom
coado:export-D120992289
Closed

coado wants to merge 1 commit into
react:mainfrom
coado:export-D120992289

Conversation

@coado

@coado coado commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

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

@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 21, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner labels Sep 21, 2026
@meta-codesync

meta-codesync Bot commented Sep 21, 2026

Copy link
Copy Markdown

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

@facebook-github-tools facebook-github-tools Bot added the p: Facebook Partner: Facebook label Sep 21, 2026
@meta-codesync meta-codesync Bot changed the title Fix Fabric umbrella search paths Fix Fabric umbrella search paths (#58622) Sep 21, 2026
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Differential Revision: D120992289
@coado
coado force-pushed the export-D120992289 branch 2 times, most recently from 8c035c5 to 539ba16 Compare September 21, 2026 19:00
coado added a commit to coado/react-native that referenced this pull request Sep 21, 2026
Summary:

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Reviewed By: cipolleschi

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Reviewed By: cipolleschi

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Reviewed By: cipolleschi

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

Source-built CocoaPods frameworks do not automatically expose the source directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils, causing dynamic-framework builds to fail while resolving headers such as <React/RendererCore.h>.

Add the five owning source directories to React-FabricComponents HEADER_SEARCH_PATHS when frameworks and source-built RNCore are used. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Reviewed By: cipolleschi

Differential Revision: D120992289
@coado
coado force-pushed the export-D120992289 branch 2 times, most recently from 3e60740 to 87547cf Compare September 22, 2026 09:26
coado added a commit to coado/react-native that referenced this pull request Sep 22, 2026
Summary:

Source-built CocoaPods frameworks do not automatically expose the directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils. Those headers are also compiled by consuming application targets, which do not inherit React-FabricComponents pod target settings.

Add the owning source directories to React-FabricComponents and propagate the four framework header roots to aggregate targets when RNCore is built from source. Keep prebuilt RNCore unchanged so its <React/...> headers continue to resolve from the prebuilt React.framework.

Changelog: [Internal]

Reviewed By: cipolleschi

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

Source-built CocoaPods frameworks do not automatically expose the directories behind stable <React/...> umbrella includes. React-FabricComponents exports modal and text headers that consume umbrellas owned by React-Fabric, React-debug, React-rendererdebug, and React-utils. Those headers are also compiled by consuming application targets, which do not inherit React-FabricComponents pod target settings.

Add the owning source directories to React-FabricComponents and propagate the four framework header roots to aggregate targets when RNCore is built from source. 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:

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
@meta-codesync meta-codesync Bot changed the title Fix Fabric umbrella search paths (#58622) Fix framework umbrella search paths (#58622) Sep 22, 2026
coado added a commit to coado/react-native that referenced this pull request Sep 22, 2026
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
@meta-codesync meta-codesync Bot closed this in 335ef07 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 335ef07.

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.

1 participant