Skip to content

Commit

Permalink
Remove libstdc++ dependency (#32247)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #32247

I don't think we need both libc++ and libstdc++.

allow-large-files

Changelog: [Internal]

Reviewed By: fkgozali

Differential Revision: D30950943

fbshipit-source-id: d0669815ff59c3e9ac45954a4a11930d1bc3959f
  • Loading branch information
neildhar authored and facebook-github-bot committed Oct 8, 2021
1 parent c99a021 commit aae9355
Show file tree
Hide file tree
Showing 8 changed files with 7 additions and 14 deletions.
1 change: 0 additions & 1 deletion React-Core.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ Pod::Spec.new do |s|
s.compiler_flags = folly_compiler_flags + ' ' + boost_compiler_flags
s.header_dir = "React"
s.framework = "JavaScriptCore"
s.library = "stdc++"
s.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/boost\" \"$(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"
Expand Down
1 change: 0 additions & 1 deletion React/React-RCTFabric.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ Pod::Spec.new do |s|
s.compiler_flags = folly_compiler_flags + ' ' + boost_compiler_flags
s.header_dir = "React"
s.framework = "JavaScriptCore"
s.library = "stdc++"
s.pod_target_xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)/ReactCommon\" \"$(PODS_ROOT)/boost\" \"$(PODS_ROOT)/DoubleConversion\" \"$(PODS_ROOT)/RCT-Folly\" \"$(PODS_ROOT)/Headers/Private/React-Core\"" }
s.xcconfig = { "HEADER_SEARCH_PATHS" => "\"$(PODS_ROOT)/boost\" \"$(PODS_ROOT)/glog\" \"$(PODS_ROOT)/RCT-Folly\"",
"OTHER_CFLAGS" => "$(inherited) -DRN_FABRIC_ENABLED" + " " + folly_flags }
Expand Down
1 change: 0 additions & 1 deletion ReactCommon/React-Fabric.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ Pod::Spec.new do |s|
s.platforms = { :ios => "11.0" }
s.source = source
s.source_files = "dummyFile.cpp"
s.library = "stdc++"
s.pod_target_xcconfig = { "USE_HEADERMAP" => "YES",
"CLANG_CXX_LANGUAGE_STANDARD" => "c++17" }

Expand Down
1 change: 0 additions & 1 deletion ReactCommon/react/renderer/graphics/React-graphics.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ Pod::Spec.new do |s|
s.author = "Facebook, Inc. and its affiliates"
s.platforms = { :ios => "11.0", :tvos => "11.0" }
s.source = source
s.library = "stdc++"
s.compiler_flags = folly_compiler_flags + ' ' + boost_compiler_flags
s.source_files = "**/*.{m,mm,cpp,h}"
s.exclude_files = "tests",
Expand Down
12 changes: 6 additions & 6 deletions packages/rn-tester/Podfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -883,19 +883,19 @@ SPEC CHECKSUMS:
Flipper-RSocket: d9d9ade67cbecf6ac10730304bf5607266dd2541
FlipperKit: d8d346844eca5d9120c17d441a2f38596e8ed2b9
fmt: ff9d55029c625d3757ed641535fd4a75fedc7ce9
glog: 837f9f3924abd268645a4d5b95736ab91c619fb1
glog: 85ecdd10ee8d8ec362ef519a6a45ff9aa27b2e85
libevent: 4049cae6c81cdb3654a443be001fb9bdceff7913
OpenSSL-Universal: 1aa4f6a6ee7256b83db99ec1ccdaa80d10f9af9b
RCT-Folly: a21c126816d8025b547704b777a2ba552f3d9fa9
RCT-Folly: 803a9cfd78114b2ec0f140cfa6fa2a6bafb2d685
RCTRequired: af2d6080a4b9ba0885b28ca78879a92066c71cab
RCTTypeSafety: c7a7f67ae5b1b986b78d817baa408fc984ab7c0c
React: f64c9f6db5428717922a3292ba6a448615a2e143
React-callinvoker: c5d61e29df57793f0dc10ec2bc01c846f863e51f
React-Core: f4cd5c72f7f1bf021ed386d3d6c1a89b5c59cfb1
React-Core: 22bc86b79dd931dbfb7fd2af91a35a98d41ceb64
React-CoreModules: 1a96917a9a5fd73a989b462ed4dee742fb92332f
React-cxxreact: cfc1663dae1ea52b465bbf021ef7b1527c5dc80c
React-Fabric: d49c26d5d651292c48dfcf0f0d78bb2e16f326a3
React-graphics: b52a1478e052dca3a9f1e8f894f4399988d67508
React-Fabric: 1489d7b89f472f291b950a483c905ca5443eb288
React-graphics: 9a84942b49ea1375ce83844041fa2ba766cfe0b9
React-jsi: c1c0108d86e1378aabf7c9146ed7d12d87c7df85
React-jsiexecutor: d08ec14edc8f2e19af51550cd1505332eae35c7b
React-jsinspector: 7d223826b0e7a61b3540c21b9eca2603b1d4e823
Expand All @@ -904,7 +904,7 @@ SPEC CHECKSUMS:
React-RCTActionSheet: 3131a0b9280aa0e51bdf54b3d79aecd8503db62c
React-RCTAnimation: 5b2a86f9b5dc9683bfcf6315c58d04e93da57fd7
React-RCTBlob: d417f5814b38d91c8e7a1f667aef7d36c6c789de
React-RCTFabric: b39da03f4147bfa1f0223ffd3c8c2de97185de04
React-RCTFabric: 921f6d8ae5211e1c0bd31d426487118a61502c8b
React-RCTImage: 1717771340c6319ed49a529faf1581c779c52d77
React-RCTLinking: 559c9223212ab2824950883220582c5e29a6fcb2
React-RCTNetwork: fe64ecd503461a8c01a0ab21d07cf9419f1cba54
Expand Down
2 changes: 0 additions & 2 deletions packages/rn-tester/RNTesterPods.xcodeproj/project.pbxproj
Original file line number Diff line number Diff line change
Expand Up @@ -791,7 +791,6 @@
OTHER_LDFLAGS = (
"$(inherited)",
"-ObjC",
"-l\"stdc++\"",
"-framework",
"\"JavaScriptCore\"",
);
Expand Down Expand Up @@ -824,7 +823,6 @@
OTHER_LDFLAGS = (
"$(inherited)",
"-ObjC",
"-l\"stdc++\"",
"-framework",
"\"JavaScriptCore\"",
);
Expand Down
2 changes: 1 addition & 1 deletion third-party-podspecs/RCT-Folly.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ Pod::Spec.new do |spec|
'folly/net/*.h',
'folly/net/detail/*.h',
'folly/portability/*.h'
spec.libraries = "stdc++", "c++abi" # NOTE Apple-only: Keep c++abi here due to https://github.com/react-native-community/releases/issues/251
spec.libraries = "c++abi" # NOTE Apple-only: Keep c++abi here due to https://github.com/react-native-community/releases/issues/251
spec.pod_target_xcconfig = { "USE_HEADERMAP" => "NO",
"CLANG_CXX_LANGUAGE_STANDARD" => "c++17",
"HEADER_SEARCH_PATHS" => "\"$(PODS_TARGET_SRCROOT)\" \"$(PODS_ROOT)/boost\" \"$(PODS_ROOT)/DoubleConversion\" \"$(PODS_ROOT)/libevent/include/\"" }
Expand Down
1 change: 0 additions & 1 deletion third-party-podspecs/glog.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ Pod::Spec.new do |spec|
spec.preserve_paths = 'src/*.h',
'src/base/*.h'
spec.exclude_files = "src/windows/**/*"
spec.libraries = "stdc++"
spec.compiler_flags = '-Wno-shorten-64-to-32'
spec.pod_target_xcconfig = { "USE_HEADERMAP" => "NO",
"HEADER_SEARCH_PATHS" => "$(PODS_TARGET_SRCROOT)/src" }
Expand Down

0 comments on commit aae9355

Please sign in to comment.