http://bugzilla.moblin.org/show_bug.cgi?id=3479





--- Comment #8 from Chen Congwu <[email protected]>  2009-08-11 18:13:27 ---
After reading the eds-dbus code, I suspect "RETURN-REV" capability can't be
implemented for existing non-atomic calls. 

The reason is: the dbus call does not return "revision" or "Contact", the
changed item is asynchronously popped up by another signal ("contacts-changed",
etc.). What's worse, the signal has a buffering mechanism, it will not emit as
soon as a contact is changed(maybe delayed to a later point to pop up a batch
of changes).
Two solutions: 1) change the dbus interface (incompatible)  2) Use the
workaround which was used by syncevolution in libebook (get the whole contact
again by another dbus call, which still can not guarantee the correct semantic
(the contacts may already be changed by another client).

I suggest not support "RETURN-REV" capability in existing non-atomic calls. For
the new added atomic calls, we can use a different dbus interface which will
return the "revision" information from the dbus method call.

Ross and Patrick, do you think so?

-- 
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.
_______________________________________________
Syncevolution-issues mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution-issues

Reply via email to