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

[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

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

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

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 loop

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 the

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 contacts

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 everyone