-
Notifications
You must be signed in to change notification settings - Fork 440
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 bzlmod support #2334
Comments
@auzhva unfortunately we could not offer any official packaging. |
While this request makes sense, we (in opentelemetry-cpp) can not provide packaging for every build system. The best way to address this is to make a contribution to the Bazel Central Registry directly. |
This issue is available for anyone to work on. Make sure to reference this issue in your pull request. |
Added Do not prepare a PR for opentelemetry-cpp, contribute to https://registry.bazel.build/ instead. |
This issue was marked as stale due to lack of activity. |
👋 I submitted a PR for the modern bazel dependency management setup here: I know there was some discussion above about submitting the config to the BCR (which I will do if this PR is merged), but my PR here is still required to modernize the bazel config in this repo. We could also theoretically remove the old version of doing things in bazel, since right now there is some duplication, but I am initially leaving it in my PR to not affect users on older versions of bazel. I'm happy to remove it though and direct older bazel users to prior releases of this library instead if you'd prefer! |
This issue was marked as stale due to lack of activity. |
still valid, linked PR should be unblocked when my other prs land |
Since bazel 6.0, bazel added bzlmod dependency management and tracking system. Currently open telemetry is built by Bazel but is not published to the Bazel Central Regestry, which complicates importing open telemetry.
It would be awesome if open telemetry would be packaged to the BCR.
Is your feature request related to a problem?
Opentelemetry imports dependencies with own
opentelemetry_cpp_deps
, which is not portable to bzlmod.Describe the solution you'd like
Create MODULE.bazel and publish to the BCR
Describe alternatives you've considered
Continue using existing, but it's... Hard...
The text was updated successfully, but these errors were encountered: