From 9b14d178c6673ef9b56c959b52868e54cc1199fe Mon Sep 17 00:00:00 2001 From: "Addisu Z. Taddese" Date: Mon, 17 Jul 2023 16:37:01 -0500 Subject: [PATCH] Address feedback Signed-off-by: Addisu Z. Taddese --- python/CMakeLists.txt | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/python/CMakeLists.txt b/python/CMakeLists.txt index 3054eb57..2501b04b 100644 --- a/python/CMakeLists.txt +++ b/python/CMakeLists.txt @@ -1,4 +1,7 @@ -# Append `_configured` to the file name so it doesn't interfere with tests +# Append `_configured` to the file name so it doesn't interfere with tests. +# This happens because pytest will load the `gz.msgs` package from the build directory +# (because there's an __init__.py file there) instead of being redirected to +# `gz.msgs10` in the install directory, which is the intent of this `__init__.py` file. set(python_init_file ${PROJECT_BINARY_DIR}/python/gz/${GS_DESIGNATION}/__init__.py_configured) configure_file(${PROJECT_SOURCE_DIR}/python/src/__init__.py.in ${python_init_file}) @@ -15,7 +18,6 @@ if (BUILD_TESTING AND NOT WIN32) if(${PYTEST_result} EQUAL 0) set(pytest_FOUND TRUE) else() - message("") message(WARNING "Pytest package not available: ${PYTEST_error}") message(WARNING "Output: ${PYTEST_output}") endif()