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

Re: svn commit: r33735 - trunk/subversion/tests/cmdline/svntest

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Mon, 20 Oct 2008 09:56:32 +0100

On Sat, 2008-10-18 at 16:28 -0500, Hyrum K. Wright wrote:
> Neels J Hofmeyr wrote:
> >
> > Hyrum K. Wright wrote:
> >> neels_at_tigris.org wrote:
> >>> Author: neels
> >>> Date: Sat Oct 18 11:17:04 2008
> >>> New Revision: 33735
> >>>
> >>> Log:
> >>> Enable number range arguments for cmdline python tests.
> >>>
> >>> * subversion/tests/cmdline/svntest/main.py
> >>> (usage): Update online help.
> >>> (run_tests): Add detection for number range arguments like `2-35'.
> >> I like this idea!

+1.

> >> <bikeshed>
> >> Perhaps we should use `2:35', given our syntax for revision ranges and such?

Nah. They are two different meanings. Although the "-r X:Y" syntax means
"the changes numbered X through Y" in the "svn log" command, that is an
anomaly because in other commands it specifies "the interval between rX
and rY", which is not the meaning we are looking for here.

> > You know, I thought just the same, when coincidentally reading the svn log
> > help right after committing. I changed it to allow both. Even if this became
> > a snowball, it wouldn't matter much I guess. Not like `setprop' at least ;)
>
> Looks good. If we can support both styles, we might as well.

It's only us developers that use this UI, so... whatever.

Thanks.
- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-10-20 12:15:26 CEST

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