Skip to content

Commit

Permalink
Add a python3-dev dependency to tf2_py. (#733)
Browse files Browse the repository at this point in the history
This is needed because it depends on Python.h being
available, and it also calls find_package(Python3 Development).

While we are in here, we also remove "Development" from the
find_package(Python3) calls in tf2_geometry_msgs and tf2_sensor_msgs.
In both of those cases, we only need the interpreter, not
the header files.

Signed-off-by: Chris Lalancette <[email protected]>
  • Loading branch information
clalancette authored Nov 22, 2024
1 parent 1377a24 commit 714f456
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tf2_geometry_msgs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ endif()
cmake_policy(SET CMP0094 NEW)
set(Python3_FIND_UNVERSIONED_NAMES FIRST)

find_package(Python3 REQUIRED COMPONENTS Interpreter Development)
find_package(Python3 REQUIRED COMPONENTS Interpreter)

find_package(geometry_msgs REQUIRED)
find_package(orocos_kdl_vendor REQUIRED)
Expand Down
1 change: 1 addition & 0 deletions tf2_py/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
<buildtool_depend>ament_cmake</buildtool_depend>

<build_depend>geometry_msgs</build_depend>
<build_depend>python3-dev</build_depend>

<depend>tf2</depend>

Expand Down
2 changes: 1 addition & 1 deletion tf2_sensor_msgs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ endif()
cmake_policy(SET CMP0094 NEW)
set(Python3_FIND_UNVERSIONED_NAMES FIRST)

find_package(Python3 REQUIRED COMPONENTS Interpreter Development)
find_package(Python3 REQUIRED COMPONENTS Interpreter)

find_package(eigen3_cmake_module REQUIRED)
find_package(Eigen3 REQUIRED)
Expand Down

0 comments on commit 714f456

Please sign in to comment.