-
Notifications
You must be signed in to change notification settings - Fork 33
Updated grpc and protobuf for newer bazel versions #137
Conversation
still can't run it: root@vultr:~/compass# bazel run //docker:layers_calculator |
@nelsonchan1996 Could you try and start with a fresh environment? i tested it on your exact bazel version, and it seems to work! |
@nelsonchan1996 I confirmed on a new machine this is not yet solved. Ill take a look at it again later |
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.
Build breaks instantly with the errors reported by @nelsonchan1996
Builds for me now, merging. |
Fixes #135
Tested on bazel 0.28.1
Updated docker-rules from v0.5.1 to v0.9.0
Updated GRPC from v1.17.2 to v1.23.0
Updated protobuf from v3.6.1.3 to v3.8.0