Philip Martin <philip.mar...@wandisco.com> writes: > Julian Foad <julianf...@btopenworld.com> writes: > >> Hi Philip. I've updated the 1.8.x nomination of issue #4476 - svnsync >> with r0 mergeinfo - to resolve the concern that's the subject of your >> veto. Please could you review and remove your veto if >> possible. Thanks. > > The new test, svnsync_tests.py 30, is a FAIL.
Something has really gone wrong, the source repository for the sync has the wrong content. On trunk I see: $ svnlook proplist -v svn-test-work/repositories/svnsync_tests-30 /D1 Properties on '/D1': svn:mergeinfo /a:1 $ svnlook proplist -v svn-test-work/repositories/svnsync_tests-30 /D2 Properties on '/D2': svn:mergeinfo /a:1 /b:1*,2 On the 1.8.x branch I see: $ svnlook proplist -v svn-test-work/repositories/svnsync_tests-30 /D1 Properties on '/D1': svn:mergeinfo $ svnlook proplist -v svn-test-work/repositories/svnsync_tests-30 /D2 Properties on '/D2': svn:mergeinfo /b:2 The load is stripping some of the mergeinfo from D1, D2 and D3. Ah! This is issue 4538. The test requires an svnadmin load that has that fixed. -- Philip Martin | Subversion Committer WANdisco // *Non-Stop Data*