Re: Mergeinfo overwritten from successive merges

Philip Martin <philip.martin_at_wandisco.com> writes:

> Bert Huijben <bert_at_qqmail.nl> writes:
>
>> Can you see if this is related to httpv1 vs httpv2.
>>
>> I'm guessing that in v1 the checkout request would have resulted in
>> the out of date error.
>
> It doesn't happen with v1, the commit is rejected.

It seems to me that mod_dav_svn should not be solely responsible here.
The FS layer has ood conflict checks that should have rejected the
commit but they do not get triggered because mod_dav_svn creates the
transaction based on HEAD rather than on the revision in the working
copy. If I tweak the transaction creation so that it is based on the
working copy revision, rather than HEAD, then the v2 commit fails on
MERGE with an ood error.

I'm a bit surprised that we chose to have mod_dav_svn create
transactions based on HEAD rather than having the client supply the
revision from the working copy. It means we have to reimplement enough
of the ood checks to compensate and making sure we implement enough is
tricky, as demonstrated in this case.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*