Julian Foad wrote on Mon, Feb 27, 2012 at 10:19:35 +0000: > Daniel Shahaf wrote: > >> danie...@apache.org wrote: > >> > Fix issue #4121: svnsync of a copy followed by delete of > >> > was-unreadable child. > >> > http://subversion.tigris.org/issues/show_bug.cgi?id=4121 > >> > > >> > * subversion/libsvn_repos/replay.c > >> > (was_readable): New helper. > >> > (path_driver_cb_func): > >> > Use new helper to determine whether to replay or discard > >> > deletions. > >> > >> I've been trying to write a regression test for this, but got stuck > >> converting my shell script to Python. > >> > >> I have a shellscript (attached) that reproduces the issue for me: with > >> r1293997 it outputs > >> > >> Copied properties for revision 1. > >> subversion/libsvn_ra_svn/client.c:2440: (apr_err=210008) > >> svnsync: E210008: Error while replaying commit > >> zsh: exit 1 ./01repro > >> > >> I am also attaching an attempt at an equivalent Python test. For me the > >> Python test passes against r1293997. Can anyone spot the difference? > > Your Python test fails against r1293997 and passes against r1293998 for me. > The fail against r1293997 looks like: > > [...] ... > /home/julianfoad/src/subversion-c/subversion/libsvn_ra_neon/replay.c:181: > (apr_err=160028) ... > svnsync: E160028: '/branch/A' is out of date > [...] > > - Julian
Thanks for your help. Between this and discussions with philipm on IRC, we've tracked down the problem to a difference in svnserve/mod_dav_svn's expectations from authz files which the test suite didn't account for. (More precisely, teh test I modeled my new test after was in itself silently broken; the other svnsync_tests were fine.) All fixed now, and I'll see about updating docs too. Thanks to you and Philip for the help. Daniel
