Skip to content

fix(ios): Podspec for React Native 0.84+ (RCT-Folly / install_modules_dependencies)#442

Open
christian-apollo wants to merge 1 commit intobamlab:masterfrom
christian-apollo:fix/ios-podspec-rn-084-install-modules-dependencies
Open

fix(ios): Podspec for React Native 0.84+ (RCT-Folly / install_modules_dependencies)#442
christian-apollo wants to merge 1 commit intobamlab:masterfrom
christian-apollo:fix/ios-podspec-rn-084-install-modules-dependencies

Conversation

@christian-apollo
Copy link
Copy Markdown

Summary

React Native 0.84+ no longer publishes a standalone RCT-Folly CocoaPods spec when using prebuilt ReactNativeDependencies. The podspec still declared s.dependency "RCT-Folly" (and duplicated several deps that install_modules_dependencies(s) already adds), which breaks pod install with:

Unable to find a specification for `RCT-Folly` depended upon by `react-native-image-resizer`

Changes

  • Remove redundant React-Codegen, RCT-Folly, RCTRequired, RCTTypeSafety, and ReactCommon/turbomodule/core lines from the New Architecture branch.
  • Rely on install_modules_dependencies(s) to attach the correct dependencies for the installed React Native version (including third-party / Folly via RN’s own helpers).

This matches the approach recommended for native modules on current RN versions.

Testing

  • pod install in an app on RN 0.84.x with New Architecture enabled
  • Existing RN 0.7x / 0.8x projects (install_modules_dependencies is the supported API)

Related: same fix is applied locally via patch-package in consumer apps until this ships in a release.

Made with Cursor

React Native 0.84 removes the standalone RCT-Folly pod when using
prebuilt ReactNativeDependencies. Manual pod dependencies duplicated
what install_modules_dependencies() already adds and caused CocoaPods
to fail with 'Unable to find a specification for RCT-Folly'.

Remove the redundant declarations and let install_modules_dependencies
configure New Architecture deps for the installed RN version.

Made-with: Cursor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant