-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
[DRAFT] Add ros_gz package #72
Conversation
Thanks for testing! Could you please also add on the other two platforms? |
Great. Let me know once you had a chance testing, and I’ll be happy to merge when you are. |
Actually I'm struggling a bit here, perhaps you can help out a bit.
Sorry, for all of the questions. I'm quite eager to understand how this works and contribute, but I'm a bit over my head here :) |
@Tobias-Fischer I hope I understand correctly - the issue is that to install a "metapackage" a custom configuration is needed. I can't seem to find a previous example of such a package. |
I guess there is nothing special about metapackages, they just install their package.xml and other metadata files, and then they just depend on the other packages that actually contains something. See https://github.com/ros-planning/moveit2/blob/main/moveit/CMakeLists.txt for an example of a metapackage that seems to be working fine: https://anaconda.org/robostack-staging/ros-humble-moveit/files . |
Hmm - if I look at the build log here: https://github.com/RoboStack/ros-humble/actions/runs/5290691692/job/14323454264?pr=72#step:15:713 - I don't see any of the source files or sub-packages being built. Looks like it's just building the meta files and that's it. Could it be that the GH actions pipeline needs to be updated? |
Hi @mkarklins - apologies for the confusion.
Now, by default, these packages pull in Fortress (see e.g. https://github.com/gazebosim/ros_gz/blob/4a5972bc6c1bcd2797a023ce934e6ef4d8e08de9/ros_gz_image/package.xml#L22-L23). As far as I know, there is no (easy?) way to pass GZ_VERSION to our vinca tool (I don't think we ever implemented these conditions).
I understand it's quite an involved process, but each individual step should be quite straightforward. Unfortunately I currently don't have the capacity to do these steps myself, but I'd be happy to help out if you get stuck @mkarklins. |
Very much appreciated for your reply!
All good! I should be able to do this myself, will report back with progress. |
We have this for most platforms now, and Windows is tracked in #68 |
This adds the
ros_gz
package.Was able to successfully build locally on macOS. Still want to run some tests with it to ensure it actually works.
Closes: #68