-
Notifications
You must be signed in to change notification settings - Fork 4.1k
bazel: use buf protoc instead of protoc in the Bazel build #67091
Copy link
Copy link
Closed
Labels
A-build-systemC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-dev-inf
Description
- Wait for
bufto support--descriptor_set_in, see discussion here: Makefile: replace protoc with buf #66913, esp Makefile: replace protoc with buf #66913 (comment) - Add a shim
sh_binarythat depends on@com_github_bufbuild_buf//cmd/bufand does nothing except callbuf protoc "$@" - In
.bazelrc, addbuild --proto_compiler=//path/to/shim:protoc - Patch
rules_goto allow configuring theprotocused (see https://github.com/bazelbuild/rules_go/blob/4cd45a2ac59bd00ba54d23ebbdb7e5e2aed69007/proto/compiler.bzl#L218; rename_protoc->protoc) - We'll probably have to patch
gazelleto allow configuring aprotocas well -- probably something likegazelle:go_protoc //path/to/shim:protoc.
Epic CRDB-8036
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-build-systemC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-dev-inf