-
Notifications
You must be signed in to change notification settings - Fork 156
wip: support bzlmod #377
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
base: master
Are you sure you want to change the base?
wip: support bzlmod #377
Conversation
#bazel_dep(name = "protobuf", version = "21.7", repo_name = "com_google_protobuf") | ||
bazel_dep(name = "grpc", version = "1.56.3", repo_name = "com_github_grpc_grpc") | ||
|
||
go_sdk = use_extension("@io_bazel_rules_go//go:extensions.bzl", "go_sdk") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go_sdk
needs to be marked as a dev dependency.
Tried this branch with the fix of @jvolkman and it works perfectly for our needs (python, go and java protobuf + grpc) |
any plans to move this forward? I'm also interested to see this merged at some point |
@pcj Any plan to move forward with this ? |
Yes, need to fix up a few things and get this merged. Thanks for the ping. |
Notably Scala and closure are in the BCR now, haven't checked the others in the workspace.bzlmod file |
I also tried this as a patch in bb-portal for the frontend portion (along with the go_sdk dev_dependency fix and adding the override for protobuf_javascript) but getting the following error :
Here's the build file I am adding here:
|
TODO
//cmd/gencopy:gencopy_test
and//cmd/gazelle:gazelle_test
MODULE.bazel
WORKSPACE.bzlmod
is empty)Fixes #319