[mail-notification/f14/master] Fix a crash (RHBZ #654826) and build with -O0
epienbro
epienbro at fedoraproject.org
Thu Feb 17 17:24:50 UTC 2011
commit a4a8d3020adf9f02d4c88820374e2a7d119b7c73
Author: Erik van Pienbroek <epienbro at fedoraproject.org>
Date: Thu Feb 17 18:23:55 2011 +0100
Fix a crash (RHBZ #654826) and build with -O0
- Don't throw assertion errors (and a possible crash) when
some icons can't be found. Fixes RHBZ #654826
- (Temporary) build with -O0 in order to get more sane stacktraces so
we can properly investigate and fix RHBZ #531088
mail-notification-5.4-add-fallback-icon.patch | 16 ++++++++++++++++
...ication-5.4-build-without-optimalisations.patch | 12 ++++++++++++
mail-notification.spec | 20 ++++++++++++++++++--
3 files changed, 46 insertions(+), 2 deletions(-)
---
diff --git a/mail-notification-5.4-add-fallback-icon.patch b/mail-notification-5.4-add-fallback-icon.patch
new file mode 100644
index 0000000..41c28e8
--- /dev/null
+++ b/mail-notification-5.4-add-fallback-icon.patch
@@ -0,0 +1,16 @@
+--- src/mn-stock.c.orig 2011-02-17 17:42:39.886767087 +0100
++++ src/mn-stock.c 2011-02-17 17:49:37.415541177 +0100
+@@ -86,6 +86,13 @@
+ gtk_icon_source_set_icon_name(icon_source, icons[i].icon_name);
+ gtk_icon_set_add_source(icon_set, icon_source);
+ gtk_icon_source_free(icon_source);
++
++ /* Add a fallback icon */
++ icon_source = gtk_icon_source_new();
++ gtk_icon_source_set_icon_name(icon_source, "mail-notification");
++ gtk_icon_source_set_state_wildcarded(icon_source, TRUE);
++ gtk_icon_set_add_source(icon_set, icon_source);
++ gtk_icon_source_free(icon_source);
+ }
+ else if (icons[i].source_stock_id)
+ {
diff --git a/mail-notification-5.4-build-without-optimalisations.patch b/mail-notification-5.4-build-without-optimalisations.patch
new file mode 100644
index 0000000..7dce834
--- /dev/null
+++ b/mail-notification-5.4-build-without-optimalisations.patch
@@ -0,0 +1,12 @@
+--- jbsrc/jb.c.orig 2011-02-08 23:11:32.767114666 +0100
++++ jbsrc/jb.c 2011-02-08 23:18:48.870910628 +0100
+@@ -443,6 +443,9 @@
+ */
+ jb_compile_options_add_ldflags(object->compile_options, "-Wl,--export-dynamic");
+
++ jb_compile_options_add_cppflags(object->compile_options, "-O0");
++ jb_compile_options_add_ldflags(object->compile_options, "-O0");
++
+ jb_compile_options_add_libs(object->compile_options, "-lm");
+
+ jb_compile_options_add_package(object->compile_options, "gettext");
diff --git a/mail-notification.spec b/mail-notification.spec
index 7cedd31..4ef9728 100644
--- a/mail-notification.spec
+++ b/mail-notification.spec
@@ -1,6 +1,6 @@
Name: mail-notification
Version: 5.4
-Release: 25%{?dist}
+Release: 26%{?dist}
Summary: Status icon that informs you if you have new mail
Group: Applications/Internet
@@ -16,6 +16,8 @@ Patch5: mail-notification-5.4-icons.patch
Patch6: mail-notification-5.4-weak.patch
Patch7: mail-notification-5.4-popup-attach.patch
Patch8: mail-notification-5.4-kde-trayicon.patch
+Patch9: mail-notification-5.4-build-without-optimalisations.patch
+Patch10: mail-notification-5.4-add-fallback-icon.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildRequires: gettext
@@ -75,7 +77,15 @@ Evolution support for Mail Notification.
%patch7 -p1 -b .popup_attach
%patch8 -p1 -b .kde_trayicon
-# Drop #line statements in C sources generated bu .gob,
+# Temporary patch to build without any optimalisations so we
+# can get better stacktraces for RHBZ #531088
+%patch9 -p0 -b .gcc
+
+# Don't throw assertion errors (and a possible crash) when
+# some icons can't be found. Fixes RHBZ #654826
+%patch10 -p0 -b .fallback_icon
+
+# Drop #line statements in C sources generated by .gob,
# for the proper debuginfo package
pushd build/src
for f in *.c *.h
@@ -199,6 +209,12 @@ export GCONF_CONFIG_SOURCE=`gconftool-2 --get-default-source`
%changelog
+* Thu Feb 17 2011 Erik van Pienbroek <epienbro at fedoraproject.org> - 5.4-26
+- Don't throw assertion errors (and a possible crash) when
+ some icons can't be found. Fixes RHBZ #654826
+- (Temporary) build with -O0 in order to get more sane stacktraces so
+ we can properly investigate and fix RHBZ #531088
+
* Mon Oct 11 2010 Dmitry Butskoy <Dmitry at Butskoy.name> - 5.4-25
- fix popup window place
(#608372, by Erik van Pienbroek <erik-fedora at vanpienbroek.nl>)
More information about the scm-commits
mailing list