[oxygen-gtk2] Upgrade to oxygen-gtk2 1.3.2 is buggy (kde#314545)
Rex Dieter
rdieter at fedoraproject.org
Fri Feb 8 18:13:31 UTC 2013
commit 2fe8589ead0372a303dafcb3bc4d02d98c9df172
Author: Rex Dieter <rdieter at math.unl.edu>
Date: Fri Feb 8 12:11:22 2013 -0600
Upgrade to oxygen-gtk2 1.3.2 is buggy (kde#314545)
...pt-initialization-if-gtkSettings-is-not-r.patch | 65 ++++++++++++++++++++
oxygen-gtk2.spec | 10 +++-
2 files changed, 74 insertions(+), 1 deletions(-)
---
diff --git a/0002-don-t-attempt-initialization-if-gtkSettings-is-not-r.patch b/0002-don-t-attempt-initialization-if-gtkSettings-is-not-r.patch
new file mode 100644
index 0000000..508616c
--- /dev/null
+++ b/0002-don-t-attempt-initialization-if-gtkSettings-is-not-r.patch
@@ -0,0 +1,65 @@
+From a89f20d522b674cd2794cd195b7e48978ac69b20 Mon Sep 17 00:00:00 2001
+From: Hugo Pereira Da Costa <hugo at oxygen-icons.org>
+Date: Thu, 7 Feb 2013 11:56:04 +0100
+Subject: [PATCH 2/2] don't attempt initialization if gtkSettings is not
+ ready/available. put back initialization in theme_init try initialization a
+ second time in instanceInit, in case it failed the first time, due to
+ gtkSettings not being ready.
+
+---
+ src/oxygenqtsettings.cpp | 3 +++
+ src/oxygenstylewrapper.cpp | 4 ++++
+ src/oxygentheme.cpp | 8 ++------
+ 3 files changed, 9 insertions(+), 6 deletions(-)
+
+diff --git a/src/oxygenqtsettings.cpp b/src/oxygenqtsettings.cpp
+index f2ad1af..9965086 100644
+--- a/src/oxygenqtsettings.cpp
++++ b/src/oxygenqtsettings.cpp
+@@ -102,6 +102,9 @@ namespace Oxygen
+
+ const bool forced( flags&Forced );
+
++ // no attempt at initializing if gtk settings is not yet set
++ if( !gtk_settings_get_default() ) return false;
++
+ if( _initialized && !forced ) return false;
+ else if( !forced ) _initialized = true;
+
+diff --git a/src/oxygenstylewrapper.cpp b/src/oxygenstylewrapper.cpp
+index ff0666b..a2572b0 100644
+--- a/src/oxygenstylewrapper.cpp
++++ b/src/oxygenstylewrapper.cpp
+@@ -3741,6 +3741,10 @@ namespace Oxygen
+ #endif
+
+ // style initialization
++ /*
++ this is normally achieved in theme_init, but is somehow not working with gnome shell
++ so that it is redone here, since duplication is avoided when initialization is called twice without modifications
++ */
+ Style::instance().initialize();
+
+ // hooks
+diff --git a/src/oxygentheme.cpp b/src/oxygentheme.cpp
+index 50b04e2..9bc49e1 100644
+--- a/src/oxygentheme.cpp
++++ b/src/oxygentheme.cpp
+@@ -52,12 +52,8 @@ void theme_init( GTypeModule* module )
+ Oxygen::RCStyle::registerType( module );
+ Oxygen::StyleWrapper::registerType( module );
+
+- // initialize oxygen here, for XUL applications
+- // special care is taken to not initialize everything
+- Oxygen::ApplicationName applicationName;
+- applicationName.initialize();
+- if( applicationName.isXul() )
+- { Oxygen::Style::instance().initialize( Oxygen::QtSettings::Forced | Oxygen::QtSettings::Colors ); }
++ // style initialization
++ Oxygen::Style::instance().initialize();
+
+ }
+
+--
+1.8.1.2
+
diff --git a/oxygen-gtk2.spec b/oxygen-gtk2.spec
index 08a4165..23bf535 100644
--- a/oxygen-gtk2.spec
+++ b/oxygen-gtk2.spec
@@ -2,13 +2,16 @@
Name: oxygen-gtk2
Summary: Oxygen GTK+2 theme
Version: 1.3.2
-Release: 1%{?dist}
+Release: 2%{?dist}
License: LGPLv2+
Group: User Interface/Desktops
URL: https://projects.kde.org/projects/playground/artwork/oxygen-gtk
Source0: ftp://ftp.kde.org/pub/kde/stable/oxygen-gtk2/%{version}/src/%{name}-%{version}.tar.bz2
+## upstream patches
+Patch102: 0002-don-t-attempt-initialization-if-gtkSettings-is-not-r.patch
+
BuildRequires: cmake
BuildRequires: gtk2-devel
@@ -31,6 +34,8 @@ otherwise breaks every time some setting is changed in KDE.
%prep
%setup -q
+%patch102 -p1 -b .0002
+
%build
mkdir -p %{_target_platform}
@@ -59,6 +64,9 @@ rm -rf %{buildroot}
%changelog
+* Fri Feb 08 2013 Rex Dieter <rdieter at fedoraproject.org> 1.3.2-2
+- Upgrade to oxygen-gtk2 1.3.2 is buggy (kde#314545)
+
* Wed Jan 30 2013 Alexey Kurov <nucleo at fedoraproject.org> - 1.3.2-1
- oxygen-gtk2-1.3.2
More information about the scm-commits
mailing list