Philip Martin <philip.mar...@wandisco.com> writes:

> Philip Martin <philip.mar...@wandisco.com> writes:
>
>> I suppose one way to fix this would be to ensure that every BDB revision
>> generates a new node-revision-id.
>
> To do this we make '/' mutable either when creating the txn, or when
> commiting the txn.  Patches to do both below.  Not sure which one is
> best.

Seems like this behaviour had been observed in the past, with one of my
patches we also need to stop special-casing BDB in ra-tests:

Index: subversion/tests/libsvn_ra/ra-test.c
===================================================================
--- subversion/tests/libsvn_ra/ra-test.c        (revision 1716725)
+++ subversion/tests/libsvn_ra/ra-test.c        (working copy)
@@ -1606,12 +1606,7 @@ commit_empty_last_change(const svn_test_opts_t *op
     SVN_TEST_ASSERT(dirent != NULL);
     SVN_TEST_STRING_ASSERT(dirent->last_author, "jrandom");
 
-    /* BDB only updates last_changed on the repos_root when there is an
-       actual change. Our other filesystems handle this differently */
-    if (!opts->fs_type || !strcasecmp(opts->fs_type, "BDB"))
-        SVN_TEST_ASSERT(dirent->created_rev == 1);
-    else
-        SVN_TEST_ASSERT(dirent->created_rev == 2);
+    SVN_TEST_ASSERT(dirent->created_rev == 2);
 
     return SVN_NO_ERROR;
 }


-- 
Philip

Reply via email to