Update iOS dependencies - Tue Apr 12 2022

[firebase-workflow-trigger]

Triggered by firebase-ios-sdk refs/tags/v8.15.0 release.

iOS

• Firebase/Analytics → 8.14.0
• Firebase/Auth → 8.14.0
• Firebase/Core → 8.14.0
• Firebase/Crashlytics → 8.14.0
• Firebase/Database → 8.14.0
• Firebase/DynamicLinks → 8.14.0
• Firebase/Firestore → 8.14.0
• Firebase/Functions → 8.14.0
• Firebase/Installations → 8.14.0
• Firebase/Messaging → 8.14.0
• Firebase/RemoteConfig → 8.14.0
• Firebase/Storage → 8.14.0

Created by Update Android and iOS dependencies workflow.

[github-actions]

✅  Integration test succeeded!

Requested by @firebase-workflow-trigger[bot] on commit 989781b
Last updated: Tue Apr 12 14:49 PDT 2022
View integration test log & download artifacts

[jonsimantov]

@dconeybe Because of the custom Firestore iOS SDK external package version, it didn't get updated automatically here. Should we update it to the 8.14.0 Cocoapods tag?

[dconeybe]

custom Firestore iOS SDK external package version

What do you mean by "custom Firestore iOS SDK external package version"?

[dconeybe]

@jonsimantov Are you referring to this line:

firebase-cpp-sdk/cmake/external/firestore.cmake

Line 63 in fd054fa

GetReleasedDep("CocoaPods-8.12.1")

[jonsimantov]

@jonsimantov Are you referring to this line:

firebase-cpp-sdk/cmake/external/firestore.cmake

Line 63 in fd054fa

GetReleasedDep("CocoaPods-8.12.1")

Ah my mistake - for some reason I thought it was failing to update due to a custom version.

So it should be changed to CocoaPods-8.14.0, right?

[dconeybe]

So it should be changed to CocoaPods-8.14.0, right?

Yes, that version should definitely be updated. Good catch!

Is the reason that it was missed because the way that the version was specified was changed from

set(version CocoaPods-8.12.1)

to

GetReleasedDep("CocoaPods-8.12.1")

?

If yes, I can definitely change it back.

[jonsimantov]

So it should be changed to CocoaPods-8.14.0, right?

Yes, that version should definitely be updated. Good catch!

Is the reason that it was missed because the way that the version was specified was changed from

set(version CocoaPods-8.12.1)

to

GetReleasedDep("CocoaPods-8.12.1")

?

If yes, I can definitely change it back.

No need to change it back - but we should fix update-dependencies.yml (line 81) to change the updated line.

[dconeybe]

No need to change it back - but we should fix update-dependencies.yml (line 81) to change the updated line.

I think I'll change it back actually. There was no need for it to change and the original line is more consistent with other cmake files for external projects. (Update: here is the PR: #899).

firebase-cpp-sdk/.github/workflows/update-dependencies.yml

Line 81 in fd054fa

sed -i~ "s/^set(version [^)]*)/set(version CocoaPods-${firestore_version})/i" cmake/external/firestore.cmake

Also, maybe (in a separate PR) the sed command should be tweaked to fail if it doesn't find a match. That would raise the visibility of this issue. Here is an example I found via a quick internet search: https://stackoverflow.com/questions/15965073/return-value-of-sed-for-no-match.