[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Patch to add a --verbose option.

From: Mo DeJong <supermo_at_bayarea.net>
Date: 2001-09-13 20:57:28 CEST

On 13 Sep 2001 10:47:04 -0400
"Paul D. Smith" <pausmith@nortelnetworks.com> wrote:

> %% Mo DeJong <supermo@bayarea.net> writes:
>
> md> After poking around in the and testing out the apr_getopt_long
> md> function, I have come to the conclusion that it is not currently
> md> possible to implement the --verbose option as described in the
> md> clients/cmdline/README file.
>
> md> -V --verbose [ KEY ] [quiet|progress|verbose|trace]
>
> I think this might be a typo.
>
> The GNU standards state that arguments given to long options use "=", so
> it should be:
>
> > -V --verbose[=KEY] [quiet|progress|verbose|trace]
>
> As an example, see the next line:
>
> md> -q --quiet alias for --verbose=quiet

That does not matter. The apr_getopt_long function supports both
"--option foo" and "--option=foo" style argument passing. It does not
support an option that may or may not take an argument, an option
either accepts an argument or it does not.

If you declared that --verbose takes an argument and then run a command
like:

svn status --verbose

It would error out inside apr_getopt_long saying that an argument is
required for the --verbose option. That is why I went with the --verbose
and --quiet options. If we added a --trace option later on all 4 levels
of verbosity mentioned in the README could be supported.

cheers
Mo

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:41 2006

This is an archived mail posted to the Subversion Dev mailing list.