[patch] add owner-changed signal to DBusGProxy

[Owen Taylor]

On Wed, 2005-07-13 at 19:36 -0400, Colin Walters wrote:

> > The reason for the hack is that trying to do it cleanly never seemed to
> > work for all the cases. GTK has some crazy complicated Makefile rules
> > for this kind of thing. 
> 
> Again, I think the GTK+ rules were written for an Automake that predates
> BUILT_SOURCES and that is the source of the complexity.  I've added Owen
> to the CC, maybe he can comment.  Ideally we'd get example Automake
> usage added to the glib-genmarshal man page if indeed the below usage is
> incorrect.
> 
> We've been doing the exact same thing in Rhythmbox though for years now,
> and from a quick glance at my gnome CVS tree, gnome-panel and evince do
> the same, and I'm sure more modules do.  If it's broken, it must be in a
> subtle way.

Not really sure what the question is. The GTK+ rules are in fact rather
overcomplicated: it isn't actually related to BUILT_SOURCES, but rather
I think it was done to match the generation of gtktypebuiltins.h.
gtktypebuiltins.h is done that way to avoid rebuilding all of GTK+
for changes to header files that don't change the numerations.

If you look at the GDK marshaler generation rules, they are pretty much
what you have below, except that they remove the .tmp file on failure.

The same thing is documented in:

 http://developer.gnome.org/dotplan/porting/ar01s15.html

Regards,
						Owen

> (Owen, for reference here are the rules in question)
> 
> dbus-gmarshal.c: Makefile dbus-gmarshal.list
> 	@GLIB_GENMARSHAL@ --prefix=_dbus_g_marshal $(srcdir)/dbus-gmarshal.list --header --body > $@.tmp && mv $@.tmp $@
> 
> dbus-gmarshal.h: Makefile dbus-gmarshal.list
> 	@GLIB_GENMARSHAL@ --prefix=_dbus_g_marshal $(srcdir)/dbus-gmarshal.list --header > $@.tmp && mv $@.tmp $@
> 
> BUILT_SOURCES += dbus-gmarshal.c dbus-gmarshal.h
> CLEANFILES = $(BUILT_SOURCES)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.freedesktop.org/archives/dbus/attachments/20050714/baa9f612/attachment.pgp