From 3f50f57b3da0a3982645d274cc9bb0cef1779bda Mon Sep 17 00:00:00 2001 From: Ramanpreet Nara Date: Fri, 1 Nov 2019 19:32:48 -0700 Subject: [PATCH] Back out "Fixes Fabric related podspec" Summary: Original commit changeset: 4b18a931a2e4 The original PR wherein these changes were introduced was problematic. When you expose headers that use C++ from the Yoga podspec, they get automatically imported in the Yoga umbrella file. This causes compilation errors, most likely because it's possible for non-c++ files to import these headers (i.e: RCTConvert.m). I didn't dig into this too much, but since Fabric still doesn't fully compile in OSS, I think it's reasonable to revert this PR for now. cc Kevin Gozali. Changelog: [iOS][Fixed] - Undo Fabric-related podspec change Reviewed By: fkgozali Differential Revision: D18284536 fbshipit-source-id: a90454b945af0235424dc56408400cd35efd4e7a --- React-Core.podspec | 3 ++- ReactCommon/React-Fabric.podspec | 18 ------------------ ReactCommon/yoga/Yoga.podspec | 7 ++----- 3 files changed, 4 insertions(+), 24 deletions(-) diff --git a/React-Core.podspec b/React-Core.podspec index 733e4c16a54..ccb27d8568d 100644 --- a/React-Core.podspec +++ b/React-Core.podspec @@ -64,7 +64,8 @@ Pod::Spec.new do |s| "React/Views/RCTPicker*", "React/Views/RCTRefreshControl*", "React/Views/RCTSlider*", - "React/Views/RCTSwitch*" + "React/Views/RCTSwitch*", + ss.private_header_files = "React/Cxx*/*.h" end s.subspec "DevSupport" do |ss| diff --git a/ReactCommon/React-Fabric.podspec b/ReactCommon/React-Fabric.podspec index 418c1d7866d..3c97712f950 100644 --- a/ReactCommon/React-Fabric.podspec +++ b/ReactCommon/React-Fabric.podspec @@ -159,24 +159,6 @@ Pod::Spec.new do |s| sss.header_dir = "react/components/view" sss.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/Folly\"" } end - - ss.subspec "safeareaview" do |sss| - sss.dependency folly_dep_name, folly_version - sss.compiler_flags = folly_compiler_flags - sss.source_files = "fabric/components/safeareaview/**/*.{m,mm,cpp,h}" - sss.exclude_files = "**/tests/*" - sss.header_dir = "react/components/safeareaview" - sss.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/Folly\"" } - end - - ss.subspec "legacyviewmanagerinterop" do |sss| - sss.dependency folly_dep_name, folly_version - sss.compiler_flags = folly_compiler_flags - sss.source_files = "fabric/components/legacyviewmanagerinterop/**/*.{m,mm,cpp,h}" - sss.exclude_files = "**/tests/*" - sss.header_dir = "react/components/legacyviewmanagerinterop" - sss.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/Folly\"" } - end end s.subspec "debug" do |ss| diff --git a/ReactCommon/yoga/Yoga.podspec b/ReactCommon/yoga/Yoga.podspec index e28757eb2ac..82051b0e8ee 100644 --- a/ReactCommon/yoga/Yoga.podspec +++ b/ReactCommon/yoga/Yoga.podspec @@ -38,13 +38,10 @@ Pod::Spec.new do |spec| '-fexceptions', '-Wall', '-Werror', + '-std=c++1y', '-fPIC' ] - spec.pod_target_xcconfig = { - "CLANG_CXX_LANGUAGE_STANDARD" => "c++14" - } - # Pinning to the same version as React.podspec. spec.platforms = { :ios => "9.0", :tvos => "9.2" } @@ -54,7 +51,7 @@ Pod::Spec.new do |spec| source_files = File.join('ReactCommon/yoga', source_files) if ENV['INSTALL_YOGA_WITHOUT_PATH_OPTION'] spec.source_files = source_files - header_files = 'yoga/{Yoga,YGEnums,YGMacros,YGValue,YGNode,YGStyle}.h' + header_files = 'yoga/{Yoga,YGEnums,YGMacros,YGValue}.h' header_files = File.join('ReactCommon/yoga', header_files) if ENV['INSTALL_YOGA_WITHOUT_PATH_OPTION'] spec.public_header_files = header_files end