Skip to content

protoc-gen-rust-grpc: Fix well-known protos for prost#2665

Open
ejona86 wants to merge 3 commits into
grpc:masterfrom
ejona86:protoc-cmake-prost
Open

protoc-gen-rust-grpc: Fix well-known protos for prost#2665
ejona86 wants to merge 3 commits into
grpc:masterfrom
ejona86:protoc-cmake-prost

Conversation

@ejona86
Copy link
Copy Markdown
Member

@ejona86 ejona86 commented May 29, 2026

This makes the well-known protos available to the protoc built by protoc-gen-rust-grpc. It needed to "install" the protos, but also run the "installed" protoc executable so that it could find the protos in the include/ directory next to its own bin/. Note that the well-known protos don't yet work with grpc-protobuf-build because of missing crate mapping.

With that in place, it can be used in the examples.

@ejona86 ejona86 requested a review from dfawley May 29, 2026 15:02
This makes the well-known protos available to the protoc built by
protoc-gen-rust-grpc. It needed to "install" the protos, but also run
the "installed" protoc executable so that it could find the protos in
the include/ directory next to its own bin/. Note that the well-known
protos don't yet work with grpc-protobuf-build because of missing crate
mapping.

With that in place, it can be used in the examples.
@ejona86 ejona86 force-pushed the protoc-cmake-prost branch from de97b61 to 62e5b90 Compare May 29, 2026 15:07
Comment thread examples/README.md
Comment on lines +40 to +41
`protoc-gen-rust-grpc` crate. When using this approach, you can also set the
`protoc-gen-rust-grpc` feature to use the same protoc for the prost examples.
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.

As discussed offline, I think we should move this new bit up to the previous section since it's about how we can avoid the protoc install for tonic builds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants