Closed tfoldi closed 5 months ago
hey @twdragon, the following changes were applied:
CMakeLists.txt
ament_target_dependencies
. This is more standard/idiomatic in ROS2, plus it looks way cleaner than the original filepackage.xml
. Now rosdep
can install the missing dependencies for the project.I made a few comments on the relevant code lines, hope that helps. Should you have any question please let me know
In general, other than the project definition and make files no
I build this branch successfully with ROS 2 Jazzy. Can this one merge to the main?
@tfoldi could you please provide a description about the issue you resolved, and some proofs?