Skip to content

Commit 904a37f

Browse files
traversaroSteveMacenski
authored andcommitted
Fix build with statically-linked Python interpreter and bump minimum required version to CMake 3.16 (#325)
* Fix build with statically-linked Python interpreter * Bump minimum required version of CMake to 3.16
1 parent f1b13f9 commit 904a37f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

spatio_temporal_voxel_layer/CMakeLists.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
cmake_minimum_required(VERSION 3.5)
1+
cmake_minimum_required(VERSION 3.16)
22
project(spatio_temporal_voxel_layer)
33

44
if(NOT WIN32)
@@ -79,9 +79,6 @@ rosidl_generate_interfaces(${PROJECT_NAME}
7979
)
8080
rosidl_get_typesupport_target(cpp_typesupport_target "${PROJECT_NAME}" "rosidl_typesupport_cpp")
8181

82-
# Get a linker error when there are undefined symbols
83-
add_link_options(-Wl,--no-undefined)
84-
8582
include_directories(
8683
include
8784
${BOOST_INCLUDE_DIRS}
@@ -109,6 +106,9 @@ target_link_libraries(${library_name}
109106
OpenVDB::openvdb
110107
)
111108

109+
# Get a linker error when there are undefined symbols
110+
target_link_options(${library_name} PRIVATE -Wl,--no-undefined)
111+
112112
ament_target_dependencies(${library_name} ${dependencies})
113113

114114
if(BUILD_TESTING)

0 commit comments

Comments
 (0)