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

[1/3] Bump grpc from 1.47.0 to 1.48.1 #18240

Closed
wants to merge 1 commit into from

Conversation

davido
Copy link
Contributor

@davido davido commented Apr 27, 2023

PART 1: prepare third_party/grpc files for new version
Composed PR: #18216.

Fixes #18167.

abseil-cpp is missing stdint.h header include, that broke recent compiler versions. This problem was fixed in this commit upstream: [1].

Due to another commit: [2] in abseil-cpp, we also have to bump grpc as well to make them compatible.

[1] abseil/abseil-cpp@36a4b07
[2] abseil/abseil-cpp@b8bbe92

@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Apr 27, 2023
@sgowroji sgowroji added the team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. label Apr 28, 2023
PART 1: prepare third_party/grpc files for new version
Composed PR: bazelbuild#18216.

Fixes bazelbuild#18167.

abseil-cpp is missing stdint.h header include, that broke recent
compiler versions. This problem was fixed in this commit upstream: [1].

Due to another commit: [2] in abseil-cpp, we also have to bump grpc as
well to make them compatible.

[1] abseil/abseil-cpp@36a4b07
[2] abseil/abseil-cpp@b8bbe92
@davido davido force-pushed the grpc_1_48_1_part_1 branch from 2c91fa7 to 465748a Compare April 28, 2023 05:11
@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 28, 2023
copybara-service bot pushed a commit that referenced this pull request Apr 28, 2023
PART 1: prepare third_party/grpc files for new version
Composed PR: #18216.

Fixes #18167.

abseil-cpp is missing stdint.h header include, that broke recent
compiler versions. This problem was fixed in this commit upstream: [1].

Due to another commit: [2] in abseil-cpp, we also have to bump grpc as
well to make them compatible.

[1] abseil/abseil-cpp@36a4b07
[2] abseil/abseil-cpp@b8bbe92

Partial commit for third_party/*, see #18240.

Signed-off-by: Pavan Singh <[email protected]>
@sgowroji
Copy link
Member

Merged at d18c506

@sgowroji sgowroji closed this Apr 28, 2023
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 28, 2023
copybara-service bot pushed a commit that referenced this pull request Apr 29, 2023
Previous PR: #18240

Closes #18241.

PiperOrigin-RevId: 528053742
Change-Id: I360b0b1db6ff22464939bfa388b185c980dd4ec4
fweikert pushed a commit to fweikert/bazel that referenced this pull request May 25, 2023
PART 1: prepare third_party/grpc files for new version
Composed PR: bazelbuild#18216.

Fixes bazelbuild#18167.

abseil-cpp is missing stdint.h header include, that broke recent
compiler versions. This problem was fixed in this commit upstream: [1].

Due to another commit: [2] in abseil-cpp, we also have to bump grpc as
well to make them compatible.

[1] abseil/abseil-cpp@36a4b07
[2] abseil/abseil-cpp@b8bbe92

Partial commit for third_party/*, see bazelbuild#18240.

Signed-off-by: Pavan Singh <[email protected]>
fweikert pushed a commit to fweikert/bazel that referenced this pull request May 25, 2023
Previous PR: bazelbuild#18240

Closes bazelbuild#18241.

PiperOrigin-RevId: 528053742
Change-Id: I360b0b1db6ff22464939bfa388b185c980dd4ec4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't bootstrap with clang >= 15 and gcc >= 13
3 participants