Re: [Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Alexander Larsson
On Fri, 2011-05-27 at 17:09 +0200, Milan Crha wrote: > On Fri, 2011-05-27 at 13:01 +0200, Alexander Larsson wrote: > > Secondly, why is it using a dbus signal for the done callbacks? > > Wouldn't it be more sane to use a direct message (without expecting a > > reply)? A signal will be sent to every

Re: [Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Alexander Larsson
On Fri, 2011-05-27 at 16:49 +0200, Milan Crha wrote: > On Fri, 2011-05-27 at 13:01 +0200, Alexander Larsson wrote: > > I needed the most recent eds to test the folks eds backend, and I picked > > up the new dbus EBook APIs. Unfortunatelly that seemed to be completely > > broken for me (evolution co

Re: [Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Milan Crha
On Fri, 2011-05-27 at 11:59 -0400, Matthew Barnes wrote: > On Fri, 2011-05-27 at 16:49 +0200, Milan Crha wrote: > > Maybe it can pump them, but it doesn't deliver them, as far as my tests > > showed, because they are usually delivered on idle, which never happen > > with blocked main loop. That's

Re: [Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Matthew Barnes
On Fri, 2011-05-27 at 16:49 +0200, Milan Crha wrote: > Maybe it can pump them, but it doesn't deliver them, as far as my tests > showed, because they are usually delivered on idle, which never happen > with blocked main loop. That's the reason why I added there this loop. > Only notice that this l

Re: [Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Milan Crha
On Fri, 2011-05-27 at 13:01 +0200, Alexander Larsson wrote: > Secondly, why is it using a dbus signal for the done callbacks? > Wouldn't it be more sane to use a direct message (without expecting a > reply)? A signal will be sent to everyone subscribing to that signal, > which will be all clients t

Re: [Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Milan Crha
On Fri, 2011-05-27 at 13:01 +0200, Alexander Larsson wrote: > I needed the most recent eds to test the folks eds backend, and I picked > up the new dbus EBook APIs. Unfortunatelly that seemed to be completely > broken for me (evolution contacts hanged, test-self in the tests > hanged), so I have de

[Evolution-hackers] Issues with new EBook dbus implementation

2011-05-27 Thread Alexander Larsson
I needed the most recent eds to test the folks eds backend, and I picked up the new dbus EBook APIs. Unfortunatelly that seemed to be completely broken for me (evolution contacts hanged, test-self in the tests hanged), so I have debugged it. Turns out that it was sending a dbus signal like get_cont

Re: [Evolution-hackers] [PATCH 1/2] e_cal_new_system_foo() should create corresponding source in GConf

2011-05-27 Thread Christophe Dumez
On 05/10/2011 01:43 PM, David Woodhouse wrote: On Tue, 2011-05-10 at 11:40 +0100, Dumez, Christophe wrote: I have tested the patch but it does not seem to help. I don't know what the reason is yet. It may depend on earlier fixes? Can you show your patch? This e-mail and any attachments may co