-
Notifications
You must be signed in to change notification settings - Fork 248
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
[humble] rosbag2_storage links to deprecated target #1830
Comments
The problem, as far as I remember, is that on Humble we have to use the Are you using a custom compiled version of |
That is correct, we do have a compiled version (latest) of thank you very much |
Sort of.
Only if Ubuntu updates it in Ubuntu 22.04 (which is highly unlikely). |
Ok understood. Is there a way we can "force" rosbag2_storage to keep pointing to the system version of yaml-cpp and not the one we are installing? I guess one solution would be to install our custom version in a different way... Note: I saw that also in the system version the target |
Hello everyone,
We have a project containing targets linking to rosbag2_storage and the standard yaml-cpp library.
During compilation, we get the following CMake warning:
We're certain that all our targets are linking to yaml-cpp using the target
yaml-cpp::yaml-cpp
.Inside "/opt/ros/humble/share/rosbag2_storage/cmake/export_rosbag2_storageExport.cmake" we noticed the following lines:
set_target_properties(rosbag2_storage::rosbag2_storage PROPERTIES INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include/rosbag2_storage" INTERFACE_LINK_LIBRARIES "pluginlib::pluginlib;rcpputils::rcpputils;rcutils::rcutils;yaml-cpp" )
and substituting "yaml-cpp" with "yaml-cpp::yaml-cpp", the warning disappears.
We are convinced that this is a dirty solution to our problem.
Do you have any suggestions on what we can do to remove the warning cleanly?
Is it possible to add this "fix" to one of the next releases of ROS2 humble?
thank you very much!
The text was updated successfully, but these errors were encountered: