diff options
author | Alexey Edelev <alexey.edelev@qt.io> | 2022-12-08 20:13:58 +0100 |
---|---|---|
committer | Alexey Edelev <alexey.edelev@qt.io> | 2022-12-09 12:06:46 +0100 |
commit | e85b0ad5676e9a70111d27e7a66b1b074425a0f9 (patch) | |
tree | 5238c0c7a7ad8814b0c2d632dd14279e8d59dddf /cmake | |
parent | d4ebb921783d1350c3b1f6602c299beb27aba83e (diff) | |
download | qtbase-e85b0ad5676e9a70111d27e7a66b1b074425a0f9.tar.gz |
Remove CMake version check when adding _lib_pri to ALL
It looks like adding dependencies to interface libraries is either not
fixed or broken in CMake versions newer than 3.20. Remove the CMake
version check to ensure that 'lib_pri' targets are executed. This will
restore the initial behavior.
Amends cfcc4ef8edc4e8256ae738bbe8c23b4204021830
Fixes: QTBUG-109240
Fixes: QTBUG-109239
Change-Id: I8c5b317fcdd0a715a1a668b4e955df1acfe4be8b
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
(cherry picked from commit 85b941f4251bb5207db65efc9ccdd1f5586c8d11)
Diffstat (limited to 'cmake')
-rw-r--r-- | cmake/QtPriHelpers.cmake | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/cmake/QtPriHelpers.cmake b/cmake/QtPriHelpers.cmake index 4b34c44fca..b8c238c203 100644 --- a/cmake/QtPriHelpers.cmake +++ b/cmake/QtPriHelpers.cmake @@ -532,11 +532,10 @@ QT.${config_module_name}_private.disabled_features = ${disabled_private_features "-DIMPLICIT_LINK_DIRECTORIES=${implicit_link_directories}" -P "${QT_CMAKE_DIR}/QtGenerateLibPri.cmake" VERBATIM) - # In CMake versions older than 3.20 add_dependencies have no effect when adding interface - # libraries. So need to add the '_lib_pri' targets to ALL to make sure that the related - # rules executed. + # add_dependencies has no effect when adding interface libraries. So need to add the + # '_lib_pri' targets to ALL to make sure that the related rules executed. unset(add_pri_target_to_all) - if(CMAKE_VERSION VERSION_LESS 3.20 AND arg_HEADER_MODULE) + if(arg_HEADER_MODULE) set(add_pri_target_to_all ALL) endif() add_custom_target(${target}_lib_pri ${add_pri_target_to_all} |