Merge pull request #13 from eholum/fix-atomic-clang-build

EzraBrooks · web-flow · commit 22bd0a117ff9 · 2023-05-30T11:44:30.000-06:00
Fix Transitive Includes and Linking for Atomic when Building with Clang
diff --git a/robotiq_driver/CMakeLists.txt b/robotiq_driver/CMakeLists.txt
@@ -36,6 +36,7 @@ target_include_directories(
   $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
   $<INSTALL_INTERFACE:include>
 )
+target_link_libraries(${PROJECT_NAME} atomic)
 ament_target_dependencies(
   ${PROJECT_NAME}
   ${THIS_PACKAGE_INCLUDE_DEPENDS}
diff --git a/robotiq_driver/include/robotiq_driver/hardware_interface.hpp b/robotiq_driver/include/robotiq_driver/hardware_interface.hpp
@@ -28,6 +28,7 @@
 
 #pragma once
 
+#include <atomic>
 #include <memory>
 #include <string>
 #include <vector>

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ target_include_directories(`
`36`	`36`	`$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>`
`37`	`37`	`$<INSTALL_INTERFACE:include>`
`38`	`38`	`)`
	`39`	`+target_link_libraries(${PROJECT_NAME} atomic)`
`39`	`40`	`ament_target_dependencies(`
`40`	`41`	`${PROJECT_NAME}`
`41`	`42`	`${THIS_PACKAGE_INCLUDE_DEPENDS}`