2014-03-19 11:34:32

by Andrzej Kaczmarek

[permalink] [raw]
Subject: [PATCH 1/2] test: Add missing import in cyclingspeed

---
test/test-cyclingspeed | 1 +
1 file changed, 1 insertion(+)

diff --git a/test/test-cyclingspeed b/test/test-cyclingspeed
index d49041f..4b8d841 100755
--- a/test/test-cyclingspeed
+++ b/test/test-cyclingspeed
@@ -15,6 +15,7 @@ try:
from gi.repository import GObject
except ImportError:
import gobject as GObject
+import bluezutils

BUS_NAME = 'org.bluez'
CYCLINGSPEED_MANAGER_INTERFACE = 'org.bluez.CyclingSpeedManager1'
--
1.9.0



2014-03-19 19:11:45

by Markus Roppelt

[permalink] [raw]
Subject: Re: [PATCH 2/2] test: Fix cyclingspeed manager interface name

Hi Andrzej,

How are you testing the cycling speed?
I get an error like this (with watcher or manager interface):

[markus@localhost test]$ ./test-cyclingspeed -b D1:83:15:3C:26:F0
Traceback (most recent call last):
File "./test-cyclingspeed", line 153, in <module>
cscmanager.RegisterWatcher(
watcher_path)
File "/usr/lib/python2.7/site-packages/dbus/proxies.py", line 70, in __call__
return self._proxy_method(*args, **keywords)
File "/usr/lib/python2.7/site-packages/dbus/proxies.py", line 145, in __call__
**keywords)
File "/usr/lib/python2.7/site-packages/dbus/connection.py", line
651, in call_blocking
message, timeout)
dbus.exceptions.DBusException:
org.freedesktop.DBus.Error.UnknownMethod: Method "RegisterWatcher"
with signature "s" on interface "org.bluez.CyclingSpeedManager1"
doesn't exist

I compiled with ./configure --enable-experimental
and then changed in /usr/lib/systemd/system/bluetooth.service to start
with the experimental flag:
ExecStart=/usr/local/libexec/bluetooth/bluetoothd -E

Is there anything else I need to do?

Greetings
Markus

2014-03-19 12:34 GMT+01:00 Andrzej Kaczmarek <[email protected]>:
> ---
> test/test-cyclingspeed | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/test-cyclingspeed b/test/test-cyclingspeed
> index 4b8d841..393f79c 100755
> --- a/test/test-cyclingspeed
> +++ b/test/test-cyclingspeed
> @@ -144,7 +144,7 @@ if __name__ == "__main__":
> device_path = device.object_path
>
> cscmanager = dbus.Interface(bus.get_object(BUS_NAME, adapter_path),
> - CYCLINGSPEED_WATCHER_INTERFACE)
> + CYCLINGSPEED_MANAGER_INTERFACE)
>
> watcher_path = "/test/watcher"
> watcher = Watcher(bus, watcher_path)
> --
> 1.9.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

2014-03-19 12:05:21

by Johan Hedberg

[permalink] [raw]
Subject: Re: [PATCH 1/2] test: Add missing import in cyclingspeed

Hi Andrzej,

On Wed, Mar 19, 2014, Andrzej Kaczmarek wrote:
> ---
> test/test-cyclingspeed | 1 +
> 1 file changed, 1 insertion(+)

Both patches have been applied. Thanks.

Johan

2014-03-19 11:34:33

by Andrzej Kaczmarek

[permalink] [raw]
Subject: [PATCH 2/2] test: Fix cyclingspeed manager interface name

---
test/test-cyclingspeed | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/test-cyclingspeed b/test/test-cyclingspeed
index 4b8d841..393f79c 100755
--- a/test/test-cyclingspeed
+++ b/test/test-cyclingspeed
@@ -144,7 +144,7 @@ if __name__ == "__main__":
device_path = device.object_path

cscmanager = dbus.Interface(bus.get_object(BUS_NAME, adapter_path),
- CYCLINGSPEED_WATCHER_INTERFACE)
+ CYCLINGSPEED_MANAGER_INTERFACE)

watcher_path = "/test/watcher"
watcher = Watcher(bus, watcher_path)
--
1.9.0