Make kpmcore link libraries private.

src/CMakeLists.txt

@@ -40,13 +40,14 @@ ki18n_wrap_ui(kpmcore_SRCS ${gui_UIFILES})
 
 add_library(kpmcore SHARED ${kpmcore_SRCS})
 target_link_libraries( kpmcore PUBLIC
-    ${UUID_LIBRARIES}
+    PRIVATE
-    ${BLKID_LIBRARIES}
+    Qt5::Core
     Qt5::DBus
+    Qt5::Gui
     KF5::I18n
     KF5::CoreAddons
     KF5::WidgetsAddons
-    PRIVATE
+    ${BLKID_LIBRARIES}
     KF5::Auth
 )
 

src/fs/filesystem.cpp

@@ -31,6 +31,7 @@
 
 #include <KLocalizedString>
 
+#include <QColor>
 #include <QFileInfo>
 #include <QStandardPaths>
 #include <QStorageInfo>

src/fs/filesystem.h

@@ -22,7 +22,6 @@
 
 #include "util/libpartitionmanagerexport.h"
 
-#include <QColor>
 #include <QList>
 #include <QStringList>
 #include <QString>
@@ -31,6 +30,7 @@
 
 #include <array>
 
+class QColor;
 class QValidator;
 class Device;
 class Report;

src/fs/luks.cpp

@@ -39,6 +39,7 @@
 #include <QStorageInfo>
 #include <QString>
 #include <QUuid>
+#include <QWidget>
 
 #include <KLocalizedString>
 #include <KPasswordDialog>

src/fs/luks.h

@@ -26,11 +26,11 @@
 #include "fs/filesystem.h"
 
 #include <QtGlobal>
-#include <QWidget>
 
 class Report;
 
 class QString;
+class QWidget;
 
 namespace FS
 {

src/plugins/dummy/CMakeLists.txt

@@ -22,6 +22,6 @@ set (pmdummybackendplugin_SRCS
 
 add_library(pmdummybackendplugin SHARED ${pmdummybackendplugin_SRCS})
 
-target_link_libraries(pmdummybackendplugin kpmcore)
+target_link_libraries(pmdummybackendplugin kpmcore KF5::I18n KF5::CoreAddons)
 
 install(TARGETS pmdummybackendplugin DESTINATION ${KDE_INSTALL_PLUGINDIR})

src/plugins/sfdisk/CMakeLists.txt

@@ -22,6 +22,6 @@ set (pmsfdiskbackendplugin_SRCS
 
 add_library(pmsfdiskbackendplugin SHARED ${pmsfdiskbackendplugin_SRCS})
 
-target_link_libraries(pmsfdiskbackendplugin kpmcore)
+target_link_libraries(pmsfdiskbackendplugin kpmcore KF5::I18n KF5::CoreAddons)
 
 install(TARGETS pmsfdiskbackendplugin DESTINATION ${KDE_INSTALL_PLUGINDIR})

src/util/externalcommand.cpp

@@ -37,6 +37,7 @@
 #include <QVariant>
 
 #include <KAuth>
+#include <KJob>
 #include <KLocalizedString>
 
 /** Creates a new ExternalCommand instance without Report.

src/util/externalcommand.h

@@ -24,8 +24,6 @@
 #include "core/copysourcedevice.h"
 #include "core/copytargetfile.h"
 
-#include <KJob>
-
 #include <QDebug>
 #include <QProcess>
 #include <QString>
@@ -33,6 +31,7 @@
 #include <QtGlobal>
 #include <QVariant>
 
+class KJob;
 class Report;
 
 /** An external command.

test/CMakeLists.txt

@@ -20,7 +20,7 @@ target_link_libraries(testhelpers)
 
 macro (kpm_test name)
     add_executable(${name} ${ARGN})
-    target_link_libraries(${name} testhelpers kpmcore)
+    target_link_libraries(${name} testhelpers kpmcore Qt5::Core)
 endmacro()
 
 ###

test/testexternalcommand.cpp

@@ -22,7 +22,7 @@
 #include "backend/corebackendmanager.h"
 #include "util/externalcommand.h"
 
-#include <QApplication>
+#include <QCoreApplication>
 #include <QDebug>
 #include <QThread>
 
@@ -52,7 +52,7 @@ void run()
 
 int main( int argc, char **argv )
 {
-    QApplication app(argc, argv);
+    QCoreApplication app(argc, argv);
     KPMCoreInitializer i(QStringLiteral("pmsfdiskbackendplugin"));
 
     runcmd a;