-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
[Bug]: ts_proto_library descriptor database error in nested workspace #451
Comments
I think this is a protoc bug, or maybe a bug in rules_proto, in the sense that the paths and arguments we pass to The content of the descriptor set is something like (ignore binary goop)
so it looks like @thesayyn do you agree it's a bug in bazelbuild/rules_proto? |
@alexeagle is there a workaround for this currently? I only hit this issue with TS and not with Python for the same protos so I'm not sure if it's a protoc bug. |
Also, I realize that in my case there is no nested workspace but I get the same error. My setup is a single workspace file at the root and then in a directory in the BUILD file I have this:
which seems like a pretty standard setup. I get this error:
but this file does exist under |
Just ran into this. I can confirm that the path to the proto does exist, but the proto descriptor contains a truncated path, as @alexeagle says. What I'm wondering is, how do any of the other proto rules work? |
One thing that's different about almost all other proto library rules, is they use |
I think I've worked out what's going on here. Firstly, setting I think if we use |
What happened?
repro: https://github.com/binoche9/repros/tree/ts-proto-library-nested-workspace
Reported by @binoche9 and I think @pcj as well.
Version
HEAD
How to reproduce
No response
Any other information?
No response
The text was updated successfully, but these errors were encountered: