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

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

From: Branko Čibej <brane_at_xbc.nu>
Date: 2004-06-19 14:28:54 CEST

kfogel@collab.net wrote:

>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?
>
>
None at all, as long as the single-rev variant's semantics don't change.
But I don't have to tell /you/ that, do I. :-)

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Jun 19 14:29:33 2004

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