Hi Luiz,
* Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> [2011-02-15 21:35:20 +0200]:
> Hi,
>
> On Tue, Feb 15, 2011 at 9:31 PM, Gustavo F. Padovan
> <padovan@xxxxxxxxxxxxxx> wrote:
> > Yeah, this breaks the API, but health is still disabled by default and it
> > is not a good idea create a new path only for it. So moving it to "/"
> > ---
> > doc/health-api.txt | 2 +-
> > health/hdp.c | 4 ++--
> > test/test-health | 2 +-
> > test/test-health-sink | 2 +-
> > 4 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/doc/health-api.txt b/doc/health-api.txt
> > index 3d0a717..9d5a54d 100644
> > --- a/doc/health-api.txt
> > +++ b/doc/health-api.txt
> > @@ -10,7 +10,7 @@ Health Device Profile hierarchy
> >
> > Service org.bluez
> > Interface org.bluez.HealthManager
> > -Object path /org/bluez/
> > +Object path /
> >
> > Methods:
> >
> > diff --git a/health/hdp.c b/health/hdp.c
> > index d4a2a30..e58bd55 100644
> > --- a/health/hdp.c
> > +++ b/health/hdp.c
> > @@ -2171,7 +2171,7 @@ int hdp_manager_start(DBusConnection *conn)
> > {
> > DBG("Starting Health manager");
> >
> > - if (!g_dbus_register_interface(conn, MANAGER_PATH,
> > + if (!g_dbus_register_interface(conn, "/",
> > HEALTH_MANAGER,
> > health_manager_methods, NULL, NULL,
> > NULL, manager_path_unregister)) {
> > @@ -2186,7 +2186,7 @@ int hdp_manager_start(DBusConnection *conn)
> >
> > void hdp_manager_stop()
> > {
> > - g_dbus_unregister_interface(connection, MANAGER_PATH, HEALTH_MANAGER);
> > + g_dbus_unregister_interface(connection, "/", HEALTH_MANAGER);
> >
> > dbus_connection_unref(connection);
> > DBG("Stopped Health manager");
> > diff --git a/test/test-health b/test/test-health
> > index 28cc894..7820a24 100755
> > --- a/test/test-health
> > +++ b/test/test-health
> > @@ -47,7 +47,7 @@ def enter_mainloop():
> > finally:
> > print "Exiting, bye"
> >
> > -hdp_manager = dbus.Interface(bus.get_object("org.bluez", "/org/bluez"),
> > +hdp_manager = dbus.Interface(bus.get_object("org.bluez", "/"),
> > "org.bluez.HealthManager")
> >
> > role = None
> > diff --git a/test/test-health-sink b/test/test-health-sink
> > index cb9d434..50eb74c 100755
> > --- a/test/test-health-sink
> > +++ b/test/test-health-sink
> > @@ -12,7 +12,7 @@ loop = gobject.MainLoop()
> >
> > bus = dbus.SystemBus()
> >
> > -hdp_manager = dbus.Interface(bus.get_object("org.bluez", "/org/bluez"),
> > +hdp_manager = dbus.Interface(bus.get_object("org.bluez", "/"),
> > "org.bluez.HealthManager")
> > app_path = hdp_manager.CreateApplication({"DataType": dbus.types.UInt16(4103),
> > "Role": "sink"})
> > --
> > 1.7.4
>
> Any reasons why this is not per adapter? This is supposed to be very
> similar to Media API which is used in a per adapter fashion.
IIRC In the first versions of the API Marcel tried to abstract the concept of
adapter and have the API works with Health Applications. But not sure if after
http://www.spinics.net/lists/linux-bluetooth/msg06788.html
this is still valid.
--
Gustavo F. Padovan
http://profusion.mobi
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
