Ben Collins-Sussman wrote:
>
> On Aug 18, 2005, at 11:18 AM, Max Bowsher wrote:
>
>>
>> I'd appreciate comments from anyone who understands the situation
>> better, and if anyone has a Win9x machine on which they could check
>> that this makes file:/// access to FSFS repositories work, that
>> would be great - although please take note that this is just quick
>> hack, and my assumption that FSFS doesn't need this locking might
>> possibly be incorrect.
>>
>> Max.
>>
>>
>
> Interesting approach. I was discussing this problem with ghudson
> last week in IRC, and was planning to fix it in a very different
> way. ghudson pointed out that it's a design mistake that the
> svn_repos_open() tries to do any locking at all. My thought was that
> the bdb shared-lock should be an implementation detail of
> libsvn_fs_bdb; I was planning to push the shared-lock code down into
> the bdb implementation of svn_fs_open().
Ghudson's right. +1 on that approach.
Of course, the catch here is that svn_fs_recover must handle the
exclusive locking itself.
-- Brane
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 18 19:58:54 2005