Re: [PATCH] Support both BDB 4.0 and 4.1

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.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org