[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[MERGE] atomic-revprops done.

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 22 Sep 2010 13:53:58 +0200

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
Received on 2010-09-22 13:55:00 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.