-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Dependency for generators #4131
Comments
Isn't there a 'dependencies' keyword for generator? |
As far as I can see there isn't: https://mesonbuild.com/Reference-manual.html#generator |
The reason is that you're using |
The problem I see by using custom_target is that I do not have a process method, so the inputs are basically hardcoded |
We could probably add a |
sounds good to me |
@nirbheek I tried to have a look at the patch but I have not much knowledge of the internals of meson. Any chance you could make a patch for this? would be awesome to have protobuf-c with meson |
Hi, protobuf-c maintainer here. I'd still like to switch protobuf-c from autotools/cmake to meson at some point, but if I recall correctly we were stuck on this issue. Any progress? Thanks! |
Can you test if #5407 fixes it for you? |
@jpakkane Just tested and that does appear to fix it. Thank you! |
I was having a look at the port of protobuf-c to meson and one issue they build an executable that is passed as a command line param to the generator in this format:
When I try to use the generator it does not find the protoc_gen_c yet, I think we need a way to tell the generator that the executable has to be ready before calling it.
The text was updated successfully, but these errors were encountered: