On 26.01.2015 15:50, Julian Foad wrote: > Branko Čibej wrote: >> On 26.01.2015 14:13, Julian Foad wrote: >>> I attach the latest version of my dump/load cross-checking patch. >>> >>> Do we want to commit this? I think we should. It has been very useful in >> uncovering dump/load bugs and inconsistencies. >> >> Absolutely +1, more tests can't hurt. > Excellent. Is anyone able to lend a hand with one of these two of the tasks I > listed: > > - Bypass authz when dumping with svnrdump, otherwise a partial dump is > obtained for tests using authz, making those tests fail.
Hmmm, this will involve changing the DAV and svnserve autocheck configurations, I think. Not really my home turf, but I can try. > - Make optional -- enabled by a configure option? I can certainly do this bit, I had a go at similar tweaks when I added the option to skip the C tests and to run with pre-cooked v4 and v6 FSFS repositories. > if I do this one: > > - Put back the dumpfile parser's ability to check for well-formed dumpfile > contents, which has been temporarily stripped out here in order to > achieve more generic parsing, as some dump/load tests rely on that. May I suggest we do this on a branch? So that we have the "make optional" bit in place before all trunk testing grinds to a horrible slowdown. :) -- Brane