ament / ament/ament_cmake

ament_add_gtest_executable: Uses old style target_link_libraries, should use keyword if possible?

Aperta
#405 1 commento 1 reazione 0 assegnatari Vedi su GitHub
enhancement help wanted
Lingua principale
CMake
Stelle
125
Fork
147
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

In relation to comment here:
https://github.com/ros2/rosidl/issues/400#issuecomment-1283021111

I found that the macro `ament_add_gtest_executable` seems to not use the newer keyword style of linking:
https://github.com/ament/ament_cmake/blob/799183ab9bcfd9b66df0de9b644abaf8c9b78e84/ament_cmake_gtest/cmake/ament_add_gtest_executable.cmake#L52-L54

Relates https://github.com/ament/ament_cmake/issues/383

\cc @sloretz @iantheengineer

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.