[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1223036 - /subversion/trunk/subversion/libsvn_delta/xdelta.c

From: Stefan Fuhrmann <eqfox_at_web.de>
Date: Sun, 25 Dec 2011 18:57:12 +0100

On 25.12.2011 12:04, Bert Huijben wrote:
>> -----Original Message-----

>> Log:
>> Minor xdelta optimization: find short matches at both end of the delta
>> window.
> All buildbots are red after this patch.
>
> At least one test FAILED, checking E:\w2k3\tests\tests.log
> FAIL: svnrdump_tests.py 10: dump: copy from previous version and modify
> FAIL: svnrdump_tests.py 11: load: copy from previous version and modify
> FAIL: svnrdump_tests.py 12: dump: modified in place
> FAIL: svnrdump_tests.py 13: load: modified in place
> FAIL: svnrdump_tests.py 26: dump: copy revprops other than svn:*
> FAIL: svnrdump_tests.py 27: load: copy revprops other than svn:*
> FAIL: svnrdump_tests.py 28: dump: subdirectory
> FAIL: svnrdump_tests.py 29: dump: subdirectory with changes on root
> FAIL: svnrdump_tests.py 32: dump: move parent& modify child file in same rev
> FAIL: svnrdump_tests.py 33: load: move parent& modify child file in same rev

Thanks for noticing! (I thought I had run the tests before committing :/ )

The problem here is that the test compares the actual dump files
produced by svnrdump with the expected ones stored in the test
data folder. With my patch, the text delta is now often smaller, i.e.
different from the expected value. Nevertheless the new diffs seem
to be correct.

So, I will XFAIL the tests in question and $somebody needs to
update the reference dump files.

-- Stefan^2.
Received on 2011-12-25 18:57:57 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.