You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2010/09/22 11:53:58 UTC

[MERGE] atomic-revprops done.

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

Re: [MERGE] atomic-revprops done.

Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
Daniel Shahaf wrote on Wed, Sep 22, 2010 at 13:53:58 +0200:
> 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.

Done in r1000693.