Don't double-build convert.cpp (#184)

Signed-off-by: Michael Carroll <michael@openrobotics.org>
gazebosim · Oct 27, 2021 · 7e49c03 · 7e49c03
1 parent a258c48
commit 7e49c03
Showing 1 changed file with 14 additions and 17 deletions.
diff --git a/ros_ign_bridge/CMakeLists.txt b/ros_ign_bridge/CMakeLists.txt
@@ -67,6 +67,19 @@ set(common_sources
   src/factories.cpp
 )
 
+set(bridge_lib
+  ros_ign_bridge_lib
+)
+
+add_library(${bridge_lib}
+  ${common_sources}
+)
+target_link_libraries(${bridge_lib}
+  ${catkin_LIBRARIES}
+  ignition-msgs${IGN_MSGS_VER}::core
+  ignition-transport${IGN_TRANSPORT_VER}::core
+)
+
 set(bridge_executables
   parameter_bridge
   static_bridge
@@ -75,31 +88,15 @@ set(bridge_executables
 foreach(bridge ${bridge_executables})
   add_executable(${bridge}
     src/${bridge}.cpp
-    ${common_sources}
   )
   target_link_libraries(${bridge}
-    ${catkin_LIBRARIES}
-    ignition-msgs${IGN_MSGS_VER}::core
-    ignition-transport${IGN_TRANSPORT_VER}::core
+    ${bridge_lib}
   )
   install(TARGETS ${bridge}
           DESTINATION ${CATKIN_PACKAGE_BIN_DESTINATION}
   )
 endforeach(bridge)
 
-set(bridge_lib
-  ros_ign_bridge_lib
-)
-
-add_library(${bridge_lib}
-  ${common_sources}
-)
-target_link_libraries(${bridge_lib}
-  ${catkin_LIBRARIES}
-  ignition-msgs${IGN_MSGS_VER}::core
-  ignition-transport${IGN_TRANSPORT_VER}::core
-)
-
 catkin_package(INCLUDE_DIRS include
                LIBRARIES ${bridge_lib})