On Mon, 2012-08-27 at 16:51 -0600, Jeremy Whiting wrote:
> Patrick,
> 
> Thanks for the review and DBus patch.  Unfortunately it seems
> incomplete.  When I run my syncpbap branch found here (I incorporated
> your build fix, that dbus commit, and much of the fyi commit also):
> http://cgit.collabora.com/git/user/jwhiting/syncevolution.git
[...]
> [DEBUG 00:00:04] AddMatch failed:
> GDBus.Error:org.freedesktop.DBus.Error.MatchRuleInvalid: Unknown key
> "path_namespace" in match rule
[...]
> Do I need to build with some flag set or something so path_namespace
> will be a known key?

The path_namespace key was implemented in D-Bus >= 1.5.0. What is your
version of the running daemon? What is your distro?

We could simply not use the prefix filter in the match rule and instead
filter on the client side - not very efficient when there are many
unrelated transfers. I'm also concerned about class B removing a match
rule which class A still depends on. To be fair, its unlikely in
SyncEvolution, because the backend runs in a process by itself that
(currently) doesn't do anything other than PBAP.

Either way, some kind of advance version check or error handling with
fallback to match rule without path_namespace would have to be added.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.


_______________________________________________
SyncEvolution mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution

Reply via email to