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

Re: [PATCH] Add a test in dav-mirror-autocheck.sh to showcase a out-of-date slave related bug

From: Mark Phippard <markphip_at_gmail.com>
Date: Mon, 18 Apr 2011 07:08:06 -0400

On Mon, Apr 18, 2011 at 6:51 AM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
> On Mon, 18 Apr 2011 16:05 +0530, "Arwin Arni" <arwin_at_collab.net> wrote:
>> Hi All,
>>
>> This patch adds a test case to
>> subversion/tests/cmdline/dav-mirror-autocheck.sh to showcase the
>> following bug.
>>
>> In a master/slave repositories setup where writes are proxied to the
>> master and reads are handles by the slave repository, if the slave
>> repository is behind the master by one or more revisions, certain
>> operations that one would expect to succeed are failing.
>>
>
> Why are you expecting it to succeed?
>
> Consider:
>
> % svn co ^/ wc
> % cd wc
> % svn mkdir -mm ^/branches/foo
> % svn cp trunk branches/bar
> % svn ci -mm
>
> which should fail with 'out of date'.
>
> Therefore I think your case should fail as well.

I know why it fails, but I would not expect it to fail as a user, even
with a proxy. I did not look at Arwin's test but it does not require
a WC to show the failure. This also fails:

$ svn mkdir url://branches/branch1
$ svn mkdir url://branches/branch2

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
Received on 2011-04-18 13:08:40 CEST

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