Use imported targets instead of qt5_use_modules

From Qt's documentation: "This macro is obsolete. Use
target_link_libraries with IMPORTED targets instead." It's only
recommended with cmake >=2.8.9 & < 2.8.12. Kube already requires
cmake 3.0.
One advantage of using the imported targets is, that
cmake complains if a target isn't found before it's used, like
Qt5Concurrent missing from the find_package_call here.

Reviewers: #kube, cmollekopf

Reviewed By: #kube, cmollekopf

Subscribers: #kube

Differential Revision: https://phabricator.kde.org/D6362

2 files changed, 15 insertions, 4 deletions

diff --git a/framework/src/domain/mime/mimetreeparser/tests/CMakeLists.txt b/framework/src/domain/mime/mimetreeparser/tests/CMakeLists.txt
index 89d97823..015efc32 100644
--- a/framework/src/domain/mime/mimetreeparser/tests/CMakeLists.txt
+++ b/framework/src/domain/mime/mimetreeparser/tests/CMakeLists.txt
@@ -10,8 +10,12 @@ include(ECMAddTests)
 
 add_executable(mimetreeparsertest interfacetest.cpp)
 add_gpg_crypto_test(mimetreeparsertest mimetreeparsertest)
-qt5_use_modules(mimetreeparsertest Core Test)
-target_link_libraries(mimetreeparsertest KF5::Mime kube_otp)
+target_link_libraries(mimetreeparsertest
+    kube_otp
+    Qt5::Core
+    Qt5::Test
+    KF5::Mime
+)
 
 find_package(Gpgmepp 1.7.1 CONFIG)
 find_package(QGpgme 1.7.1 CONFIG)
diff --git a/framework/src/domain/settings/CMakeLists.txt b/framework/src/domain/settings/CMakeLists.txt
index dc9d01b1..e9ea76c6 100644
--- a/framework/src/domain/settings/CMakeLists.txt
+++ b/framework/src/domain/settings/CMakeLists.txt
@@ -2,5 +2,12 @@ include_directories(${CMAKE_CURRENT_BINARY_DIR})
 cmake_policy(SET CMP0063 NEW)
 add_executable(sinkactiontest sinkactiontest.cpp)
 add_test(sinkactiontest sinkactiontest)
-qt5_use_modules(sinkactiontest Core Test Concurrent)
-target_link_libraries(sinkactiontest sink actionplugin KF5::Mime mailplugin)
+target_link_libraries(sinkactiontest
+    sink
+    actionplugin
+    mailplugin
+    Qt5::Core
+    Qt5::Concurrent
+    Qt5::Test
+    KF5::Mime
+)