Skip to content

Conversation

dirk-thomas
Copy link
Member

@dirk-thomas dirk-thomas commented Jul 7, 2020

Fix a test which has been failing on Windows debug since it was added in #236: Build Status

The same logic already happens in the CMake extra file of the package (

# Provides PYTHON_EXECUTABLE_DEBUG
find_package(python_cmake_module REQUIRED)
find_package(PythonExtra REQUIRED)
) but isn't available when the test within the package is run.

Windows debug build testing launch_testing_ament_cmake: Build Status

@dirk-thomas dirk-thomas added the bug Something isn't working label Jul 7, 2020
@dirk-thomas dirk-thomas requested a review from hidmic July 7, 2020 18:38
@dirk-thomas dirk-thomas self-assigned this Jul 7, 2020
@dirk-thomas dirk-thomas force-pushed the dirk-thomas/fix-windows-debug-test branch from a08ab21 to 79be16d Compare July 7, 2020 22:03
cmake_minimum_required(VERSION 3.5)

project(launch_testing_ament_cmake NONE)
project(launch_testing_ament_cmake)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is necessary for FindPythonLibs to determine the Python libraries and include dirs.

@dirk-thomas
Copy link
Member Author

CI builds testing launch_testing_ament_cmake:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@dirk-thomas dirk-thomas merged commit 24ae132 into master Jul 7, 2020
@delete-merged-branch delete-merged-branch bot deleted the dirk-thomas/fix-windows-debug-test branch July 7, 2020 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants