Skip to content

Commit 73d3429

Browse files
chrfalchmeta-codesync[bot]
authored andcommitted
fixed issue with xcconfig being overwritten (#54838)
Summary: To be able to change the target xcconfig from within `add_rn_core`, we need to make sure it is not overwritten by the line that sets the config. This commit fixes this by moving it after the line: `spec.pod_target_xcconfig = current_config` ## Changelog: [IOS] [FIXED] - Moved setting xcconfig to after `add_rn_core` pod function is called. Pull Request resolved: #54838 Test Plan: Run RNTester with precompiled binaries. Reviewed By: huntie Differential Revision: D88849730 Pulled By: cipolleschi fbshipit-source-id: 632e23f5a464ea779035ca3e090c0a3c9475a260
1 parent a22ab42 commit 73d3429

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

packages/react-native/scripts/cocoapods/new_architecture.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,9 +135,13 @@ def self.install_modules_dependencies(spec, new_arch_enabled, folly_version = He
135135

136136
depend_on_js_engine(spec)
137137
add_rn_third_party_dependencies(spec)
138-
add_rncore_dependency(spec)
139138

140139
spec.pod_target_xcconfig = current_config
140+
141+
# add_rncore_dependency must be called after setting pod_target_xcconfig
142+
# because it reads and modifies the xcconfig. If called before, its changes
143+
# would be overwritten by the assignment above.
144+
add_rncore_dependency(spec)
141145
end
142146

143147
def self.extract_react_native_version(react_native_path, file_manager: File, json_parser: JSON)

0 commit comments

Comments
 (0)