Control: tags -1 + patch I made some more minor tweaks to v2:
- Update debian/changelog - Drop debian/patches/add_glib_for_nm, no longer necessary v3 of the debdiff is attached. I was told that it's probably not worth forwarding this upstream, as KDE4 development is pretty much dormant. I plan to NMU kde-runtime in a week or so and upload to DELAYED/10 unless I hear back from you. Please holler, if you have any objections. Regards, Michael -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth?
diff -Nru kde-runtime-17.08.3/debian/changelog kde-runtime-17.08.3/debian/changelog --- kde-runtime-17.08.3/debian/changelog 2018-01-11 09:22:25.000000000 +0100 +++ kde-runtime-17.08.3/debian/changelog 2018-04-14 15:02:04.000000000 +0200 @@ -1,3 +1,12 @@ +kde-runtime (4:17.08.3-1.1) UNRELEASED; urgency=medium + + * Non-maintainer upload. + * Update solid-networkstatus to use libnm. (Closes: #862883) + * Drop debian/patches/add_glib_for_nm, no longer necessary with the above + changes. + + -- Michael Biebl <bi...@debian.org> Sat, 14 Apr 2018 13:02:04 +0000 + kde-runtime (4:17.08.3-1) unstable; urgency=medium * Team upload. diff -Nru kde-runtime-17.08.3/debian/control kde-runtime-17.08.3/debian/control --- kde-runtime-17.08.3/debian/control 2018-01-06 11:49:18.000000000 +0100 +++ kde-runtime-17.08.3/debian/control 2018-04-14 15:02:04.000000000 +0200 @@ -24,7 +24,7 @@ libjpeg-dev, libkactivities-dev, liblzma-dev, - libnm-util-dev (>= 0.7) [linux-any], + libnm-dev (>= 1.0.0) [linux-any], libntrack-qt4-dev [linux-any], libopenexr-dev, libphonon-dev (>= 4:4.6.0really4.4.3), @@ -35,7 +35,6 @@ libwebp-dev, libx11-dev, libxcursor-dev, - network-manager-dev (>= 0.7.0) [linux-any], pkg-config, pkg-kde-tools (>= 0.12), shared-mime-info, diff -Nru kde-runtime-17.08.3/debian/patches/add_glib_for_nm kde-runtime-17.08.3/debian/patches/add_glib_for_nm --- kde-runtime-17.08.3/debian/patches/add_glib_for_nm 2017-05-22 19:34:05.000000000 +0200 +++ kde-runtime-17.08.3/debian/patches/add_glib_for_nm 1970-01-01 01:00:00.000000000 +0100 @@ -1,25 +0,0 @@ -From: Debian/Kubuntu Qt/KDE Maintainers <debian-qt-kde@lists.debian.org> -Date: Sat, 18 Jun 2016 16:24:29 +0200 -Subject: add_glib_for_nm - ---- - solid-networkstatus/kded/CMakeLists.txt | 6 ++++++ - 1 file changed, 6 insertions(+) - -diff --git a/solid-networkstatus/kded/CMakeLists.txt b/solid-networkstatus/kded/CMakeLists.txt -index 633c7f1394..5dfcbf4f8d 100644 ---- a/solid-networkstatus/kded/CMakeLists.txt -+++ b/solid-networkstatus/kded/CMakeLists.txt -@@ -16,6 +16,12 @@ set_package_properties(NetworkManager PROPERTIES DESCRIPTION "The NetworkManager - ) - - if(NETWORKMANAGER_FOUND) -+ find_package(PkgConfig REQUIRED) -+ pkg_check_modules(GLIB_PKG glib-2.0) -+ -+ if (GLIB_PKG_FOUND) -+ include_directories(${GLIB_PKG_INCLUDE_DIRS}) -+ endif() - macro_ensure_version("0.7.0" ${NETWORKMANAGER_VERSION} NM_0_7) - if(NM_0_7) - MESSAGE(STATUS "Adding in-process NetworkManager service-wart to kded module") diff -Nru kde-runtime-17.08.3/debian/patches/cmake-add-NetworkManager-module.patch kde-runtime-17.08.3/debian/patches/cmake-add-NetworkManager-module.patch --- kde-runtime-17.08.3/debian/patches/cmake-add-NetworkManager-module.patch 1970-01-01 01:00:00.000000000 +0100 +++ kde-runtime-17.08.3/debian/patches/cmake-add-NetworkManager-module.patch 2018-04-14 15:02:04.000000000 +0200 @@ -0,0 +1,56 @@ +From 5711ab8b4b053e5d2f4f0b5831ff2a1d37cdf9fa Mon Sep 17 00:00:00 2001 +From: Lubomir Rintel <lkund...@v3.sk> +Date: Tue, 19 Dec 2017 19:45:23 +0100 +Subject: [PATCH] cmake: add NetworkManager module + +This one looks for libnm as opposed to the deprecated libnm-glib. +--- + cmake/modules/FindNetworkManager.cmake | 40 ++++++++++++++++++++++++++++++++++ + 1 file changed, 40 insertions(+) + create mode 100644 cmake/modules/FindNetworkManager.cmake + +Index: kde-runtime-17.08.3/cmake/modules/FindNetworkManager.cmake +=================================================================== +--- /dev/null ++++ kde-runtime-17.08.3/cmake/modules/FindNetworkManager.cmake +@@ -0,0 +1,40 @@ ++# - Try to find NetworkManager ++# Once done this will define ++# ++# NETWORKMANAGER_FOUND - system has NetworkManager ++# NETWORKMANAGER_INCLUDE_DIRS - the NetworkManager include directories ++# NETWORKMANAGER_LIBRARIES - the libraries needed to use NetworkManager ++# NETWORKMANAGER_CFLAGS - Compiler switches required for using NetworkManager ++# NETWORKMANAGER_VERSION - version number of NetworkManager ++ ++# Copyright (c) 2006, Alexander Neundorf, <neund...@kde.org> ++# Copyright (c) 2007, Will Stephenson, <wstephen...@kde.org> ++# ++# Redistribution and use is allowed according to the terms of the BSD license. ++# For details see the accompanying COPYING-CMAKE-SCRIPTS file. ++ ++ ++IF (NETWORKMANAGER_INCLUDE_DIRS) ++ # in cache already ++ SET(NetworkManager_FIND_QUIETLY TRUE) ++ENDIF (NETWORKMANAGER_INCLUDE_DIRS) ++ ++IF (NOT WIN32) ++ # use pkg-config to get the directories and then use these values ++ # in the FIND_PATH() and FIND_LIBRARY() calls ++ find_package(PkgConfig) ++ PKG_SEARCH_MODULE( NETWORKMANAGER libnm ) ++ENDIF (NOT WIN32) ++ ++IF (NETWORKMANAGER_FOUND) ++ IF (NOT NetworkManager_FIND_QUIETLY) ++ MESSAGE(STATUS "Found libnm ${NETWORKMANAGER_VERSION}: ${NETWORKMANAGER_LIBRARY_DIRS}") ++ ENDIF (NOT NetworkManager_FIND_QUIETLY) ++ELSE (NETWORKMANAGER_FOUND) ++ IF (NetworkManager_FIND_REQUIRED) ++ MESSAGE(FATAL_ERROR "Could NOT find libnm, check FindPkgConfig output above!") ++ ENDIF (NetworkManager_FIND_REQUIRED) ++ENDIF (NETWORKMANAGER_FOUND) ++ ++MARK_AS_ADVANCED(NETWORKMANAGER_INCLUDE_DIRS) ++ diff -Nru kde-runtime-17.08.3/debian/patches/series kde-runtime-17.08.3/debian/patches/series --- kde-runtime-17.08.3/debian/patches/series 2017-05-22 19:34:05.000000000 +0200 +++ kde-runtime-17.08.3/debian/patches/series 2018-04-14 15:02:04.000000000 +0200 @@ -5,6 +5,7 @@ disable_kwalletd_autotests kubuntu_nodisplay_knetattach.diff kubuntu_shutup_shutup_shutup.diff -add_glib_for_nm hurd.diff Make-sure-people-are-not-trying-to-sneak-invisible-charac.patch +cmake-add-NetworkManager-module.patch +solid-networkstatus-Port-to-libnm.patch diff -Nru kde-runtime-17.08.3/debian/patches/solid-networkstatus-Port-to-libnm.patch kde-runtime-17.08.3/debian/patches/solid-networkstatus-Port-to-libnm.patch --- kde-runtime-17.08.3/debian/patches/solid-networkstatus-Port-to-libnm.patch 1970-01-01 01:00:00.000000000 +0100 +++ kde-runtime-17.08.3/debian/patches/solid-networkstatus-Port-to-libnm.patch 2018-04-14 15:02:04.000000000 +0200 @@ -0,0 +1,91 @@ +From 42df200733673c9385c043e876cccfebd6c616ce Mon Sep 17 00:00:00 2001 +From: Michael Biebl <bi...@debian.org> +Date: Wed, 18 Apr 2018 19:08:05 +0200 +Subject: [PATCH 2/2] solid-networkstatus: Port to libnm + +libnm-util has been deprecated for three years and is eventually going +to disappear. +As libnm has been introduced in NetworkManager 1.0.0, the version checks +are no longer necessary, so drop them. +The NetworkManager.h include is replaced with nm-dbus-interface.h to not +pull in any glib/gio headers. +--- + solid-networkstatus/kded/CMakeLists.txt | 13 +++++-------- + solid-networkstatus/kded/networkmanagerstatus.cpp | 12 +----------- + 2 files changed, 6 insertions(+), 19 deletions(-) + +diff --git a/solid-networkstatus/kded/CMakeLists.txt b/solid-networkstatus/kded/CMakeLists.txt +index 633c7f1394..55ef67158e 100644 +--- a/solid-networkstatus/kded/CMakeLists.txt ++++ b/solid-networkstatus/kded/CMakeLists.txt +@@ -8,7 +8,7 @@ set(kded_networkstatus_PART_SRCS + wicdcustomtypes.cpp + ) + +-macro_optional_find_package(NetworkManager) ++macro_optional_find_package(NetworkManager 1.0.0) + set_package_properties(NetworkManager PROPERTIES DESCRIPTION "The NetworkManager headers" + URL "http://projects.gnome.org/NetworkManager" + TYPE OPTIONAL +@@ -16,13 +16,10 @@ set_package_properties(NetworkManager PROPERTIES DESCRIPTION "The NetworkManager + ) + + if(NETWORKMANAGER_FOUND) +- macro_ensure_version("0.7.0" ${NETWORKMANAGER_VERSION} NM_0_7) +- if(NM_0_7) +- MESSAGE(STATUS "Adding in-process NetworkManager service-wart to kded module") +- set(kded_networkstatus_PART_SRCS ${kded_networkstatus_PART_SRCS} networkmanagerstatus.cpp) +- include_directories(${NETWORKMANAGER_INCLUDE_DIRS}) +- add_definitions(-DNM_BACKEND_ENABLED) +- endif(NM_0_7) ++ MESSAGE(STATUS "Adding in-process NetworkManager service-wart to kded module") ++ set(kded_networkstatus_PART_SRCS ${kded_networkstatus_PART_SRCS} networkmanagerstatus.cpp) ++ include_directories(${NETWORKMANAGER_INCLUDE_DIRS}) ++ add_definitions(-DNM_BACKEND_ENABLED) + endif(NETWORKMANAGER_FOUND) + + macro_optional_find_package(QNtrack) +diff --git a/solid-networkstatus/kded/networkmanagerstatus.cpp b/solid-networkstatus/kded/networkmanagerstatus.cpp +index 2a42684cf9..a5f4fcedc4 100644 +--- a/solid-networkstatus/kded/networkmanagerstatus.cpp ++++ b/solid-networkstatus/kded/networkmanagerstatus.cpp +@@ -28,11 +28,7 @@ + + #include <QtDBus/QDBusReply> + +-#include <NetworkManager.h> +- +-#if !defined(NM_CHECK_VERSION) +- #define NM_CHECK_VERSION(x,y,z) 0 +-#endif ++#include <nm-dbus-interface.h> + + NetworkManagerStatus::NetworkManagerStatus( QObject *parent ) + : SystemStatusInterface( parent ), +@@ -85,23 +81,17 @@ Solid::Networking::Status NetworkManagerStatus::convertNmState( uint nmState ) + case NM_STATE_CONNECTING: + status = Solid::Networking::Connecting; + break; +-#if NM_CHECK_VERSION(0,8,992) + case NM_STATE_CONNECTED_LOCAL: + case NM_STATE_CONNECTED_SITE: + case NM_STATE_CONNECTED_GLOBAL: +-#else +- case NM_STATE_CONNECTED: +-#endif + status = Solid::Networking::Connected; + break; + case NM_STATE_DISCONNECTED: + status = Solid::Networking::Unconnected; + break; +-#if NM_CHECK_VERSION(0,8,992) + case NM_STATE_DISCONNECTING: + status = Solid::Networking::Disconnecting; + break; +-#endif + } + + return status; +-- +2.17.0 +
signature.asc
Description: OpenPGP digital signature