GNOME Bugzilla – Bug 316925
Export presence over Galago
Last modified: 2005-11-01 19:13:54 UTC
Attaching a prototype patch to export presence over Galago. It needs more work but sort-of works.
Created attachment 52499 [details] [review] First draft
Created attachment 52513 [details] [review] Second draft
Hmm, could you use the coding style that Gossip use, would make looking at the patch a lot easier. It would also be nice to have the Galago support optional.
Created attachment 53256 [details] [review] Third draft OK, I had a quick butchers at this and fixed the coding style which as Micke says would have made things a lot quicker for me. I have also fixed up some unnecessary warnings and added a load of debugging to know what is happening. I know very little about Galago at the moment, and apart from seeing it talk to libgalago OK I couldn't see how the information we pass on is being used - what application can I use to do this? I did look at dbus-viewer to see something that resembled my contact list, but would like something better (gnome-presence-applet?). Current issues for me: - I didn't get around to making libgalago an optional component yet (in configure), that is essential. - I wasn't sure if Galago expects the "contact-removed" signal to mean actually removed from the roster permanently or the same meaning we have (i.e. remove temporarily like when you go online/offline). - There are some issues when the session presence changes and it looks up the account to use with Galago. With my environment I have 6 accounts (mostly for testing) and it doesn't work so well in some situations. I get the "Cannot find account" error, but, that might be a gossip-session thing, I need to look at this further when I have time. Apart from all that, it looks like a good start, thanks Ross, looking forward to the next patch ;)
For basic debugging I use get-presence from libgalago/tests, for real-world use I am using a Secret Application. I'll work on some of the remaining problems today.
Created attachment 53279 [details] [review] Forth draft This patch makes Galago conditional.
OK Ross, I have now committed this since it works with the get-presence test app that comes with libgalago. It needs some work in places but it is a start. Thanks for the patch ;)