On Sun, 23 Apr 2017 12:22:57 +0200 Federico Vaga <federico.v...@vaga.pv.it> wrote:
> For some reason the list command does not use anymore `getopt()` > to parse the arguments, instead it uses a custum implementation. > > During this change [5da0eff trace-cmd: Add regex for listing of events] > the variable `optind` has been forgotten. > > To reproduce the problem try to use invalid arguments. The application > will not report the correct invalid argument > > $ ./trace-cmd list -a > list: invalid option -- 'i' > > Signed-off-by: Federico Vaga <federico.v...@vaga.pv.it> > --- > trace-cmd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/trace-cmd.c b/trace-cmd.c > index 1a62faa..a05df92 100644 > --- a/trace-cmd.c > +++ b/trace-cmd.c > @@ -706,7 +706,7 @@ int main (int argc, char **argv) > break; > default: > fprintf(stderr, "list: invalid option > -- '%c'\n", > - argv[optind][1]); > + argv[i][1]); Hmm, I had this fixed back in September. I haven't push in a long time. Well, I applied your other patches (Thanks!). I'm going to finally start updating trace-cmd this week. -- Steve > usage(argv); > } > }