Skip to content

Commit

Permalink
Stop using python_cmake_module.
Browse files Browse the repository at this point in the history
We really don't need it anymore, and can just use the
builtin find_package(Python3).

Signed-off-by: Chris Lalancette <[email protected]>
  • Loading branch information
clalancette committed May 14, 2024
1 parent c0e59c4 commit 80e7ce6
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
9 changes: 0 additions & 9 deletions lttngpy/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,6 @@ if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
add_compile_options(-Wall -Wextra -Wpedantic)
endif()

# Figure out Python3 debug/release before anything else can find_package it
if(WIN32 AND CMAKE_BUILD_TYPE STREQUAL "Debug")
find_package(python_cmake_module REQUIRED)
find_package(PythonExtra REQUIRED)

# Force FindPython3 to use the debug interpreter where ROS 2 expects it
set(Python3_EXECUTABLE "${PYTHON_EXECUTABLE_DEBUG}")
endif()

find_package(ament_cmake REQUIRED)

if(WIN32 OR APPLE OR ANDROID)
Expand Down
1 change: 0 additions & 1 deletion lttngpy/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
<author email="[email protected]">Christophe Bedard</author>

<buildtool_depend>ament_cmake</buildtool_depend>
<buildtool_depend>python_cmake_module</buildtool_depend>

<depend>liblttng-ctl-dev</depend>

Expand Down

0 comments on commit 80e7ce6

Please sign in to comment.