Re: svn diff interface

Greg Hudson <ghudson@MIT.EDU> writes:

> The "dumb" proposal is:
>
> Iterative case: svn diff [-r {N|N:M}] TARGET ...
> Comparative case: svn diff2 TARGET TARGET
>
> This interface allows for "svn diff" between two wcpaths, and isn't
> heuristic at all, but does require users to remember which command
> does what they want.

Is "remembering which command does what" a problem? If we come up
with a complicated one command does all, how often are users going to
get the arguments wrong and get the wrong diff?

Given that the iterative case appears to have support, I favour the
(not so) "dumb" proposal above. For a start it is easy to document

    svn diff compares different revisions for each target
    svn diff2 compares the targets against each other

That's two lines, try explaining any other proposal in two lines.

I think the command synopsis is

  svn diff [-r { N | :M | N:M }] [TARGET ...]

  svn diff2 [-r { N | :M | N:M }] TARGET1 [TARGET2]

and we could even drop the TARGET@REV stuff.

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