Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package kde-gtk-config5 for openSUSE:Factory 
checked in at 2021-08-04 22:28:39
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/kde-gtk-config5 (Old)
 and      /work/SRC/openSUSE:Factory/.kde-gtk-config5.new.1899 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "kde-gtk-config5"

Wed Aug  4 22:28:39 2021 rev:119 rq:909801 version:5.22.4

Changes:
--------
--- /work/SRC/openSUSE:Factory/kde-gtk-config5/kde-gtk-config5.changes  
2021-07-29 21:32:12.364753179 +0200
+++ 
/work/SRC/openSUSE:Factory/.kde-gtk-config5.new.1899/kde-gtk-config5.changes    
    2021-08-04 22:29:18.701770748 +0200
@@ -1,0 +2,8 @@
+Mon Aug  2 11:58:51 UTC 2021 - Fabian Vogt <fab...@ritter-vogt.de>
+
+- Add patch to drop gtk-2.0 dependency and adjust BuildRequires:
+  * 0001-Drop-unused-dependency-on-gtk-2.0.patch
+- Add patch to fix setting the default GTK theme on first login:
+  * 0001-Make-sure-to-actually-commit-GSettings-changes.patch
+
+-------------------------------------------------------------------

New:
----
  0001-Drop-unused-dependency-on-gtk-2.0.patch
  0001-Make-sure-to-actually-commit-GSettings-changes.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ kde-gtk-config5.spec ++++++
--- /var/tmp/diff_new_pack.Ii2JwM/_old  2021-08-04 22:29:19.137770216 +0200
+++ /var/tmp/diff_new_pack.Ii2JwM/_new  2021-08-04 22:29:19.141770211 +0200
@@ -29,6 +29,9 @@
 Source1:        
https://download.kde.org/stable/plasma/%{version}/kde-gtk-config-%{version}.tar.xz.sig
 Source2:        plasma.keyring
 %endif
+# PATCH-FIX-UPSTREAM
+Patch1:         0001-Make-sure-to-actually-commit-GSettings-changes.patch
+Patch2:         0001-Drop-unused-dependency-on-gtk-2.0.patch
 BuildRequires:  extra-cmake-modules >= 0.0.9
 BuildRequires:  gsettings-desktop-schemas
 BuildRequires:  kf5-filesystem
@@ -42,8 +45,9 @@
 BuildRequires:  cmake(KF5GuiAddons)
 BuildRequires:  cmake(KF5IconThemes)
 BuildRequires:  cmake(Qt5Svg)
-BuildRequires:  pkgconfig(giomm-2.4)
-BuildRequires:  pkgconfig(gtk+-2.0)
+BuildRequires:  pkgconfig(gio-2.0)
+BuildRequires:  pkgconfig(glib-2.0)
+BuildRequires:  pkgconfig(gobject-2.0)
 BuildRequires:  pkgconfig(gtk+-3.0)
 BuildRequires:  pkgconfig(x11)
 # Needed for syncing GTK+ settings

++++++ 0001-Drop-unused-dependency-on-gtk-2.0.patch ++++++
>From 090cfb2db4bc9c032cd7a7c12cf2a317c248e3c6 Mon Sep 17 00:00:00 2001
From: Fabian Vogt <fab...@ritter-vogt.de>
Date: Mon, 2 Aug 2021 13:52:38 +0200
Subject: [PATCH] Drop unused dependency on gtk+-2.0

FWICT it's not actually used anywhere.
---
 CMakeLists.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 45d02ef..f102642 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -20,7 +20,6 @@ find_package(Sass REQUIRED)
 find_package(XSettingsd)
 
 pkg_check_modules(GTK+3 REQUIRED IMPORTED_TARGET gtk+-3.0)
-pkg_check_modules(GTK+2 REQUIRED IMPORTED_TARGET gtk+-2.0)
 pkg_check_modules(GLib2 REQUIRED IMPORTED_TARGET glib-2.0)
 pkg_check_modules(GObject REQUIRED IMPORTED_TARGET gobject-2.0)
 pkg_check_modules(GIO REQUIRED IMPORTED_TARGET gio-2.0)
-- 
2.32.0

++++++ 0001-Make-sure-to-actually-commit-GSettings-changes.patch ++++++
>From da887d179bff6597355b0214c4e6c56f327b6ce3 Mon Sep 17 00:00:00 2001
From: Fabian Vogt <fab...@ritter-vogt.de>
Date: Mon, 2 Aug 2021 13:47:36 +0200
Subject: [PATCH] Make sure to actually commit GSettings changes

g_settings_set_* is async, so make sure the change is actually performed before
returning and eventually exiting. This mostly affected the gtk_theme update
binary, because that isn't long running unlike the kded module.
---
 kded/configeditor.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/kded/configeditor.cpp b/kded/configeditor.cpp
index a51f48b..587ff7c 100644
--- a/kded/configeditor.cpp
+++ b/kded/configeditor.cpp
@@ -38,12 +38,15 @@ void ConfigEditor::setGtkConfigValueGSettings(const QString 
&paramName, const QV
     } else if (paramValue.type() == QVariant::Type::Bool) {
         g_settings_set_boolean(gsettings, paramName.toUtf8().constData(), 
paramValue.toBool());
     }
+
+    g_settings_sync();
 }
 
 void ConfigEditor::setGtkConfigValueGSettingsAsEnum(const QString &paramName, 
int paramValue, const QString &category)
 {
     g_autoptr(GSettings) gsettings = 
g_settings_new(category.toUtf8().constData());
     g_settings_set_enum(gsettings, paramName.toUtf8().constData(), paramValue);
+    g_settings_sync();
 }
 
 void ConfigEditor::setGtkConfigValueSettingsIni(const QString &versionString, 
const QString &paramName, const QVariant &paramValue)
-- 
2.32.0

Reply via email to