I believe the branch is complete.  The new API (svn_ra_change_rev_prop2())
is in place, fully implemented over all layers, along with Python tests,
and code in svnsync, svnrdump, and svn_client_revprop_set2() uses it.
(The new test is prop_tests.py 34, atomic_over_ra().)

A diff between trunk and the branch is available here:

    http://people.apache.org/~danielsh/atomic-revprop-rN.diff
    http://people.apache.org/~danielsh/atomic-revprop-rN.log.txt

A couple of optional, non-core features have not been implemented (nor, to
my knowledge, requested).  They are listed in the "Potential work" section
of the BRANCH-README, and repeated here:

[[[
Potential work
==============

* svn_log__ interface
  - extend svn_log__change_rev_prop() with a "was old value provided?" boolean

* repos layer hooks interface
  - pass the old value to pre-revprop-change

* svn propedit --revprop --old-value
  (Not planned currently.)
]]]

I'll wait a few days for any comments, objections, reviews, etc., and if
I don't hear anything I'll then merge the branch to trunk.

Thanks to Jon Foster for coming up on Monday with idea and patches that
overcame the last major obstacle (preservation of err->apr_err in ra_dav)
and to Stefan Sperling for his help throughout the life of this branch.

Daniel

Reply via email to