You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by "C. Michael Pilato" <cm...@collab.net> on 2012/12/21 18:32:00 UTC

Re: svn commit: r1425042 - in /subversion/trunk: ./ subversion/svnrdump/dump_editor.c

On 12/21/2012 12:27 PM, cmpilato@apache.org wrote:
> Author: cmpilato
> Date: Fri Dec 21 17:27:03 2012
> New Revision: 1425042
> 
> URL: http://svn.apache.org/viewvc?rev=1425042&view=rev
> Log:
> Reintegrate the 'issue-4116-dev' branch into trunk, fixing (as far as
> I can tell) issue #4116 ("svnrdump doesn't work with serf").

I'd really appreciate any extra eyes I could get on this stuff -- at the
code level is fine (great, even), but even just some adhoc testing would be
appreciated.  If/as you test these results, keep in mind that the failure
case that spawned the issue was only triggered when running 'svnrdump dump'
with a revision range (that didn't start with r0) and *not* in incremental
mode.  That's the only time 'svnrdump dump' invokes the wonky ra_serf
"update" logic.  Thanks!

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development