Re: [Bug-sysutils] argp patch for chsh

[Marco Gerards]

Barry deFreese <address@hidden> writes:

> Whoops, I was missing my .cvsrc on that last one.  Here is a diff -u
> copy.. Sorry.

Great!  Keep up the good work. :)

>       /* If we have no options, enter interactive mode */
> -     if (optind == 1) {
> +     if (argc == 1) {
>               fprintf(stdout,
>                       _("Changing the login shell for `%s'\n"
>                         "Enter the new value, or press enter to keep "

I don't fully understand the code, but in this case interactive mode
(right?)  is not used when I run `chsh marco'.  How about using this:

if (args->shell == NULL)
....

--
Marco