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

Re: Diff options

From: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2001-09-04 18:32:07 CEST

> My idea was to follow the second approach, adding an UNKNOWN_OPTIONS
> variable to each subcommand's svn_cl__cmd_desc_t. Then after we
> have determined the subcommand, we error out if that subcommand
> doesn't expect unknown options.

As Ben said, this way lies disaster if we ever want to add our own
options to the diff command (or, the way we have it now, to the global
space).

I know it's gross, but I think if we want to be able to pass
unrecognized options to the diff command it will have to be explicit
(--diff-options=--expand-tabs). We can recognize commonly-used
options like -c and -u natively, of course.

(Incidentally, just so my position is clear on option spaces, I am
happy with making some options work differently for different
commands; I think it's okay for "svn diff -u" to do something
different than "svn co -u". But I have never been happy with the CVS
practice of making options mean one thing before the command name and
a different thing after.)

---------------------------------------------------------------------
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:39 2006

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