Skip to content

Add patched grid_map#101

Open
TobinHall wants to merge 2 commits into
ubi-agni:mainfrom
TobinHall:main
Open

Add patched grid_map#101
TobinHall wants to merge 2 commits into
ubi-agni:mainfrom
TobinHall:main

Conversation

@TobinHall

Copy link
Copy Markdown

I have forked grid_map and patched it to build on noble.

The patch was minimal changes to build - unpinning an exact cmake requirement and removing deprected api calls.

I tried to match the ROS2 branches of the repo as closely as possible. I have only validated that the changes are able to be built.

I am happy for this patched version to be moved to the ros-o organisation, but I have not had time to start a discussion.

@rhaschke

Copy link
Copy Markdown
Member

Looks like the patches fail on Jammy.

@TobinHall

Copy link
Copy Markdown
Author

Ah, it seems like there are libraries in 22.04 that don't like the pinned c++11 version.
I have updated them all to c++17 and it now builds in both jammy and noble.

I have pushed a change to the same branch.

Patch is minimal changes to successfully build. Changes followed the ros2 branch of the repo as much as possible.
Comment thread ros-one.repos Outdated
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