On Thu, 2024-09-12 at 11:11 -0400, Luiz Augusto von Dentz wrote:
> Hi Bastien,
>
> On Thu, Sep 12, 2024 at 5:29 AM Bastien Nocera <hadess@xxxxxxxxxx>
> wrote:
> >
> > Exit after handling --help, so as to avoid the daemon waiting to
> > communicate with a D-Bus service that might not be running.
> > ---
> > src/shared/shell.c | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/src/shared/shell.c b/src/shared/shell.c
> > index c31487190d0f..f7e8b3300373 100644
> > --- a/src/shared/shell.c
> > +++ b/src/shared/shell.c
> > @@ -1324,9 +1324,7 @@ void bt_shell_init(int argc, char **argv,
> > const struct bt_shell_opt *opt)
> > return;
> > case 'h':
> > usage(argc, argv, opt);
> > - data.argc = 1;
> > - data.argv = &cmplt;
> > - data.mode = 1;
> > + exit(EXIT_SUCCESS);
>
> This would undo shared/shell: Print commands when --help option is
> given, so you might want to add print_cmds call.
That wasn't enough, because the menu info isn't populated at this
point, so we'd only get the top-level help. It was quite a bit more
involved, but I hope I got all variants covered in the v2 patchset.
Cheers
>
> > goto done;
> > case 's':
> > if (optarg && data.init_fd < 0) {
> > --
> > 2.46.0
> >
> >
>
>
