-
Notifications
You must be signed in to change notification settings - Fork 589
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
Problems with 'validate_proto' not declared in package 'validate' #1199
Comments
This hit me as well. rules_cc has removed cc_proto_library as of v0.1.0. See the Incompatible Changes section here: The fix is to use load("@com_google_protobuf//bazel:cc_proto_library.bzl", "cc_proto_library") |
The rules_proto repo has been deprecated entirely as well. It should no longer be used. See the README here: Instead, load("@com_google_protobuf//bazel:proto_library.bzl", "proto_library") |
I'll take a stab at removing |
Closes bufbuild#1199 rules_proto has been deprecated and should no longer be used. rules_cc removed the proto-related bazel rules as of 0.1.0. Instead, these rules now live in com_google_protobuf and should be used from there. Several other dependencies also had to be updated to support the changes in com_google_protobuf and cc_proto.
I believe I have fixed the issue for this repo in #1210. This problem will also affect rules_buf and gazelle as well, since gazelle generates build files which currently use rules_proto. |
Closes #1199 This is a more minimal change to simply migrate away from rules_cc's cc_proto_library without updating all the external deps. This will prepare for the upcoming 0.1.X release of rules_cc where cc_proto_library is removed.
Curently when I try to build my project I get:
It seams that the version of rules_cc that is being used does not correspond to the version expected by protoc-gen-validate?
my modules:
and the config:
Any idea what can I do?
The text was updated successfully, but these errors were encountered: