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

Re: FSFS failing regression tests?

From: Branko Čibej <brane_at_xbc.nu>
Date: 2005-08-09 23:15:46 CEST

C. Michael Pilato wrote:

>David James <james82@gmail.com> writes:
>
>
>
>>On 8/4/05, Michael Brouwer <michael@tlaloc.net> wrote:
>>
>>
>>>If the user enters a log message stating
>>>
>>>* foo deleted
>>>* bar modified
>>>
>>>And someone else has already deleted foo, allowing this transaction
>>>to go through will result in an incorrect log message since someone
>>>else already deleted foo. I'd say that that warrants giving the user
>>>an out of date error and letting him/her fix the log message before
>>>committing.
>>>
>>>
>>Good argument! I'm convinced.
>>
>>Shall we fix the test to reflect the new behaviour? fs-test looks like
>>it's still failing
>>
>>
>
>fs-test is failing for different reasons altogether, entirely
>unrelated to merges of double-deletes.
>
>Further, despite attempts to make corrections to folks along the way,
>I'm 100% convinced that either:
>
> - most of the participants in this discussion have absolutely no
> idea what is and isn't actually going on, when this double-delete
> consideration does and doesn't actually come into play, and so
> on, or
>
> - folks are just talking about some future behavior of Subversion
> which was not the original intent of this thread.
>
>And so, just in case the former is true, I repeat: the disallowing of
>double-deletes by svn_fs_commit_txn() only happens when some other
>user slips in a delete of something I wanted to delete *between the
>time I start my commit transaction and the time the pre-commit
>auto-merge code is run*.
>
>
Yup. That's why I believe that the FSFS implementation of the merge is
wrong. Granted that log messages could be made to "lie", I don't believe
svn_fs_merge correctness should be subservient to possible log message
inconsistencies.

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Aug 9 23:17:35 2005

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.