Re: unrecognized option '--help

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, "D(dot) S(dot)" <spider(at)skuggor(dot)se>, pgsql-bugs(at)postgresql(dot)org
Subject: Re: unrecognized option '--help
Date: 2015-05-22 02:13:19
Message-ID: 20150522021319.GG2028@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On 2015-05-21 21:59:56 -0400, Tom Lane wrote:
> As I recall, Alvaro's argument for this was "I typed multiple words of a
> command and then want to check syntax, so I add --help to the end of what
> I'd already typed and hit return, with the idea of recalling the command
> and deleting the --help off the end so I don't have to retype what I
> already entered."

Something around that, yes.

> This use-case is only going to work reliably if --help is recognized
> regardless of what's in front of it. Otherwise, if you're right in
> suspecting that you got something wrong, getopt parsing will fail
> before it gets to your --help --- and what it will print is "please
> use --help", which is exactly the symptom being complained of here.

I don't think it really is the symptom complained about here. Right now
"vacuumdb dbname --verbose" works (i.e. recognizes verbose as an
option), whereas "vacuumdb dbname --help" doesn't. The latter is what's
complained about here. And the reason for that is that
--help/-?/--version/-v aren't part of the getopt_long() call.

Sure, vacuumdb --dbname --help or something like that isn't going to
work, but I think that's fairly minor in comparison to the above. And
much easier to understand.

> As I said, maybe that's okay. It'd certainly be 99.99% okay ... but
> the other hundredth of a percent could be pretty painful.

I can't see how treating it like the other options we already have could
make it more painful?

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2015-05-22 02:24:57 Re: unrecognized option '--help
Previous Message Tom Lane 2015-05-22 02:08:42 Re: unrecognized option '--help