QtGui is not needed in interfaces

interfaces/CMakeLists.txt

@@ -66,7 +66,6 @@ generate_export_header(kdeconnectinterfaces EXPORT_FILE_NAME ${CMAKE_CURRENT_BIN
 
 target_link_libraries(kdeconnectinterfaces
 LINK_PUBLIC
-    Qt5::Gui
     Qt5::DBus
 LINK_PRIVATE
     KF5::ConfigCore

interfaces/devicesmodel.cpp

@@ -26,7 +26,6 @@
 #include <QString>
 #include <QDBusInterface>
 #include <QDBusPendingReply>
-#include <QIcon>
 #include <QDBusServiceWatcher>
 
 #include "dbusinterfaces.h"
@@ -253,10 +252,6 @@ QVariant DevicesModel::data(const QModelIndex& index, int role) const
     switch (role) {
         case Qt::SizeHintRole:
             return QSize(0, 32);
-        case IconModelRole: {
-            QString icon = data(index, IconNameRole).toString();
-            return QIcon::fromTheme(icon);
-        }
         case IdModelRole:
             return device->id();
         case NameModelRole:

interfaces/devicesmodel.h

@@ -22,7 +22,6 @@
 #define DEVICESMODEL_H
 
 #include <QAbstractListModel>
-#include <QPixmap>
 #include <QList>
 
 #include "interfaces/kdeconnectinterfaces_export.h"

interfaces/notificationsmodel.cpp

@@ -24,8 +24,6 @@
 #include <QDebug>
 #include <QDBusInterface>
 
-#include <QIcon>
-
 //#include "modeltest.h"
 
 //In older Qt released, qAsConst isnt available
@@ -184,8 +182,6 @@ QVariant NotificationsModel::data(const QModelIndex& index, int role) const
 
     //FIXME: This function gets called lots of times, producing lots of dbus calls. Add a cache?
     switch (role) {
-        case IconModelRole:
-            return QIcon::fromTheme(QStringLiteral("device-notifier"));
         case IdModelRole:
             return notification->internalId();
         case NameModelRole:

interfaces/notificationsmodel.h

@@ -23,7 +23,6 @@
 
 #include <QAbstractItemModel>
 #include <QAbstractListModel>
-#include <QPixmap>
 #include <QList>
 
 #include "interfaces/dbusinterfaces.h"