Hello community, here is the log from the commit of package libdbusmenu-qt5 for openSUSE:Factory checked in at 2020-09-23 18:35:51 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libdbusmenu-qt5 (Old) and /work/SRC/openSUSE:Factory/.libdbusmenu-qt5.new.4249 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "libdbusmenu-qt5" Wed Sep 23 18:35:51 2020 rev:9 rq:833992 version:0.9.3+16.04.20160218 Changes: -------- --- /work/SRC/openSUSE:Factory/libdbusmenu-qt5/libdbusmenu-qt5.changes 2015-07-14 17:20:10.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.libdbusmenu-qt5.new.4249/libdbusmenu-qt5.changes 2020-09-23 18:36:01.637088126 +0200 @@ -1,0 +2,14 @@ +Wed Sep 9 06:29:29 UTC 2020 - Antonio Larrosa <alarr...@suse.com> + +- Rework patch trusting the compiler will remove dead code properly + when debug output is disabled (as it is in the spec file): + * noqDebug-qWarnings.patch + +------------------------------------------------------------------- +Fri Sep 4 16:45:22 UTC 2020 - Antonio Larrosa <alarr...@suse.com> + +- Update to 0.9.3+16.04.20160218: + * Ported tests to Qt5 +- Use a Source url that works + +------------------------------------------------------------------- Old: ---- libdbusmenu-qt_0.9.3+15.10.20150604.orig.tar.gz New: ---- 0.9.3+16.04.20160218-0ubuntu1.tar.gz ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libdbusmenu-qt5.spec ++++++ --- /var/tmp/diff_new_pack.R1LTmR/_old 2020-09-23 18:36:04.893090963 +0200 +++ /var/tmp/diff_new_pack.R1LTmR/_new 2020-09-23 18:36:04.893090963 +0200 @@ -1,7 +1,7 @@ # # spec file for package libdbusmenu-qt5 # -# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany. +# Copyright (c) 2020 SUSE LLC # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -12,20 +12,21 @@ # license that conforms to the Open Source Definition (Version 1.9) # published by the Open Source Initiative. -# Please submit bugfixes or comments via http://bugs.opensuse.org/ +# Please submit bugfixes or comments via https://bugs.opensuse.org/ # %define rname libdbusmenu-qt +%define version_ %(echo %{version} | sed -e "s/+/-/") Name: libdbusmenu-qt5 -Version: 0.9.3+15.10.20150604 +Version: 0.9.3+16.04.20160218 Release: 0 -Url: https://launchpad.net/libdbusmenu-qt/ +URL: https://launchpad.net/libdbusmenu-qt/ Summary: A Qt implementation of the DBusMenu protocol -License: LGPL-2.0+ +License: LGPL-2.0-or-later Group: System/Libraries -Source: http://archive.ubuntu.com/ubuntu/pool/main/libd/%{rname}/%{rname}_%{version}.orig.tar.gz +Source: https://github.com/unity8-team/%{rname}/archive/%{version}-0ubuntu1.tar.gz Source1: baselibs.conf # PATCH-FIX-UPSTREAM noqDebug-qWarnings.patch -- libdbusmenu uses it's own qDebug's and qWarnings, # which are useless, and annoy users, so this patch just disables them in release mode @@ -69,7 +70,7 @@ This package contains development files for libdbusmenu-qt5. %prep -%setup -q -n %{rname}-%{version} +%setup -q -n %{rname}-%{version_}-0ubuntu1 %patch1 -p1 %if 0%{?suse_version} <= 1310 %patch2 -p1 ++++++ noqDebug-qWarnings.patch ++++++ --- /var/tmp/diff_new_pack.R1LTmR/_old 2020-09-23 18:36:04.933090997 +0200 +++ /var/tmp/diff_new_pack.R1LTmR/_new 2020-09-23 18:36:04.933090997 +0200 @@ -1,163 +1,29 @@ -diff --git a/CMakeLists.txt b/CMakeLists.txt -index 0ee9154..597a7de 100644 ---- a/CMakeLists.txt -+++ b/CMakeLists.txt -@@ -91,7 +91,7 @@ install(FILES ${CMAKE_BINARY_DIR}/dbusmenu-${QT_SUFFIX}.pc +Index: libdbusmenu-qt-0.9.3-16.04.20160218-0ubuntu1/CMakeLists.txt +=================================================================== +--- libdbusmenu-qt-0.9.3-16.04.20160218-0ubuntu1.orig/CMakeLists.txt ++++ libdbusmenu-qt-0.9.3-16.04.20160218-0ubuntu1/CMakeLists.txt +@@ -107,7 +107,7 @@ install(FILES ${CMAKE_BINARY_DIR}/dbusme + ) add_subdirectory(src) - if (NOT USE_QT5) # TODO port tests to Qt5 -add_subdirectory(tests) +#add_subdirectory(tests) - endif() add_subdirectory(tools) -diff --git a/src/dbusmenuexporter.cpp b/src/dbusmenuexporter.cpp -index f25718d..8868efb 100644 ---- a/src/dbusmenuexporter.cpp -+++ b/src/dbusmenuexporter.cpp -@@ -160,7 +160,9 @@ void DBusMenuExporterPrivate::fillLayoutItem(DBusMenuLayoutItem *item, QMenu *me - Q_FOREACH(QAction *action, menu->actions()) { - int actionId = m_idForAction.value(action, -1); - if (actionId == -1) { -+#ifndef QT_NO_DEBUG - DMWARNING << "No id for action"; -+#endif - continue; - } - -@@ -185,7 +187,9 @@ void DBusMenuExporterPrivate::addAction(QAction *action, int parentId) - { - int id = m_idForAction.value(action, -1); - if (id != -1) { -+#ifndef QT_NO_DEBUG - DMWARNING << "Already tracking action" << action->text() << "under id" << id; -+#endif - return; - } - QVariantMap map = propertiesForAction(action); -diff --git a/src/dbusmenuimporter.cpp b/src/dbusmenuimporter.cpp -index 39f5995..d629d69 100644 ---- a/src/dbusmenuimporter.cpp -+++ b/src/dbusmenuimporter.cpp -@@ -94,7 +94,9 @@ public: - QDBusPendingCallWatcher *refresh(int id) - { - #ifdef BENCHMARK -+#ifndef QT_NO_DEBUG - DMDEBUG << "Starting refresh chrono for id" << id; -+#endif - sChrono.start(); - #endif - QDBusPendingCall call = m_interface->asyncCall("GetLayout", id, 1, QStringList()); -@@ -193,7 +195,9 @@ public: - updateActionShortcut(action, value); - } else if (key == "children-display") { - } else { -+#ifndef QT_NO_DEBUG - DMWARNING << "Unhandled property update" << key; -+#endif - } - } - -@@ -241,7 +245,9 @@ public: - action->setProperty(DBUSMENU_PROPERTY_ICON_DATA_HASH, dataHash); - QPixmap pix; - if (!pix.loadFromData(data)) { -+#ifndef QT_NO_DEBUG - DMWARNING << "Failed to decode icon-data property for action" << action->text(); -+#endif - action->setIcon(QIcon()); - return; - } -@@ -312,7 +318,9 @@ public: - } - - if (watcher->isError()) { -+#ifndef QT_NO_DEBUG - DMWARNING << watcher->error().message(); -+#endif - return false; - } - -@@ -437,18 +445,24 @@ void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher) - - QDBusPendingReply<uint, DBusMenuLayoutItem> reply = *watcher; - if (!reply.isValid()) { -+#ifndef QT_NO_DEBUG - DMWARNING << reply.error().message(); + if(WITH_DOC) +Index: libdbusmenu-qt-0.9.3-16.04.20160218-0ubuntu1/src/debug_p.h +=================================================================== +--- libdbusmenu-qt-0.9.3-16.04.20160218-0ubuntu1.orig/src/debug_p.h ++++ libdbusmenu-qt-0.9.3-16.04.20160218-0ubuntu1/src/debug_p.h +@@ -26,7 +26,11 @@ + #define _DMBLUE "\033[34m" + #define _DMRED "\033[31m" + #define _DMRESET "\033[0m" ++#ifndef QT_NO_DEBUG + #define _DMTRACE(level, color) (level().nospace() << color << __PRETTY_FUNCTION__ << _DMRESET ":").space() ++#else ++#define _DMTRACE(level, color) QT_NO_QDEBUG_MACRO() +#endif - return; - } - - #ifdef BENCHMARK -+#ifndef QT_NO_DEBUG - DMDEBUG << "- items received:" << sChrono.elapsed() << "ms"; -+#endif - #endif - DBusMenuLayoutItem rootItem = reply.argumentAt<1>(); - - QMenu *menu = d->menuForId(parentId); - if (!menu) { -+#ifndef QT_NO_DEBUG - DMWARNING << "No menu for id" << parentId; -+#endif - return; - } - -@@ -475,7 +489,9 @@ void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher) - } - } - #ifdef BENCHMARK -+#ifndef QT_NO_DEBUG - DMDEBUG << "- Menu filled:" << sChrono.elapsed() << "ms"; -+#endif - #endif - } - -@@ -514,7 +530,9 @@ void DBusMenuImporter::slotMenuAboutToShow() - QPointer<QObject> guard(this); - - if (!d->waitForWatcher(watcher, ABOUT_TO_SHOW_TIMEOUT)) { -+#ifndef QT_NO_DEBUG - DMWARNING << "Application did not answer to AboutToShow() before timeout"; -+#endif - } - - #ifdef BENCHMARK -@@ -543,7 +561,9 @@ void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher * - - QDBusPendingReply<bool> reply = *watcher; - if (reply.isError()) { -+#ifndef QT_NO_DEBUG - DMWARNING << "Call to AboutToShow() failed:" << reply.error().message(); -+#endif - return; - } - bool needRefresh = reply.argumentAt<0>(); -@@ -555,7 +575,9 @@ void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher * - d->m_idsRefreshedByAboutToShow << id; - QDBusPendingCallWatcher *watcher2 = d->refresh(id); - if (!d->waitForWatcher(watcher2, REFRESH_TIMEOUT)) { -+#ifndef QT_NO_DEBUG - DMWARNING << "Application did not refresh before timeout"; -+#endif - } - } - } -diff --git a/src/debug_p.h b/src/debug_p.h -index bff37bd..ecdd3a5 100644 ---- a/src/debug_p.h -+++ b/src/debug_p.h -@@ -36,12 +36,10 @@ - #define DMVAR(var) DMDEBUG << #var ":" << var - - #define DMRETURN_IF_FAIL(cond) if (!(cond)) { \ -- DMWARNING << "Condition failed: " #cond; \ - return; \ - } - - #define DMRETURN_VALUE_IF_FAIL(cond, value) if (!(cond)) { \ -- DMWARNING << "Condition failed: " #cond; \ - return (value); \ - } + // Simple macros to get KDebug like support + #define DMDEBUG _DMTRACE(qDebug, _DMBLUE)