-
Notifications
You must be signed in to change notification settings - Fork 446
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
Add bazel build for grpc example #838
Comments
I'm interesting in this. There some problems in #666 before.
I'm not familiar with |
For the gcc compiler, we can document the minimum supported gcc version for gRPC/otlp as defined by gRPC (4.9+ or 5.1+ whatever works) : We can further split our Bazeil build CI into two parts - 4.8 compiler without building grpc/otlp, 5.1+ compiler with grpc/otlp. This will allow us to upgrade Bazel and gRPC versions both. If all are fine, I can create a PR for updating our docs, and splitting Bazel build. |
I am putting it as help needed, probably someone with good knowledge |
This issue was marked as stale due to lack of activity. It will be closed in 7 days. |
#910 fixes it. |
grpc example ( https://github.com/open-telemetry/opentelemetry-cpp/tree/main/examples/grpc ) supports only cmake build as of now. bazel build needs to be added.
cc @jsuereth added bug as discussed here : #729 (review)
The text was updated successfully, but these errors were encountered: