Skip to content

Commit 74686db

Browse files
facontidavideclaude
andcommitted
build: replace pcl_ros with direct PCL component deps and fix ament exports
- Use find_package(PCL COMPONENTS common io) instead of pcl_ros - Link against PCL_COMMON_LIBRARIES/PCL_IO_LIBRARIES instead of PCL_LIBRARIES - Fix ament_export_targets with EXPORT for proper downstream consumption - Update package.xml deps: libpcl-common + libpcl-io replace pcl_ros - Remove redundant PCL_INCLUDE_DIRS from target_include_directories Based on improvements from PR #55. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent aa947ab commit 74686db

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

cloudini_ros/CMakeLists.txt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,6 @@ target_link_libraries(test_cloudini_subscriber
107107
${PCL_COMMON_LIBRARIES}
108108
)
109109

110-
target_include_directories(test_cloudini_subscriber
111-
PRIVATE
112-
${PCL_INCLUDE_DIRS}
113-
)
114-
115110
install(TARGETS test_cloudini_subscriber
116111
ARCHIVE DESTINATION lib
117112
LIBRARY DESTINATION lib

0 commit comments

Comments
 (0)