-
Notifications
You must be signed in to change notification settings - Fork 269
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
Unknown CMake command "PROTOBUF_GENERATE_CPP". #956
Comments
I think your Protobuf version is too new. What OS are you in? Would you mind using the issue template next time and filling the relevant info? Thanks! ✌️ Cross-referencing this related PR: #715 |
Probably you need to define to |
My apologies. I did not use the template because none of it seemed relevant, but still, here it is : What version of protobuf is needed ? The
I already tried patching the relevant (I think)
|
I am afraid that patch will not work if |
In a sense, probably we just need to move the line https://github.com/ignitionrobotics/ign-gazebo/blob/ign-gazebo5/CMakeLists.txt#L140 before any call to |
Adding to the Now I hit another compilation bug, which seems related to
I guess I should open a separate issue for this ? The path should obviously be Not sure (yet) how |
Any pointer about how those
|
I guess the patch in gazebosim/gz-physics#282 should fix the problem (note that the PR is against ign-physics4, but I guess that probably a similar fix can be applied to ign-physics 2). |
See https://cmake.org/cmake/help/latest/manual/cmake-packages.7.html#creating-relocatable-packages for more details on CMake relocatable packages (that I guess it is what is required by easybuild, even if I am not an expert). |
We do not (usually) install in place, indeed. So I guess they need to be relocatable. Thanks for the pointer. I will try the patch (or a similar patch). |
Not sure what is your use case for ign-gazebo, but in that case probably you may be interested in #626 . |
I am just trying to build all of |
🤔 , now the configuration passes, but the build fails... Maybe building gazebo is not such a good idea, but if it does not build, doesn't that mean that other components were not built correctly ?
|
It is not a bad idea, but probably it is more work then necessary, as for Gazebo 11 you will not need ignition-gazebo or ignition-physics, that are exactly the one that are giving you problems. However, I guess we are definitely OT at this point here, so perhaps it could make sense to open an issue in osrf/gazebo repo instead? |
Ok. I will do this. |
If it helps, instead of building the entire collection-citadel from source as instructed on our docs, you can just get the Gazebo 11 dependencies from gazebo11.yaml. |
I'm trying to build
ign-gazebo3_3.8.0
, and I encounter:According to this page
https://stackoverflow.com/questions/52533396/cmake-cant-find-protobuf-protobuf-generate-cpp
there would be something missing in the
CMakeLists.txt
, but even patching it did not resolve the issue.I have
protobuf/3.12.3
andcmake/3.16.5
Any idea ?
Below is the full log:
The text was updated successfully, but these errors were encountered: