-
Notifications
You must be signed in to change notification settings - Fork 536
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
Confusing error message if rules_proto isn't loaded #1080
Comments
This issue has been automatically marked as stale because it has not had any activity for 180 days. It will be closed if no further activity occurs in 30 days. |
This issue was automatically closed because it went 30 days without a reply since it was labeled "Can Close?" |
can a maintainer reopen this issue? this seems to be still valid issue. maybe @alexeagle? |
This issue has been automatically marked as stale because it has not had any activity for 180 days. It will be closed if no further activity occurs in 30 days. |
Protobuf team is taking ownership of `py_proto_library` and the implementation was moved to protobuf repository. Remove py_proto_library from rules_python, to prevent divergent implementations. Make a redirect with a deprecation warning, so that this doesn't break any users. Expected side effect of this change is also that the protobuf version is sufficiently updated that there is no more use of legacy struct providers. Closes #1935 Closes #1924 Closes #1925 Closes #1703 Closes #1707 Closes #1597 Closes #1293 Closes #1080 Fixes #1438
🐞 bug report
Affected Rule
The issue is caused by the rule: py_proto_libraryIs this a regression?
NoDescription
A clear and concise description of the problem...If rules_proto isn't loaded, using py_proto_library fails with a somewhat confusing error message:
Maybe the error message could be improved slightly, e.g. by checking
hasattr(proto_common, 'ProtoLangToolchainInfo')
.🔬 Minimal Reproduction
WORKSPACE:
BUILD:
test.proto:
🔥 Exception or Error
🌍 Your Environment
Debian GNU/Linux
Output of
bazel version
:6.0.0
Rules_python version:
0.18.1
The text was updated successfully, but these errors were encountered: