-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Open
Labels
Area: FabricSupport Facebook FabricSupport Facebook FabricIntegration Follow-upNew ArchitectureBroad category for issues that apply to the RN "new" architecture of Turbo Modules + FabricBroad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabricbug
Milestone
Description
Upstream changed the TRACE_TAG_REACT_APPS macro to TRACE_TAG_REACT.
However, we already weren't able to support the TRACE_TAG_REACT_APPS macro. See #14699 issue because it relies on something from Meta's internal build system.
So creating copy of this file and changing the macro back to TRACE_TAG_REACT_CXX_BRIDGE and added a comment to them "// [Windows]".
Link to the integration where the error originated
#14814
Link to commit
facebook/react-native@d0ab697#diff-5faa26e4dbd6748666d12a3df40102e564f2e47cd252fd4f2b3eb79107a9d614
Forked files
vnext\ReactCommon\TEMP_UntilReactCommonUpdate\cxxreact\TraceSection.h
vnext\ReactCommon\TEMP_UntilReactCommonUpdate\cxxreact\CxxNativeModule.cpp
Metadata
Metadata
Assignees
Labels
Area: FabricSupport Facebook FabricSupport Facebook FabricIntegration Follow-upNew ArchitectureBroad category for issues that apply to the RN "new" architecture of Turbo Modules + FabricBroad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabricbug