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

Re: [SVNMERGE][PATCH] handle -r X-Y, where X > Y

From: Archie Cobbs <archie_at_dellroad.org>
Date: 2006-05-16 04:13:26 CEST

Giovanni Bajo wrote:
> Madan U Sreenivasan <madan@collab.net> wrote:
>>> [[[
>>> Accept -rX-Y, for all svnmerge commands, where X > Y.
>>> Before this patch, empty RevisionSet()s were created when X > Y.
>
> Rationale? "svn merge" accepts the reversed order with a different semantic
> (reversed merge). I'm worried about the confusion that can issue. I am +1 on a
> patch that errors out when X > Y until there is agreement on how to best handle
> this.

I agree... arguably revisions "456-123" more likely means "the empty set".
Automated tools that use svnmerge might emit such reversed ranges,
and they would probably expect the empty set too.

This reminds me of an annoying problem I discovered in MySQL today,
which is that you can't say "... WHERE foo IN ( )" .. i.e, query
membership in an empty set (this is what some automatically generated
query strings may try to do).

-Archie

__________________________________________________________________________
Archie Cobbs * CTO, Awarix * http://www.awarix.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 16 04:12:32 2006

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