Changeset View
Changeset View
Standalone View
Standalone View
CMakeLists.txt
Show All 11 Lines | |||||
12 | 12 | | |||
13 | include(ECMSetupVersion) | 13 | include(ECMSetupVersion) | ||
14 | include(FeatureSummary) | 14 | include(FeatureSummary) | ||
15 | include(KDEInstallDirs) | 15 | include(KDEInstallDirs) | ||
16 | include(KDECMakeSettings) | 16 | include(KDECMakeSettings) | ||
17 | include(KDECompilerSettings NO_POLICY_SCOPE) | 17 | include(KDECompilerSettings NO_POLICY_SCOPE) | ||
18 | include(KDEClangFormat) | 18 | include(KDEClangFormat) | ||
19 | 19 | | |||
20 | find_package(Qt5 ${QT_MIN_VERSION} CONFIG REQUIRED COMPONENTS Widgets DBus X11Extras) | 20 | find_package(Qt5 ${QT_MIN_VERSION} CONFIG REQUIRED COMPONENTS Widgets DBus Network X11Extras) | ||
anthonyfieroni: That's not right at least. It should be optional when KF5NetworkManagerQt is found. It looks… | |||||
Yes, this looks wrong but I don't understand why I even have to add this manually... NetworkManager-Qt CMake looks for Qt5 Network and it also has it in its PUBLIC link interface. broulik: Yes, this looks wrong but I don't understand why I even have to add this manually... | |||||
21 | find_package(KF5 ${KF5_MIN_VERSION} REQUIRED COMPONENTS Activities Auth IdleTime Config DBusAddons Solid I18n GlobalAccel KIO NotifyConfig Wayland DocTools Crash Notifications) | 21 | find_package(KF5 ${KF5_MIN_VERSION} REQUIRED COMPONENTS Activities Auth IdleTime Config DBusAddons Service Solid I18n GlobalAccel NotifyConfig Wayland DocTools Crash Notifications WidgetsAddons XmlGui) | ||
22 | find_package(KF5Screen CONFIG REQUIRED) | 22 | find_package(KF5Screen CONFIG REQUIRED) | ||
23 | find_package(LibKWorkspace CONFIG REQUIRED) | 23 | find_package(LibKWorkspace CONFIG REQUIRED) | ||
24 | 24 | | |||
25 | find_package(KF5BluezQt ${KF5_MIN_VERSION}) | 25 | find_package(KF5BluezQt ${KF5_MIN_VERSION}) | ||
26 | set_package_properties(KF5BluezQt | 26 | set_package_properties(KF5BluezQt | ||
27 | PROPERTIES DESCRIPTION "Qt wrapper for BlueZ 5 DBus API" | 27 | PROPERTIES DESCRIPTION "Qt wrapper for BlueZ 5 DBus API" | ||
28 | TYPE OPTIONAL | 28 | TYPE OPTIONAL | ||
29 | PURPOSE "Support for wireless energy saving actions" | 29 | PURPOSE "Support for wireless energy saving actions" | ||
▲ Show 20 Lines • Show All 56 Lines • Show Last 20 Lines |
That's not right at least. It should be optional when KF5NetworkManagerQt is found. It looks weird to present at all.