Re: 1.8.x backport issue #4476 - svnsync with r0 mergeinfo - resolve the veto

Philip Martin <philip.martin_at_wandisco.com> writes:

> Julian Foad <julianfoad_at_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*