buildbarn / bb-storage

Storage daemon, capable of storing data for the Remote Execution protocol
Apache License 2.0
137 stars 91 forks source link

build: Modify patch to remote-apis for greater compatibility #170

Closed minor-fixes closed 1 year ago

minor-fixes commented 1 year ago

This change modifies the patch to the bazelbuild/remote-apis repo to also (re)generate code for remote_asset.proto, so that bazelbuild/bb-remote-asset can share this patch, and use a library built from codegen run with a recent (and patched) version of protoc-gen-go (that generates types that takes a ServiceRegistrar in generated Register* methods instead of a concrete *grpc.Server).

Tested: bazel test //... still passes on my system