Hi Lucas,
>>>>> Having no description doesn't mean it's not an available command. So add
>>>>> command to the output of "help" command, even if it has no description.
>>>>> ---
>>>>> client/main.c | 5 ++---
>>>>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>>>>
>>>>> diff --git a/client/main.c b/client/main.c
>>>>> index b0a66d8..54af807 100644
>>>>> --- a/client/main.c
>>>>> +++ b/client/main.c
>>>>> @@ -1182,10 +1182,9 @@ static void rl_handler(char *input)
>>>>> printf("Available commands:\n");
>>>>>
>>>>> for (i = 0; cmd_table[i].cmd; i++) {
>>>>> - if (cmd_table[i].desc)
>>>>> - printf("\t%s %s\t%s\n", cmd_table[i].cmd,
>>>>> + printf("\t%s %s\t%s\n", cmd_table[i].cmd,
>>>>> cmd_table[i].arg ? : " ",
>>>>> - cmd_table[i].desc);
>>>>> + cmd_table[i].desc ? : "");
>>>>
>>>>
>>>> if I remember this correctly, then I was doing it to be able to hide
>>>> commands from the help output. Mainly commands that are on purpose
>>>> duplicates.
>>>
>>> So the right fix is to add the missing descriptions then? Lucas, can you
>>> provide an updated patch that does this?
>>
>> actually I wanted to hide these commands on purpose.
>
> So you can't pair a device starting from remote using bluetoothctl on
> purpose? Or is adding a description to at least "default-agent" ok?
is this for the "default-agent" command? I forgot to add a description for that one?
Regards
Marcel
--
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
