Skip to content

all: bump protobuf to 3.12.0 - #7052

Merged
dapengzhang0 merged 4 commits into
grpc:masterfrom
dapengzhang0:bump-protobuf
May 20, 2020
Merged

all: bump protobuf to 3.12.0#7052
dapengzhang0 merged 4 commits into
grpc:masterfrom
dapengzhang0:bump-protobuf

Conversation

@dapengzhang0

Copy link
Copy Markdown
Contributor

Version 3.12.0 is needed for the feature of support for proto3 field presence #7051

@dapengzhang0
dapengzhang0 requested a review from ejona86 May 20, 2020 18:47

@ejona86 ejona86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One very important typo fix, but otherwise LGTM

Comment thread repositories.bzl Outdated
sha256 = "e60211a40473f6be95b53f64559f82a3b2971672b11710db2fc9081708e25699",
strip_prefix = "protobuf-0425fa932ce95a32bb9f88b2c09b995e9ff8207b",
urls = ["https://github.com/google/protobuf/archive/0425fa932ce95a32bb9f88b2c09b995e9ff8207b.zip"], # Commit with fixed javalite on 3.11.x branch
name = "b37e96e81842af659605908a421960a5dc809acbc888f6b947bc320f8628e5b1",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This name should not be changed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oops, pasting too fast. I'm surprised Kokoro Bazel build passed.

@dapengzhang0

Copy link
Copy Markdown
Contributor Author

@ejona86 I made a workaround in examples pom.xml. Sounds good to you?

@dapengzhang0
dapengzhang0 requested a review from ejona86 May 20, 2020 22:43

@ejona86 ejona86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, looks fine.

@dapengzhang0
dapengzhang0 merged commit 0d65467 into grpc:master May 20, 2020
@dapengzhang0
dapengzhang0 deleted the bump-protobuf branch May 20, 2020 23:31
@dapengzhang0 dapengzhang0 added the TODO:backport PR needs to be backported. Removed after backport complete label May 21, 2020
@ejona86 ejona86 removed the TODO:backport PR needs to be backported. Removed after backport complete label May 26, 2020
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 13, 2021
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