Skip to content
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

Switch to using the liboctomap-dev key for a dependency. #5

Merged

Conversation

clalancette
Copy link
Contributor

This is now available in rosdep.

See ros/rosdistro#41623 .

To be perfectly frank, when I test built this locally, I didn't see much of a difference before and after this patch. Regardless, if this is something that DART depends on, we should enable it here.

This is now available in rosdep.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette
Copy link
Contributor Author

One other note; assuming this is accepted, this should be backported to Jazzy.

@azeey azeey merged commit f7e15d7 into gazebo-release:rolling Jun 24, 2024
@clalancette clalancette deleted the clalancette/switch-to-liboctomap-dev branch June 25, 2024 12:01
@clalancette
Copy link
Contributor Author

@azeey Can you do a bloom-release for this into Rolling so we can make sure that this compiles (I tested locally, but the buildfarm can be more interesting). Once we confirm that, I'll open the backport PR to Jazzy.

@azeey
Copy link
Collaborator

azeey commented Jun 25, 2024

Sure. liboctomap-dev was already being pulled by libfcl-dev, so I expect this to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants