[SVNMERGE][PATCH] svnmerge rollback

[Madan U Sreenivasan]

On Fri, 19 May 2006 22:00:21 +0530, Archie Cobbs <archie at dellroad.org>  
wrote:

> Giovanni Bajo wrote:
>> - Change "svnmerge init" so that it doesn't record an initial range of  
>> 1-REV
>> (where REV is the base rev for the branch), but just REV. After this,  
>> we can
>> always assume that the lowest-numbered revision in the revisionset of
>> integrated properties is the revision at which the branch was created  
>> (and
>> we can forbid it to be rolledback for instance).

I second Giovanni, we need to internally only maintain N-REV as the  
svnmerge-integrated property, where N is the branch creation revision.
It will help us intrinsically maintain the revision at which the source  
was created without having to query for it everytime.

> This doesn't seem right. It seems like you'd be changing the semantics
> of the property. Revisions 1-N have in fact been merged, albeit  
> trivially.

Right, so the display_revision() functions should *understand* N-REV as  
1-REV and display hence.
The user always sees 1-REV as the integrated revision range. This would  
not change.

> If you just need to avoid an svn call, why not instead record the first
> revision "N" explicitly as an additional bit of information.

nah, store N-REV. Why some extra bytes?

> Either way, you'll need compatibility and upgrade code, but the latter
> approach seems cleaner and more consistent.

Giovanni's detect 1-REV style and silently change it to N-REV sounds a  
good approach to me.

Regards,
Madan.