search

electrode-io / electrode-native

A platform to ease integration&delivery of React Native apps in existing mobile applications
https://native.electrode.io
Other
723 stars 113 forks source link

Fix pod install with repo update to update specs #1900

Closed rthic23 closed 9 months ago

rthic23 commented 9 months ago

-> Updating Manifest pod source with new spec doesn't update the pod with new version or find the pod source. Pod install with repo-update will update the local pod specs. pod install --repo-update -> CI builds tends to fail installing these pods when adding new pod spec sources.

friederbluemle commented 9 months ago

Hmm, so while --repo-update can fix certain issues, I am not sure if it's the right approach to always/unconditionally pass that flag. This should be an explicit operation on the user side. We simply don't know what repos users have configured locally.

leekuo commented 9 months ago

I believe the --repo-update flag just runs pod repo update before pod install. Alternatively, the user can also just run pod repo update before generating the iOS container.

Source: https://github.com/CocoaPods/CocoaPods/blob/479e71131962e3e560e4cc0601d39860c2a590ce/lib/cocoapods/command/install.rb#L32

rthic23 commented 9 months ago

@friederbluemle @leekuo closing it, as this is not required from ERN platform. I have also tested pod repo update from generated container. Works good.