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

Re: svn commit: r1656893 - in /subversion/trunk/subversion: svnsync/sync.c tests/cmdline/svnsync_tests_data/mergeinfo-contains-r0.dump tests/cmdline/svnsync_tests_data/mergeinfo-contains-r0.expected.dump

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Wed, 4 Feb 2015 17:45:15 +0000

So we're on the third version of the code and third version of the test, for a tiny edge-case feature. Clearly it's fragile. I had a bad feeling about writing it this way in the first place.

I have proceeded to implement 'svnsync' cross-checking in the test suite: running every repository produced by a test through 'svnsync' and checking there is no change.

This turns up at least one further bug: assertion failure on deleting a mergeinfo property.

I'll get to fixing that and committing the extra testing tomorrow.

- Julian
Received on 2015-02-04 18:48:11 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.