Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Release] Test with gRPC 1.44 and absl 20211102 #11857

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

andrewheard
Copy link
Contributor

#no-changelog

Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on green

Comment on lines +1325 to +1326
"https://github.com/abseil/abseil-cpp.git",
"20211102.0" ..< "20211103.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@paulb777 To quickly test this I switched to the https://github.com/abseil/abseil-cpp.git repo instead of https://github.com/firebase/abseil-cpp-SwiftPM.git (which doesn't have 20211102). Merging this would degrade clone times somewhat when using SPM. I'll see if I can find some instructions for how to proceed.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, that explains why I didn't see the problem today that I saw yesterday. 😛

Yep, ideally we should get the abseil binary. the instructions are around somewhere ....

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to clarify, the abseil binary repo was fine (https://github.com/google/abseil-cpp-binary/tree/1.2021110200.0). The https://github.com/firebase/abseil-cpp-SwiftPM.git repo is used for source builds of Firestore.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We discussed offline and for this release we will use the https://github.com/abseil/abseil-cpp.git repo since it will only affect clone times when using the Firestore source distribution (hidden behind the FIREBASE_SOURCE_FIRESTORE environment variable).

Note: When we previously used the 1.44.0 gRPC release in Firebase 10.2, we had a mismatch in the abseil version between SPM and CocoaPods. Context:

"0.20220203.1" ..< "0.20220204.0"

This is why we don't have 20211103 tagged in the https://github.com/firebase/abseil-cpp-SwiftPM.git repo.

@andrewheard andrewheard merged commit fce4e1b into ah/firestore-spm Sep 27, 2023
@andrewheard andrewheard deleted the ah/spm-grpc-absl branch September 27, 2023 17:20
@firebase firebase locked and limited conversation to collaborators Oct 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants