Make sure that tests use the module build directory but not install path
If we configure module that is already installed it's expected that tests use the build directory to resolve all the depdencies, but not the install directory. This is especially sensetive if the module cmake scripts were changed. This changes the order of find_<package|dependency> PATHS that are used to locate Qt package. QT_EXAMPLES_CMAKE_PREFIX_PATH now is used at first, so we guarantee that we look into the build directory at first place. This trick only works if build directory is added to CMAKE_FIND_ROOT_PATH. The reason for that is the internal CMake logic that tries to relocate the search PATHs and put the paths that are subdirs of or exact CMAKE_FIND_ROOT_PATH higher in search list. Fixes: QTBUG-115730 Change-Id: Icab721f0a6eac7301c626350ab214cc4545b368b Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
This commit is contained in:
parent
801b8304fb
commit
d4eda5d34d
@ -718,6 +718,7 @@ macro(qt_build_tests)
|
||||
|
||||
# Prepending to QT_BUILD_CMAKE_PREFIX_PATH helps find components of Qt6, because those
|
||||
# find_package calls use NO_DEFAULT_PATH, and thus CMAKE_PREFIX_PATH is ignored.
|
||||
list(PREPEND CMAKE_FIND_ROOT_PATH "${QT_BUILD_DIR}")
|
||||
list(PREPEND QT_BUILD_CMAKE_PREFIX_PATH "${QT_BUILD_DIR}/${INSTALL_LIBDIR}/cmake")
|
||||
|
||||
if(QT_BUILD_STANDALONE_TESTS)
|
||||
|
@ -134,9 +134,9 @@ foreach(module ${@INSTALL_CMAKE_NAMESPACE@_FIND_COMPONENTS})
|
||||
${@INSTALL_CMAKE_NAMESPACE@_FIND_VERSION}
|
||||
${_@INSTALL_CMAKE_NAMESPACE@_FIND_PARTS_QUIET}
|
||||
PATHS
|
||||
${QT_BUILD_CMAKE_PREFIX_PATH}
|
||||
${_qt_cmake_dir}
|
||||
${_qt_additional_packages_prefix_paths}
|
||||
${QT_BUILD_CMAKE_PREFIX_PATH}
|
||||
${__qt_find_package_host_qt_path}
|
||||
${_qt_additional_host_packages_prefix_paths}
|
||||
${__qt_use_no_default_path_for_qt_packages}
|
||||
|
@ -16,10 +16,10 @@ set(${CMAKE_FIND_PACKAGE_NAME}_FIND_REQUIRED FALSE)
|
||||
if(NOT @INSTALL_CMAKE_NAMESPACE@_FOUND)
|
||||
find_dependency(@INSTALL_CMAKE_NAMESPACE@ @main_qt_package_version@
|
||||
PATHS
|
||||
${QT_BUILD_CMAKE_PREFIX_PATH}
|
||||
"${CMAKE_CURRENT_LIST_DIR}/.."
|
||||
"${_qt_cmake_dir}"
|
||||
${_qt_additional_packages_prefix_paths}
|
||||
${QT_BUILD_CMAKE_PREFIX_PATH}
|
||||
${__qt_use_no_default_path_for_qt_packages}
|
||||
)
|
||||
endif()
|
||||
|
@ -112,9 +112,9 @@ macro(_qt_internal_find_qt_dependencies target target_dep_list find_dependency_p
|
||||
NAMES
|
||||
${__qt_${target}_pkg_names}
|
||||
PATHS
|
||||
${QT_BUILD_CMAKE_PREFIX_PATH}
|
||||
${${find_dependency_path_list}}
|
||||
${_qt_additional_packages_prefix_paths}
|
||||
${QT_BUILD_CMAKE_PREFIX_PATH}
|
||||
${__qt_use_no_default_path_for_qt_packages}
|
||||
)
|
||||
endif()
|
||||
|
Loading…
Reference in New Issue
Block a user