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

Re: Deletion revision in Ev2

From: Hyrum K Wright <hyrum.wright_at_wandisco.com>
Date: Mon, 7 May 2012 14:02:42 -0500

On Mon, May 7, 2012 at 1:43 PM, Greg Stein <gstein_at_gmail.com> wrote:
> On Mon, May 7, 2012 at 12:37 PM, Hyrum K Wright
> <hyrum.wright_at_wandisco.com> wrote:
>>...
>> In this case, we are legitimately deleting B/C, but the "revision this
>> thing is being deleted from" doesn't exist until commit finalization.
>
> True. I figured as the FS, we always had a revision... but that's not true :-P

Yep. I'm going mostly off test failures here; there may be other
(pathologic) cases which are still valid, but are not captured in the
tests.

>>...
>> Simply eliminating the check in editor.c:delete_cb(), and also
>> changing can_modify() to allow invalid revision to act as a wildcard
>> (i.e., "don't bother checking, I know this should be modifiable")
>> makes the test pass, but I don't know that that's the generally
>> correct solution.
>
> It is valid. I'll make the necessary changes.

Thanks!

-Hyrum

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/
Received on 2012-05-07 21:03:14 CEST

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.