Re: [PATCH 1/2] bluez4: Install simple-agent utility for pairing bluetooth devices

[Zhong Hongbo <hongbo.zhong@windriver.com>]

On 11/13/2013 08:34 PM, Burton, Ross wrote:
> On 13 November 2013 11:38, Phil Blundell <pb@pbcl.net> wrote:
>> On Wed, 2013-11-13 at 18:08 +0800, Zhong Hongbo wrote:
>>> From: Zhong Hongbo <hongbo.zhong@windriver.com>
>>>
>>> Signed-off-by: Zhong Hongbo <hongbo.zhong@windriver.com>
>> [...]
>>>        install -m 0644 ${S}/input/input.conf ${D}/${sysconfdir}/bluetooth/
>>> +     install -m 0755 ${S}/test/simple-agent ${D}/${bindir}/simple-agent
>> What does simple-agent actually do?  Is it generally enough useful that
>> it should be installed unconditionally?
> It's a dumb agent for responding to incoming pairing requests, as it
> works interactively from a terminal it's purely for testing purposes.
> I'd prefer to see it (and probably more utilities) installed in
> bluez4-tests as it's not generally useful.
Agree.

But a issue is: How do place this test file?  On the /usr/bin directory? 
or /usb/bin/bluez? or others?

The below place the test to /usr/bin. as the following:

FILES_${PN}-test += "\
   ${bindir}/test-* \
   ${bindir}/simple-* \
   ${bindir}/agent \
   ${bindir}/attest \
   ${bindir}/avtest \
   ${bindir}/bdaddr \
   ${bindir}/btiotest \
   ${bindir}/gaptest \
   ${bindir}/hsmicro \
   ${bindir}/hsplay \
   ${bindir}/hstest \
   ${bindir}/ipctest \
   ${bindir}/l2test \
   ${bindir}/list-devices \
   ${bindir}/lmptest \
   ${bindir}/monitor-bluetooth \
   ${bindir}/mpris-player \
   ${bindir}/rctest \
   ${bindir}/scotest \
   ${bindir}/sdptest \
   ${bindir}/uuidtest \
"

Thanks,
Hongbo
>
> Ross
>
>