AIS-Bonn / AIS-Bonn/nimbro_network

Wrong catch_ros include dir in certain situations

Aperta
#19 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
C++
Stelle
156
Fork
43
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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/AIS-Bonn/nimbro_network/blob/7e1de20c8fb33fb1bf22ab9a8c4e93d098b9c4a3/nimbro_topic_transport/CMakeLists.txt#L170

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

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.