-
Notifications
You must be signed in to change notification settings - Fork 31
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
Convenience CMake for Gazebo Metapackage #69
Comments
Original comment by Michael Grey (Bitbucket: mxgrey, GitHub: mxgrey). This sounds like a really great idea. However, I think I’d recommend implementing it as a meta-package per release rather than cramming the logic into In other words, we would have a repo called We could have a function in |
Original comment by Michael Carroll (Bitbucket: Michael Carroll, GitHub: mjcarroll). I agree, that Do you think it would make sense for it to source the information from something like gazebodistro or should it have another copy of the data internally? |
Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).
You're in luck, that's exactly what we already have, and that's how we've been releasing the
We currently have a copy, which is not great. |
Original comment by Silvio Traversaro (Bitbucket: traversaro).
I implemented a really minimal vcstool |
more modern example:
I would expect this to create the following cmake targets:
and probably any package dependencies not explicitly requested such as:
and maybe some version variables like
|
I thought about this before, and to make version changes even easier, I think we could do something like this as a step further:
The thing that I actually ran into is how to deal with sub-components, so:
|
ooh, yeah subcomponents make it tough also, if these cmake helpers are provided by |
Original report (archived issue) by Michael Carroll (Bitbucket: Michael Carroll, GitHub: mjcarroll).
Based off of discussion over on ros_ign github, it may make sense to create some CMake logic that would allow a one-shot configuration of multiple ignition packages.
We could replace logic like this:
With something like
Which would bring in the required packages and populate the necessary versions for downstream use.
The text was updated successfully, but these errors were encountered: