Re: [PATCH] Support both BDB 4.0 and 4.1

Philip Martin wrote:

>Philip Martin <philip@codematters.co.uk> writes:
>
>
>
>>Branko ÄŒibej <brane@xbc.nu> writes:
>>
>>
>>
>>>I think we shouldn't announce support for DBD 4.1 just yet; I'm
>>>attaching this patch in the hope that people will test it on Unix
>>>boxes. I'm testing it on Windows right now (with both BDB versions),
>>>and will commit once some Unix test reports come in.
>>>
>>>
>>I have built this (a shared build) and run the regression tests over
>>ra_local, the ra_dav tests are running right now. BDB 4.1 needs an
>>APRUTIL more recent than that in Apache 2.0.42.
>>
>>
>
>The ra_dav regression tests fail. I've spent a while rebuilding
>things to try to make sure I haven't done anything stupid, the tests
>work with BDB 4.0 and fail with BDB 4.1. The only difference in the
>builds is the path given with --with-berkeley-db to Subversion and
>Apache/APR. I'm using Subversion r3421 with Branko's patch and
>Apache/APR from a couple of hours ago.
>
>The tests fail when mod_dav_svn writes to a repository, usually during
>the initial import of the greek tree. For example, this works with
>BDB 4.0 and fails with BDB 4.1
>
>$ svnadmin create /tmp/repo
>$ svn mkdir file:///tmp/repo/foo1
>
>Committed revision 1.
>$ svn mkdir http://localhost:8888/tmp/repo/foo2
>../svn/subversion/libsvn_ra_dav/util.c:163: (apr_err=120022, src_err=0)
>svn: APR does not understand this error code
>svn:
>failed to open activity db; check repos perms.
>
>The error is coming from dav_svn_store_activity in mod_dav_svn,
>apr_dbm_open fails to open /tmp/repo/dav/activities.
>
Hm, obviously we'll have to patch APR, too. Damn them for breaking the
API in a minor release, anyway.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org