Re: svn commit: r1653988 - in /subversion/trunk/subversion: svnrdump/load_editor.c tests/cmdline/svnrdump_tests.py

I (Julian Foad) wrote:
> I (Julian Foad) wrote:
>> Bert Huijben wrote:
>>>> URL: http://svn.apache.org/r1653988
>>>> Log:
>>>> Fix issue #4551 "svnrdump load commits wrong properties, or fails, on a
>>>> non-deltas dumpfile", [...]
>>>
>>> I also see new test failures on the two new tests for ra_serf.
>>
>> It's another real bug in svnrdump. It calls
>> commit_editor->change_dir_prop() twice for the same property name, [...]

> r1654186 adds such a regression test. [...]

> [Since http://svn.apache.org/r1654194 ] tests for this issue now pass,
> although svndumpfilter is still violating the editor rules.

r1654271 fixes svnrdump's bad editor driving.

[...]
> We should definitely backport the issue #4551 fixes when they're ready.

I have proposed the fixes for this issue for backport to 1.8.x and 1.7.x.

- Julian
Received on 2015-01-26 15:14:27 CET