ament / ament/ament_cmake

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

Ouverte
#405 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub
enhancement help wanted
Langage dominant
CMake
Étoiles
125
Forks
147
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.