Skip to content

Commit c9e6b65

Browse files
author
=
committed
Remove unnecessary code check for dependencies, using PCL_SUBSYS_DEPEND instead.
1 parent 1276628 commit c9e6b65

File tree

6 files changed

+12
-28
lines changed

6 files changed

+12
-28
lines changed

cuda/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ set(SUBSYS_NAME cuda)
22
set(SUBSYS_DESC "Point cloud CUDA libraries")
33
set(SUBSYS_DEPS)
44

5-
option(BUILD_CUDA "Build the CUDA-related subsystems" ${DEFAULT})
5+
option(BUILD_CUDA "Build the CUDA-related subsystems" OFF)
66

77
if(NOT (BUILD_CUDA AND CUDA_FOUND))
88
return()

examples/CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,7 @@ set(SUBSYS_NAME examples)
22
set(SUBSYS_DESC "PCL examples")
33
set(SUBSYS_DEPS common io features search kdtree octree filters keypoints segmentation sample_consensus outofcore stereo geometry surface)
44

5-
set(DEFAULT FALSE)
6-
set(REASON "Code examples are disabled by default.")
7-
PCL_SUBSYS_OPTION(build ${SUBSYS_NAME} ${SUBSYS_DESC} ${DEFAULT} ${REASON})
5+
PCL_SUBSYS_OPTION(build ${SUBSYS_NAME} ${SUBSYS_DESC} OFF)
86
PCL_SUBSYS_DEPEND(build NAME ${SUBSYS_NAME} DEPS ${SUBSYS_DEPS})
97

108
if(NOT build)

gpu/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ set(SUBSYS_NAME gpu)
22
set(SUBSYS_DESC "Point cloud GPU libraries")
33
set(SUBSYS_DEPS)
44

5-
option(BUILD_GPU "Build the GPU-related subsystems" ${DEFAULT})
5+
option(BUILD_GPU "Build the GPU-related subsystems" OFF)
66

77
if(NOT (BUILD_GPU AND CUDA_FOUND))
88
return()

people/CMakeLists.txt

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,10 @@
11
set(SUBSYS_NAME people)
22
set(SUBSYS_DESC "Point cloud people library")
33
set(SUBSYS_DEPS common kdtree search sample_consensus filters io visualization geometry segmentation octree)
4-
5-
if(NOT VTK_FOUND)
6-
set(DEFAULT FALSE)
7-
set(REASON "VTK was not found.")
8-
else()
9-
set(DEFAULT TRUE)
10-
set(REASON)
11-
include_directories("${CMAKE_CURRENT_SOURCE_DIR}/include")
12-
endif()
4+
set(EXT_DEPS vtk)
135

146
PCL_SUBSYS_OPTION(build "${SUBSYS_NAME}" "${SUBSYS_DESC}" ON)
15-
PCL_SUBSYS_DEPEND(build NAME ${SUBSYS_NAME} DEPS ${SUBSYS_DEPS})
7+
PCL_SUBSYS_DEPEND(build NAME ${SUBSYS_NAME} DEPS ${SUBSYS_DEPS} EXT_DEPS ${EXT_DEPS})
168

179
PCL_ADD_DOC("${SUBSYS_NAME}")
1810

tools/CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,8 @@ set(SUBSYS_NAME tools)
22
set(SUBSYS_DESC "Useful PCL-based command line tools")
33
set(SUBSYS_DEPS io)
44
set(SUBSYS_OPT_DEPS filters sample_consensus segmentation search kdtree features surface octree registration recognition geometry keypoints ml visualization vtk)
5-
set(DEFAULT ON)
6-
set(REASON "")
75

8-
PCL_SUBSYS_OPTION(build ${SUBSYS_NAME} ${SUBSYS_DESC} ${DEFAULT} ${REASON})
6+
PCL_SUBSYS_OPTION(build ${SUBSYS_NAME} ${SUBSYS_DESC} ON)
97
PCL_SUBSYS_DEPEND(build NAME ${SUBSYS_NAME} DEPS ${SUBSYS_DEPS} OPT_DEPS ${SUBSYS_OPT_DEPS})
108

119
if(NOT build)

visualization/CMakeLists.txt

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,7 @@ set(SUBSYS_NAME visualization)
22
set(SUBSYS_DESC "Point cloud visualization library")
33
set(SUBSYS_DEPS common io kdtree geometry search octree)
44

5-
if(NOT VTK_FOUND)
6-
set(DEFAULT FALSE)
7-
set(REASON "VTK was not found.")
8-
else()
9-
set(DEFAULT TRUE)
10-
set(REASON)
11-
include_directories("${CMAKE_CURRENT_SOURCE_DIR}/include")
12-
endif()
13-
14-
PCL_SUBSYS_OPTION(build "${SUBSYS_NAME}" "${SUBSYS_DESC}" ${DEFAULT} "${REASON}")
5+
PCL_SUBSYS_OPTION(build "${SUBSYS_NAME}" "${SUBSYS_DESC}" ON)
156
PCL_SUBSYS_DEPEND(build NAME ${SUBSYS_NAME} DEPS ${SUBSYS_DEPS} EXT_DEPS vtk OPT_DEPS openni openni2 ensenso davidSDK dssdk rssdk)
167

178
if(ANDROID)
@@ -147,6 +138,11 @@ endif()
147138
set(LIB_NAME "pcl_${SUBSYS_NAME}")
148139
PCL_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME} SOURCES ${srcs} ${incs} ${common_incs} ${impl_incs} ${common_impl_incs} ${vtk_incs})
149140

141+
target_include_directories(${LIB_NAME} PUBLIC
142+
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
143+
$<INSTALL_INTERFACE:include>
144+
)
145+
150146
# apple workaround (continued)
151147
if(APPLE)
152148
target_link_libraries("${LIB_NAME}" "-framework Cocoa")

0 commit comments

Comments
 (0)