Skip to content

Commit 35076b9

Browse files
committed
Remove include_directories in pcl_targets.cmake and update accordingly.
1 parent 4e06bd6 commit 35076b9

File tree

24 files changed

+63
-62
lines changed

24 files changed

+63
-62
lines changed

2d/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ set(impl_incs
3131

3232
set(LIB_NAME "pcl_${SUBSYS_NAME}")
3333
PCL_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME})
34+
target_link_libraries(${LIB_NAME} INTERFACE pcl_filters)
3435
PCL_MAKE_PKGCONFIG(${LIB_NAME} COMPONENT ${SUBSYS_NAME} DESC ${SUBSYS_DESC} PCL_DEPS ${SUBSYS_DEPS} HEADER_ONLY)
3536

3637
#Install include files

apps/3d_rec_framework/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ PCL_ADD_INCLUDES("${SUBSUBSYS_NAME}" "${SUBSYS_NAME}/${SUBSUBSYS_NAME}/pipeline/
8989

9090
set(LIB_NAME "pcl_${SUBSUBSYS_NAME}")
9191
PCL_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSUBSYS_NAME} SOURCES ${srcs} ${impl_incs_pipeline} ${incs_utils} ${incs_fw} ${incs_fw_global} ${incs_fw_local} ${incc_tools_framework} ${incs_pipelines} ${incs_pc_source})
92-
target_link_libraries("${LIB_NAME}" pcl_apps pcl_common pcl_io pcl_filters pcl_visualization pcl_segmentation pcl_surface pcl_features pcl_sample_consensus pcl_search pcl_registration)
92+
target_link_libraries("${LIB_NAME}" pcl_apps pcl_common pcl_io pcl_filters pcl_keypoints pcl_recognition pcl_visualization pcl_segmentation pcl_surface pcl_features pcl_sample_consensus pcl_search pcl_registration)
9393

9494
if(WITH_OPENNI)
9595
target_link_libraries("${LIB_NAME}" ${OPENNI_LIBRARIES})

apps/CMakeLists.txt

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ PCL_ADD_EXECUTABLE(pcl_test_search_speed COMPONENT ${SUBSYS_NAME} SOURCES src/te
4747
target_link_libraries(pcl_test_search_speed pcl_common pcl_io pcl_search pcl_kdtree pcl_visualization)
4848

4949
PCL_ADD_EXECUTABLE(pcl_nn_classification_example COMPONENT ${SUBSYS_NAME} SOURCES src/nn_classification_example.cpp)
50-
target_link_libraries(pcl_nn_classification_example pcl_common pcl_io pcl_features pcl_kdtree)
50+
target_link_libraries(pcl_nn_classification_example pcl_apps pcl_common pcl_io pcl_features pcl_kdtree)
5151
target_include_directories(pcl_nn_classification_example PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
5252

5353
PCL_ADD_EXECUTABLE(pcl_pyramid_surface_matching COMPONENT ${SUBSYS_NAME} SOURCES src/pyramid_surface_matching.cpp)
@@ -124,7 +124,7 @@ if(VTK_FOUND)
124124
src/pcd_video_player/pcd_video_player.ui
125125
BUNDLE)
126126

127-
target_link_libraries(pcl_pcd_video_player pcl_common pcl_io pcl_visualization pcl_segmentation pcl_features pcl_surface ${QTX}::Widgets)
127+
target_link_libraries(pcl_pcd_video_player pcl_common pcl_io pcl_registration pcl_visualization pcl_segmentation pcl_features pcl_surface ${QTX}::Widgets)
128128
target_include_directories(pcl_pcd_video_player PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
129129
endif()
130130

@@ -228,23 +228,23 @@ if(VTK_FOUND)
228228
endif() # QHULL_FOUND
229229

230230
PCL_ADD_EXECUTABLE(pcl_ni_agast COMPONENT ${SUBSYS_NAME} SOURCES src/ni_agast.cpp BUNDLE)
231-
target_link_libraries(pcl_ni_agast pcl_common pcl_io pcl_filters pcl_visualization pcl_segmentation pcl_sample_consensus pcl_features pcl_keypoints pcl_surface pcl_search)
231+
target_link_libraries(pcl_ni_agast pcl_apps pcl_common pcl_io pcl_filters pcl_visualization pcl_segmentation pcl_sample_consensus pcl_features pcl_keypoints pcl_surface pcl_search)
232232
target_include_directories(pcl_ni_agast PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
233233

234234
PCL_ADD_EXECUTABLE(pcl_ni_brisk COMPONENT ${SUBSYS_NAME} SOURCES src/ni_brisk.cpp BUNDLE)
235-
target_link_libraries(pcl_ni_brisk pcl_common pcl_io pcl_filters pcl_visualization pcl_segmentation pcl_sample_consensus pcl_features pcl_keypoints pcl_surface pcl_search)
235+
target_link_libraries(pcl_ni_brisk pcl_apps pcl_common pcl_io pcl_filters pcl_visualization pcl_segmentation pcl_sample_consensus pcl_features pcl_keypoints pcl_surface pcl_search)
236236
target_include_directories(pcl_ni_brisk PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
237237

238238
PCL_ADD_EXECUTABLE(pcl_ni_susan COMPONENT ${SUBSYS_NAME} SOURCES src/ni_susan.cpp BUNDLE)
239-
target_link_libraries(pcl_ni_susan pcl_common pcl_visualization pcl_features pcl_keypoints pcl_search)
239+
target_link_libraries(pcl_ni_susan pcl_apps pcl_common pcl_visualization pcl_features pcl_keypoints pcl_search)
240240
target_include_directories(pcl_ni_susan PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
241241

242242
PCL_ADD_EXECUTABLE(pcl_ni_trajkovic COMPONENT ${SUBSYS_NAME} SOURCES src/ni_trajkovic.cpp BUNDLE)
243-
target_link_libraries(pcl_ni_trajkovic pcl_common pcl_visualization pcl_features pcl_keypoints pcl_search)
243+
target_link_libraries(pcl_ni_trajkovic pcl_apps pcl_common pcl_visualization pcl_features pcl_keypoints pcl_search)
244244
target_include_directories(pcl_ni_trajkovic PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
245245

246246
PCL_ADD_EXECUTABLE(pcl_openni_klt COMPONENT ${SUBSYS_NAME} SOURCES src/openni_klt.cpp BUNDLE)
247-
target_link_libraries(pcl_openni_klt pcl_common pcl_io pcl_visualization pcl_tracking)
247+
target_link_libraries(pcl_openni_klt pcl_common pcl_io pcl_keypoints pcl_visualization pcl_tracking)
248248
endif() # WITH_OPENNI
249249
endif() # VTK_FOUND
250250

apps/cloud_composer/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
set(SUBSUBSYS_NAME cloud_composer)
77
set(SUBSUBSYS_DESC "Cloud Composer - Application for Manipulating Point Clouds")
8-
set(SUBSUBSYS_DEPS common io visualization filters apps)
8+
set(SUBSUBSYS_DEPS common io visualization features filters apps)
99
set(SUBSUBSYS_EXT_DEPS vtk ${QTX})
1010
set(REASON "")
1111
set(DEFAULT OFF)
@@ -71,7 +71,7 @@ set(PCL_LIB_TYPE STATIC)
7171
PCL_ADD_LIBRARY(pcl_cc_tool_interface COMPONENT ${SUBSUBSYS_NAME} SOURCES ${INTERFACE_HEADERS} ${INTERFACE_SOURCES})
7272

7373

74-
target_link_libraries(pcl_cc_tool_interface pcl_common pcl_filters pcl_search pcl_visualization ${QTX}::Widgets)
74+
target_link_libraries(pcl_cc_tool_interface pcl_common pcl_features pcl_filters pcl_search pcl_visualization ${QTX}::Widgets)
7575

7676
set(PCL_LIB_TYPE ${PCL_LIB_TYPE_ORIGIN})
7777

apps/in_hand_scanner/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
set(SUBSUBSYS_NAME in_hand_scanner)
22
set(SUBSUBSYS_DESC "In-hand scanner for small objects")
33
set(SUBSUBSYS_DEPS common features io kdtree apps)
4-
set(SUBSUBSYS_LIBS pcl_common pcl_features pcl_io pcl_kdtree)
4+
set(SUBSUBSYS_LIBS pcl_common pcl_geometry pcl_features pcl_io pcl_kdtree)
55
set(SUBSUBSYS_EXT_DEPS ${QTX} OpenGL OpenGL_GLU openni)
66
set(REASON "")
77
set(DEFAULT OFF)

apps/modeler/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ PCL_ADD_EXECUTABLE(
112112
${incs}
113113
${impl_incs})
114114

115-
target_link_libraries("${EXE_NAME}" pcl_common pcl_io pcl_kdtree pcl_filters pcl_visualization pcl_segmentation pcl_surface pcl_features pcl_sample_consensus pcl_search ${QTX}::Widgets)
115+
target_link_libraries("${EXE_NAME}" pcl_common pcl_io pcl_kdtree pcl_filters pcl_registration pcl_visualization pcl_segmentation pcl_surface pcl_features pcl_sample_consensus pcl_search ${QTX}::Widgets)
116116
target_include_directories(${EXE_NAME} PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/include)
117117

118118
# Install include files

cmake/pcl_targets.cmake

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,6 @@ macro(PCL_SUBSYS_DEPEND _var)
118118
if(NOT _status)
119119
set(${_var} FALSE)
120120
PCL_SET_SUBSYS_STATUS(${_name} FALSE "Requires ${_dep}.")
121-
else()
122-
PCL_GET_SUBSYS_INCLUDE_DIR(_include_dir ${_dep})
123-
include_directories(${PROJECT_SOURCE_DIR}/${_include_dir}/include)
124121
endif()
125122
endforeach()
126123
endif()
@@ -134,12 +131,6 @@ macro(PCL_SUBSYS_DEPEND _var)
134131
endif()
135132
endforeach()
136133
endif()
137-
if(ARGS_OPT_DEPS)
138-
foreach(_dep ${ARGS_OPT_DEPS})
139-
PCL_GET_SUBSYS_INCLUDE_DIR(_include_dir ${_dep})
140-
include_directories(${PROJECT_SOURCE_DIR}/${_include_dir}/include)
141-
endforeach()
142-
endif()
143134
endif()
144135
endmacro()
145136

@@ -266,30 +257,39 @@ function(PCL_CUDA_ADD_LIBRARY _name)
266257
endif()
267258

268259
REMOVE_VTK_DEFINITIONS()
260+
if(NOT ARGS_SOURCES)
261+
add_library(${_name} INTERFACE)
262+
263+
target_include_directories(${_name} INTERFACE
264+
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
265+
$<INSTALL_INTERFACE:${INCLUDE_INSTALL_ROOT}>
266+
)
269267

270-
add_library(${_name} ${PCL_LIB_TYPE} ${ARGS_SOURCES})
271-
272-
PCL_ADD_VERSION_INFO(${_name})
273-
274-
target_compile_options(${_name} PRIVATE $<$<COMPILE_LANGUAGE:CUDA>: ${GEN_CODE} --expt-relaxed-constexpr>)
275-
276-
target_include_directories(${_name} PUBLIC
277-
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
278-
$<INSTALL_INTERFACE:${INCLUDE_INSTALL_ROOT}>
279-
)
280-
281-
target_include_directories(${_name} PRIVATE ${CUDA_TOOLKIT_INCLUDE})
282-
283-
if(MSVC)
284-
target_link_libraries(${_name} delayimp.lib) # because delay load is enabled for openmp.dll
268+
else()
269+
add_library(${_name} ${PCL_LIB_TYPE} ${ARGS_SOURCES})
270+
271+
PCL_ADD_VERSION_INFO(${_name})
272+
273+
target_compile_options(${_name} PRIVATE $<$<COMPILE_LANGUAGE:CUDA>: ${GEN_CODE} --expt-relaxed-constexpr>)
274+
275+
target_include_directories(${_name} PUBLIC
276+
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
277+
$<INSTALL_INTERFACE:${INCLUDE_INSTALL_ROOT}>
278+
)
279+
280+
target_include_directories(${_name} PRIVATE ${CUDA_TOOLKIT_INCLUDE})
281+
282+
if(MSVC)
283+
target_link_libraries(${_name} delayimp.lib) # because delay load is enabled for openmp.dll
284+
endif()
285+
286+
set_target_properties(${_name} PROPERTIES
287+
VERSION ${PCL_VERSION}
288+
SOVERSION ${PCL_VERSION_MAJOR}.${PCL_VERSION_MINOR}
289+
DEFINE_SYMBOL "PCLAPI_EXPORTS")
290+
set_target_properties(${_name} PROPERTIES FOLDER "Libraries")
285291
endif()
286292

287-
set_target_properties(${_name} PROPERTIES
288-
VERSION ${PCL_VERSION}
289-
SOVERSION ${PCL_VERSION_MAJOR}.${PCL_VERSION_MINOR}
290-
DEFINE_SYMBOL "PCLAPI_EXPORTS")
291-
set_target_properties(${_name} PROPERTIES FOLDER "Libraries")
292-
293293
install(TARGETS ${_name}
294294
RUNTIME DESTINATION ${BIN_INSTALL_DIR} COMPONENT pcl_${ARGS_COMPONENT}
295295
LIBRARY DESTINATION ${LIB_INSTALL_DIR} COMPONENT pcl_${ARGS_COMPONENT}

cuda/common/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ set(common_incs
2828
)
2929

3030
set(LIB_NAME "pcl_${SUBSYS_NAME}")
31+
PCL_CUDA_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME})
3132
set(EXT_DEPS CUDA)
3233
PCL_MAKE_PKGCONFIG(${LIB_NAME} COMPONENT ${SUBSYS_NAME} DESC "${SUBSYS_DESC}"
3334
PCL_DEPS "${SUBSYS_DEPS}" EXT_DEPS "" HEADER_ONLY)

cuda/features/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ set(incs
2424

2525
set(LIB_NAME "pcl_${SUBSYS_NAME}")
2626
PCL_CUDA_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME} SOURCES ${srcs} ${incs})
27-
target_link_libraries(${LIB_NAME} pcl_common)
27+
target_link_libraries(${LIB_NAME} pcl_common pcl_io pcl_cuda_common)
2828

2929
set(EXT_DEPS "")
3030
#set(EXT_DEPS CUDA)

cuda/io/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ set(incs
3535

3636
set(LIB_NAME "pcl_${SUBSYS_NAME}")
3737
PCL_CUDA_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME} SOURCES ${srcs} ${incs})
38-
target_link_libraries(${LIB_NAME} pcl_common)
38+
target_link_libraries(${LIB_NAME} pcl_common pcl_io pcl_cuda_common)
3939

4040
PCL_MAKE_PKGCONFIG(${LIB_NAME} COMPONENT ${SUBSYS_NAME} DESC ${SUBSYS_DESC} PCL_DEPS ${SUBSYS_DEPS} EXT_DEPS ${SUBSYS_EXT_DEPS})
4141

0 commit comments

Comments
 (0)