[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: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2004-06-18 19:35:21 CEST

On Fri, 2004-06-18 at 11:27, kfogel@collab.net wrote:
> Branko Čibej <brane@xbc.nu> writes:
> > >Does someone other than 'svnlook' use this particular limited -r
> > >syntax?
> >
> > "svn up" comes to mind.
>
> 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.

> > Really, that's just a special case of our -r
> > X[:Y] syntax, so you should be able to use the common parser in
> > svnlook and simply bail out if the second rev is given.
>
> Yes, the common parser bails correctly, and the strategy you describe
> above would work. But it's a much more involved change -- we'd have
> to start using svn_opt_revision_t, which means all of a sudden we're
> supporting -r{DATE} syntax in svnlook (!), which means now we're
> adding a new feature, when all I meant to do was fix a bug.
>
> I didn't really want to go down that road.

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.

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

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.