Re: svn commit: r10025 - trunk/subversion/svnlook

Greg Hudson <ghudson@MIT.EDU> writes:
> > Er, the whole point was that 'svnlook' does not accept the -rX:Y
> > syntax that other commands accept.
>
> "svn up -r X:Y" isn't accepted either.

D'oh!

/me ducks in embarrassment :)

> Supporting -r{DATE} in svnlook sounds like a fine idea. You weren't
> really fixing a bug so much as tightening up a parser, so your change
> isn't really appropriate for back-porting anyway.

I see what you're saying -- the compat issue doesn't matter so much,
so why not do it?

One question: if we're going to use the regular parser anyway, why not
just accept the full -rX:Y syntax?

The only reason we don't automatically translate

   $ svn diff -r7

into

   $ svn diff -r6:7

is because of the complications introduced by having a working copy
(and thereby implied base revisions). But svnlook doesn't have this
problem, so it could keep its current semantics *and* support -rX:Y.

Any objections?

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jun 18 20:36:29 2004