From f4dbd15f0bb089e48db9e7aafb4d6bcd73684d61 Mon Sep 17 00:00:00 2001 From: Riccardo Cipolleschi Date: Thu, 25 Jan 2024 02:02:40 -0800 Subject: [PATCH] Fix RCTFabric podspec (#42639) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/42639 When I reverted part of the deprecation of `RCT_NEW_ARCH_ENABLED`, I forget a little bit which was breaking RCTFabric podspec. This diff fixes that. ## Changelog: [Internal] - Bring back `RCT_NEW_ARCH_ENABLED` to Fabric to make the `RCTThirdPartyFabricComponentsProvider` work again. Reviewed By: cortinico Differential Revision: D53048270 fbshipit-source-id: d21e833c10b332fb70147cc65b690f88016655e6 --- packages/react-native/React/React-RCTFabric.podspec | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/react-native/React/React-RCTFabric.podspec b/packages/react-native/React/React-RCTFabric.podspec index 56ec36bede8626..bed68271fd4acc 100644 --- a/packages/react-native/React/React-RCTFabric.podspec +++ b/packages/react-native/React/React-RCTFabric.podspec @@ -20,6 +20,7 @@ folly_config = get_folly_config() folly_compiler_flags = folly_config[:compiler_flags] folly_version = folly_config[:version] boost_compiler_flags = '-Wno-documentation' +new_arch_flags = ENV['RCT_NEW_ARCH_ENABLED'] == '1' ? ' -DRCT_NEW_ARCH_ENABLED=1' : '' header_search_paths = [ "\"$(PODS_TARGET_SRCROOT)/ReactCommon\"", @@ -52,13 +53,13 @@ Pod::Spec.new do |s| s.source_files = "Fabric/**/*.{c,h,m,mm,S,cpp}" s.exclude_files = "**/tests/*", "**/android/*", - s.compiler_flags = folly_compiler_flags + ' ' + boost_compiler_flags + s.compiler_flags = folly_compiler_flags + ' ' + boost_compiler_flags + new_arch_flags s.header_dir = header_dir s.module_name = module_name s.framework = ["JavaScriptCore", "MobileCoreServices"] s.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => header_search_paths, - "OTHER_CFLAGS" => "$(inherited) " + folly_compiler_flags, + "OTHER_CFLAGS" => "$(inherited) " + folly_compiler_flags + new_arch_flags, "CLANG_CXX_LANGUAGE_STANDARD" => "c++20" }.merge!(ENV['USE_FRAMEWORKS'] != nil ? { "PUBLIC_HEADERS_FOLDER_PATH" => "#{module_name}.framework/Headers/#{header_dir}"