On 12.01.2015 13:54, Julian Foad wrote:
> I (Julian Foad) wrote:
>> The few differences I have found seem to be already known (but not necessarily
>> adequately addressed). These include: svnrdump always sets 'Prop-delta:
>> true' even when unnecessary; svnrdump doesn't output some of the
>> checksum headers that svnadmin does; they put different numbers of blank lines
>> between sections in some cases.
>>
>> In its current state, about 4 or 5 tests fail. A few of these are svnadmin tests
>> that deliberately create an invalid repo, that then fails to dump.
> There were 4 failures in svnadmin_tests.py, all for that reason. The attached, updated patch avoids these spurious test failures, and also has a log message.
>
>> At least one
>> other fail appears to be a bug in the test suite's dumpfile parser.
> The other failure is not a parser bug as I speculated, but an interesting real difference.
>
> svnmucc_tests.py 2 'basic svnmucc tests' generates a no-op modification to an empty file, in revision 16. 'svnadmin dump' outputs this:
> [[[
> I: Node-path: boozle/buz/svnmucc-test.py
> I: Node-kind: file
> I: Node-action: change
> I:
> I:
> ]]]
>
> whereas 'svnrdump dump' outputs this:
> [[[
> I: Node-path: boozle/buz/svnmucc-test.py
> I: Node-kind: file
> I: Node-action: change
> I: Text-delta: true
> I: Text-delta-base-md5: d41d8cd98f00b204e9800998ecf8427e
> I: Text-content-length: 4
> I: Text-content-md5: d41d8cd98f00b204e9800998ecf8427e
> I: Content-length: 4
> I:
> I: SVN
> I:
> ]]]
The latter looks like an empty delta. This could be an artefact of how a
particular RA layer represents a no-op change.
-- Brane
Received on 2015-01-12 16:09:11 CET