Re: PATCH: client shows help instead of certain specific error messages

["Dave Cridland [Home]" <da...@cridland.net>]

On Wed, 2003-04-09 at 17:55, Robert Pluim wrote:
> Brian Denny writes:
>  > On Wed, Apr 09, 2003 at 12:30:21AM +0100, Julian Foad wrote:
>  > > 
>  > > This is what it wanted to display (the code in diff-cmd.c sets up this 
>  > > error message):
>  > > 
>  > > >~/src/subversion> svn diff http://any.url/
>  > > >subversion/clients/cmdline/diff-cmd.c:92: (apr_err=205000)
>  > > >svn: Client error in parsing arguments
>  > > >svn: Second URL is required.
>  > 
>  > hm, i wonder whether we should still show the help anyway.  on the one
>  > hand, usually after i get an option-parsing error (with any program) the
>  > first think i'll want to do is look up the correct usage.  on the other
>  > hand, i can always type 'svn help diff' myself.

> Normally I'd agree with this, but the help output for svn diff is,
> uhhm, "substantial" is probably the best word.

Why not simply append "Type `svn help diff' for usage.", or something
similar, as happens when you type "svn" or "svn aardvark".

>  > bikeshed, anyone?
> 
> I'm partial to purple, myself.  With yellow polka dots.  Goes with my
> socks.

I'd like to propose abandoning the bikeshed concept entirely, and
instead redesigning the bike to be rain-resistant.

Dave.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org