rpms/dbus-glib/F-7 dbus-glib-proxy-signals-once.patch, NONE, 1.1 dbus-glib.spec, 1.15, 1.16

[Daniel Williams (dcbw)]

Author: dcbw

Update of /cvs/extras/rpms/dbus-glib/F-7
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv10911

Modified Files:
	dbus-glib.spec 
Added Files:
	dbus-glib-proxy-signals-once.patch 
Log Message:
* Mon Sep 24 2007 Dan Williams <dcbw at redhat.com> - 0.73-3
- Dispatch NameOwnerChanged signals to proxies only once (fdo #12505)



dbus-glib-proxy-signals-once.patch:

--- NEW FILE dbus-glib-proxy-signals-once.patch ---
diff -up dbus-glib-0.73/dbus/dbus-gproxy.c.foo dbus-glib-0.73/dbus/dbus-gproxy.c
--- dbus-glib-0.73/dbus/dbus-gproxy.c.foo	2007-02-08 20:59:39.000000000 -0500
+++ dbus-glib-0.73/dbus/dbus-gproxy.c	2007-09-24 14:54:39.000000000 -0400
@@ -1232,8 +1235,15 @@ dbus_g_proxy_manager_filter (DBusConnect
 						  dbus_message_get_interface (message));
 
 	      owner_list = g_hash_table_lookup (manager->proxy_lists, tri);
-	      if (owner_list != NULL)
-		full_list = g_slist_concat (full_list, g_slist_copy (owner_list->proxies));
+	      if (owner_list != NULL) {
+	        GSList *elt;
+
+	        /* Ignore duplicates when adding to full_list */
+	        for (elt = owner_list->proxies; elt; elt = g_slist_next (elt)) {
+	          if (!g_slist_find (full_list, elt->data))
+	            full_list = g_slist_append (full_list, elt->data);
+	        }
+	      }
 	      g_free (tri);
 	    }
 	}



Index: dbus-glib.spec
===================================================================
RCS file: /cvs/extras/rpms/dbus-glib/F-7/dbus-glib.spec,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -r1.15 -r1.16
--- dbus-glib.spec	2 Aug 2007 02:33:40 -0000	1.15
+++ dbus-glib.spec	24 Sep 2007 21:32:14 -0000	1.16
@@ -8,11 +8,12 @@
 Summary: GLib bindings for D-Bus
 Name: dbus-glib
 Version: 0.73
-Release: 2%{?dist}
+Release: 3%{?dist}
 URL: http://www.freedesktop.org/software/dbus/
 Source0: http://dbus.freedesktop.org/releases/%{name}-%{version}.tar.gz
 Source1: dbus-bus-introspect.xml
 Patch0: broken-xml.patch
+Patch1: dbus-glib-proxy-signals-once.patch
 License: AFL/GPL
 Group: System Environment/Libraries
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
@@ -59,6 +60,7 @@
 %prep
 %setup -q
 %patch0 -p1 -b .broken-xml
+%patch1 -p1 -b .proxy-signals-once
 
 %build
 
@@ -114,6 +116,9 @@
 %endif
 
 %changelog
+* Mon Sep 24 2007 Dan Williams <dcbw at redhat.com> - 0.73-3
+- Dispatch NameOwnerChanged signals to proxies only once (fdo #12505)
+
 * Wed Aug  1 2007 Matthias Clasen <mclasen at redhat.com> - 0.73-2
 - Fix a problem with introspection (#248150)