-
Notifications
You must be signed in to change notification settings - Fork 73
Update protobuf to 3.20.0 #113
Update protobuf to 3.20.0 #113
Conversation
e90450c to
6b168d2
Compare
|
I don't think we should be updating precompiled protobuf to rc versions. |
|
I'll update this when 3.20 is released. |
|
3.20 was released. |
Also add prebuit protoc binaries for Apple silicon.
6b168d2 to
6f02e98
Compare
|
Updated! |
proto/private/dependencies.bzl
Outdated
| "sha256": "0019dfc4b32d63c1392aa264aed2253c1e0c2fb09216f8e2cc269bbfb8bb49b5", | ||
| "urls": [ | ||
| "https://mirror.bazel.build/github.com/protocolbuffers/protobuf/releases/download/v3.20.0/protoc-3.20.0-osx-aarch64.zip", | ||
| "https://github.com/protocolbuffers/protobuf/releases/download/v3.20.0/protoc-3.20.0-osx-aarch64.zip", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File not found
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems there's an underscore missing:
https://github.com/protocolbuffers/protobuf/releases/download/v3.20.0/protoc-3.20.0-osx-aarch_64.zip
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And checksum is also different.
comius
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
I mirrored the binaries. |
And add prebuilt protoc binaries for Apple silicon.