-
Notifications
You must be signed in to change notification settings - Fork 794
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
[rolling] grid_map_pcl needs PCLConfig.cmake but not included in package.xml #341
Conversation
…evel) (ANYbotics#308) * Replaced pcl_ros dependency with PCL * Removed PCL version specification * Removed pcl_ros dependency * Updated changelog * Updated package.xml with correct PCL dependency
In compile time, it needs PCLConfig.cmake for configuration, which is in libpcl-dev(in rosdep, it is defined as libpcl-all-dev)
Maybe this PR is subset of #245 ? Even if not, the topic seems related. |
Hello, It appears this is still affecting non-EOL distro's. Do you mind changing the target branch to |
Sure. There’s no problem to rebase it. Any actions for me to do? |
Naw, just a rebase, and test it if you have time to make sure the key resolves in |
Looks like grid_map/grid_map_pcl/package.xml Line 24 in 886ba24
Same with grid_map/grid_map_pcl/package.xml Line 24 in d97bf39
A more optimal way to do dependencies is exactly what you did. I would be happy to accept this change in |
See #423 for a rebased and correct fix. |
In compile time, it needs PCLConfig.cmake for configuration, which is in libpcl-dev(in rosdep, it is defined as libpcl-all-dev)
On,
,
I failed to build grid_map_pcl like below: