Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove protobuf-build completely #545

Closed
BusyJay opened this issue Sep 14, 2021 · 0 comments · Fixed by #595
Closed

remove protobuf-build completely #545

BusyJay opened this issue Sep 14, 2021 · 0 comments · Fixed by #595

Comments

@BusyJay
Copy link
Member

BusyJay commented Sep 14, 2021

Is your feature request related to a problem? Please describe.
grpcio-proto depends on protobuf-build to build dependency, protobuf-build depends on grpcio-compiler, hence cause circular dependency. It's not easy to maintain.

Describe the solution you'd like
Remove protobuf-build completely. Instead, using custom scripts to generate files instead.

BusyJay added a commit that referenced this issue Nov 1, 2022
Since 0.11, prost-build never bundles protoc binary anymore. To avoid
requiring protoc for grpcio-proto users, we bundle the sources instead
of generating at compile time. This can lead to potential problems like:
- Miss match version between generating tools and runtime
- Repository becomes bloated

But there seems to be no better alternative.

Since this PR when using prost with grpcio, protoc is also required in PATH.

This PR also removes protobuf-build dependencies. From now on, it will be easier to publish new version of grpcio-compiler.

Close #545.

Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant