Skip to content

fix(c++): Remove c++ generation - #76

Merged
rustybee42 merged 1 commit into
mainfrom
rb/cpp-proto
Aug 28, 2026
Merged

rustybee42 merged 1 commit into
mainfrom
rb/cpp-proto

Conversation

@rustybee42

@rustybee42 rustybee42 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

C++ code must be generated using the exact same version of protobuf and gRPC that is used by the libraries using the code later.

If we ever use C++ gRPC at all, we most likely want to depend on the distro packages to avoid doing the time and resource consuming manual build. These are different among the supported distributions, so we can't have fixed generated code.

Instead, the code generation step must be done during the build step of the consuming projects (e.g. using cmake).

@rustybee42 rustybee42 self-assigned this Jun 8, 2026
C++ code must be generated using the exact same version of protobuf and
gRPC that is used by the libraries using the code later.

Since we want to depend on the distro packages to avoid doing the time
and resource consuming manual build, these are different among the
supported distibutions, so we can't have fixed generated code.

Instead, the code generation step is moved to the configure/build stage
of the consuming projects.

@iamjoemccormick iamjoemccormick left a comment

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.

Seems harmless since it sounds like nothing could / should use these anyway.

@rustybee42
rustybee42 merged commit 28107f4 into main Aug 28, 2026
2 checks passed
@rustybee42
rustybee42 deleted the rb/cpp-proto branch August 28, 2026 06:05
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