From 48b2b7914bec1443bcfc5587cda87adcf087f493 Mon Sep 17 00:00:00 2001 From: Thibault Malbranche Date: Thu, 20 May 2021 15:21:53 -0700 Subject: [PATCH] fix(hermes): fixed hermes build on iOS (#31559) Summary: While testing 0.65, we noticed issues with hermes on iOS in the template projects These changes create a subspec to the react-core pod so that it can access hermes header correctly. ## Changelog Not applicable Pull Request resolved: https://github.com/facebook/react-native/pull/31559 Test Plan: I've ran e2e manual test. Tested RNTester manually also. Then tested a project inited with hermes and the default template Reviewed By: mhorowitz Differential Revision: D28564642 Pulled By: Huxpro fbshipit-source-id: cfcb3363254f62a0e514ec99159b32f841ee4463 --- React-Core.podspec | 2 +- React/CxxBridge/RCTCxxBridge.mm | 4 ++-- packages/rn-tester/RNTester/AppDelegate.mm | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/React-Core.podspec b/React-Core.podspec index a6f3c6d561a..cfe68212321 100644 --- a/React-Core.podspec +++ b/React-Core.podspec @@ -48,7 +48,7 @@ Pod::Spec.new do |s| s.header_dir = "React" s.framework = "JavaScriptCore" s.library = "stdc++" - s.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/boost-for-react-native\" \"$(PODS_ROOT)/DoubleConversion\" \"$(PODS_ROOT)/RCT-Folly\"", "DEFINES_MODULE" => "YES" } + s.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/boost-for-react-native\" \"$(PODS_ROOT)/DoubleConversion\" \"$(PODS_ROOT)/RCT-Folly\" \"${PODS_ROOT}/Headers/Public/React-hermes\" \"${PODS_ROOT}/Headers/Public/hermes-engine\"", "DEFINES_MODULE" => "YES" } s.user_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_ROOT)/Headers/Private/React-Core\""} s.default_subspec = "Default" diff --git a/React/CxxBridge/RCTCxxBridge.mm b/React/CxxBridge/RCTCxxBridge.mm index be1798f1939..bc890fa5527 100644 --- a/React/CxxBridge/RCTCxxBridge.mm +++ b/React/CxxBridge/RCTCxxBridge.mm @@ -40,7 +40,7 @@ #import #ifndef RCT_USE_HERMES -#if __has_include() +#if __has_include() #define RCT_USE_HERMES 1 #else #define RCT_USE_HERMES 0 @@ -48,7 +48,7 @@ #endif #if RCT_USE_HERMES -#import +#import #else #import "JSCExecutorFactory.h" #endif diff --git a/packages/rn-tester/RNTester/AppDelegate.mm b/packages/rn-tester/RNTester/AppDelegate.mm index c54f94fae65..dc5d65c1cfa 100644 --- a/packages/rn-tester/RNTester/AppDelegate.mm +++ b/packages/rn-tester/RNTester/AppDelegate.mm @@ -8,7 +8,7 @@ #import "AppDelegate.h" #ifndef RCT_USE_HERMES -#if __has_include() +#if __has_include() #define RCT_USE_HERMES 1 #else #define RCT_USE_HERMES 0