rosmon icon indicating copy to clipboard operation
rosmon copied to clipboard

Wrong catch_ros include dir in certain situations

Open xqms opened this issue 3 years ago • 0 comments

As discovered in https://github.com/AIS-Bonn/catch_ros/pull/15, we currently prefer the catch_ros headers from e.g. /opt/ros/noetic/include in favor of the (correct) ones from the current workspace.

The reason is this line: https://github.com/xqms/rosmon/blob/bc88c34c7bf96483f6ff4f9e46193cf40b27f57c/rosmon_core/CMakeLists.txt#L143

... which adds the catch_ros headers to the end of all include paths. Instead, we should use catkin's list_insert_in_workspace_order() function to correctly insert the additional include path.

xqms avatar Aug 20 '21 09:08 xqms