Patrick,

Ok, I've updated my syncpbap branch here:
http://cgit.collabora.com/git/user/jwhiting/syncevolution.git but the
completeCb and errorCb both never get called, so it just sits there.
The transfer does happen, as the file is created and vcard data pumped
in,  but something is not right with the signalwatch.  Is there a way
I can check that the signalwatch is watching the right thing?

I've attached a log of obexd-client when running syncevolution. and
the transfer does get the sub path of
/org/bluez/obex/session0/transfer0 so the signalwatch path should be
matching.  Also the complete signal is getting sent according to that
log.

Btw, feel free to offer any feedback on my use of StringPiece, I think
what I have should work, but I've never used it before, so we'll see.

thanks,
Jeremy

On Tue, Aug 28, 2012 at 12:29 AM, Patrick Ohly <[email protected]> wrote:
> 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.
>
>
obex-client[922]: client/transport.c:obc_transport_register() transport 
0x806c6ac name Bluetooth registered
obex-client[922]: client/manager.c:manager_init() Module bluetooth loaded
obex-client[922]: client/opp.c:opp_init() 
obex-client[922]: client/driver.c:obc_driver_register() driver 0x806c954 
service OPP registered
obex-client[922]: client/manager.c:manager_init() Module opp loaded
obex-client[922]: client/ftp.c:ftp_init() 
obex-client[922]: client/driver.c:obc_driver_register() driver 0x806c854 
service FTP registered
obex-client[922]: client/driver.c:obc_driver_register() driver 0x806c86c 
service PCSUITE registered
obex-client[922]: client/manager.c:manager_init() Module ftp loaded
obex-client[922]: client/pbap.c:pbap_init() 
obex-client[922]: client/driver.c:obc_driver_register() driver 0x806c734 
service PBAP registered
obex-client[922]: client/manager.c:manager_init() Module pbap loaded
obex-client[922]: client/sync.c:sync_init() 
obex-client[922]: client/driver.c:obc_driver_register() driver 0x806c6bc 
service SYNC registered
obex-client[922]: client/manager.c:manager_init() Module sync loaded
obex-client[922]: client/map.c:map_init() 
obex-client[922]: client/driver.c:obc_driver_register() driver 0x806ca00 
service MAP registered
obex-client[922]: client/manager.c:manager_init() Module map loaded
obex-client[922]: client/main.c:main() Entering main loop
obex-client[922]: client/session.c:obc_session_ref() 0x876f970: ref=2
obex-client[922]: client/bluetooth.c:bluetooth_connect() 
obex-client[922]: client/session.c:obc_session_create() session 0x876f970 
transport Bluetooth driver PBAP
obex-client[922]: client/bluetooth.c:manager_reply() adapter path 
/org/bluez/561/hci0
obex-client[922]: client/bluetooth.c:transport_connect() port 19
obex-client[922]: client/bluetooth.c:transport_callback() 
obex-client[922]: client/session.c:transport_func() 
obex-client[922]: client/bluetooth.c:bluetooth_getpacketopt() 
obex-client[922]: client/pbap.c:pbap_probe() /org/bluez/obex/session0
obex-client[922]: client/session.c:obc_session_ref() 0x876f970: ref=3
obex-client[922]: client/session.c:obc_session_register() Session(0x876f970) 
registered /org/bluez/obex/session0
obex-client[922]: client/session.c:obc_session_unref() 0x876f970: ref=2
obex-client[922]: client/session.c:obc_session_ref() 0x876f970: ref=3
obex-client[922]: client/session.c:obc_session_unref() 0x876f970: ref=2
obex-client[922]: client/transfer.c:obc_transfer_register() 0x876dde8 
registered /org/bluez/obex/session0/transfer0
obex-client[922]: client/session.c:obc_session_ref() 0x876f970: ref=3
obex-client[922]: client/session.c:obc_session_ref() 0x876f970: ref=4
obex-client[922]: client/session.c:session_process_queue() Transfer(0x876dde8) 
started
obex-client[922]: client/session.c:obc_session_unref() 0x876f970: ref=3
obex-client[922]: client/session.c:session_notify_complete() 
Transfer(0x876dde8) complete
obex-client[922]: client/session.c:obc_session_ref() 0x876f970: ref=4
obex-client[922]: client/transfer.c:obc_transfer_unregister() 0x876dde8 
unregistered /org/bluez/obex/session0/transfer0
obex-client[922]: client/transfer.c:obc_transfer_free() 0x876dde8
obex-client[922]: client/session.c:obc_session_unref() 0x876f970: ref=3
obex-client[922]: client/session.c:obc_session_unref() 0x876f970: ref=2

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

Reply via email to