Re: [atomic-revprops] status update

2010-08-11 Thread Daniel Shahaf
Julian Foad wrote on Wed, Aug 11, 2010 at 11:33:49 +0100: > On Wed, 2010-08-11, Stefan Sperling wrote: > > The BRANCH-README suggest that we add an interface like > > 'svn propedit --revprop pname pval2 --old-value=pval'. > > I don't think there is a valid use case for non-atomic revprop changes.

Re: [atomic-revprops] status update

2010-08-11 Thread Julian Foad
On Wed, 2010-08-11, Stefan Sperling wrote: > The BRANCH-README suggest that we add an interface like > 'svn propedit --revprop pname pval2 --old-value=pval'. > I don't think there is a valid use case for non-atomic revprop changes. > Why not just make all revprop changes atomic by default if the

Re: [atomic-revprops] status update

2010-08-11 Thread Stefan Sperling
On Wed, Aug 11, 2010 at 03:51:38AM +0300, Daniel Shahaf wrote: > (hitted "send" too soon) > > Daniel Shahaf wrote on Wed, Aug 11, 2010 at 03:40:51 +0300: > > For the last few weeks I've been working on the atomic-revprop branch. > > > > (Its goal is port the new svn_fs_change_rev_prop2() to libsv

Re: [atomic-revprops] status update

2010-08-10 Thread Hyrum K. Wright
On Tue, Aug 10, 2010 at 8:28 PM, Daniel Shahaf wrote: > Hyrum K. Wright wrote on Tue, Aug 10, 2010 at 20:10:54 -0500: >> Do you think the work is sufficiently non-disruptive to be moved back to >> trunk? > > Yes. > > Updated plan: I'll do the "marshal error codes in ra_dav" work on trunk, > re-te

Re: [atomic-revprops] status update

2010-08-10 Thread Daniel Shahaf
Hyrum K. Wright wrote on Tue, Aug 10, 2010 at 20:10:54 -0500: > Do you think the work is sufficiently non-disruptive to be moved back to > trunk? Yes. Updated plan: I'll do the "marshal error codes in ra_dav" work on trunk, re-test/review the status of the ra_dav part of the branch, and merge th

Re: [atomic-revprops] status update

2010-08-10 Thread Hyrum K. Wright
On Tue, Aug 10, 2010 at 7:51 PM, Daniel Shahaf wrote: > (hitted "send" too soon) > > Daniel Shahaf wrote on Wed, Aug 11, 2010 at 03:40:51 +0300: >> For the last few weeks I've been working on the atomic-revprop branch. >> >> (Its goal is port the new svn_fs_change_rev_prop2() to libsvn_ra, which >

Re: [atomic-revprops] status update

2010-08-10 Thread Daniel Shahaf
(hitted "send" too soon) Daniel Shahaf wrote on Wed, Aug 11, 2010 at 03:40:51 +0300: > For the last few weeks I've been working on the atomic-revprop branch. > > (Its goal is port the new svn_fs_change_rev_prop2() to libsvn_ra, which > should allow callers to specify both a value and an optional