Hi Martin, On Wed, Jul 17, 2013 at 6:00 AM, <martin.xu@xxxxxxxxxxxxxxx> wrote: > From: Martin Xu <martin.xu@xxxxxxxxxxxxxxx> > > The definition of Target should be the same with Target parameter > of CreateSession. > > And add TargetUUID for the external profile that can't be shown from Target > --- > doc/obex-api.txt | 18 +++++++++++++++++- > 1 file changed, 17 insertions(+), 1 deletion(-) > > Changelog: > v2: Add description why we need add TargetUUID property > > diff --git a/doc/obex-api.txt b/doc/obex-api.txt > index 22449c4..924df2f 100644 > --- a/doc/obex-api.txt > +++ b/doc/obex-api.txt > @@ -68,7 +68,23 @@ Properties string Source [readonly] > > string Target [readonly] > > - Target UUID > + The currently supported targets are the following: > + > + "ftp" > + "map" > + "opp" > + "pbap" > + "sync" > + > + string TargetUUID [readonly] > + > + The UUID of target > + > + TargetUUID is useful when the target is from > + external profile and can't be exported from > + Property Target > + > + Opp doesn't have Target UUID, and is optional here > > string Root [readonly] > > -- > 1.7.10.4 You still didn't address any of my concerns, neither of these properties is supposed to be show to the user, which probably don't know what those names mean, and any logic to match session type don't need to check both friendly name _and_ UUID, the latter should be enough even for proprietary/non-standard ones. The documentation of CreateSession is the one that needs adjustment, it should clearly state Target is a UUID and friendly names can be use for convenience but I expect them to be used just in scripts because application should really be using UUIDs anyway as in org.bluez.Device1.UUIDs, their friendly names are never exposed in any of our APIs. -- Luiz Augusto von Dentz -- 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
