RE: svnsync crash on empty repo
Date: Sat, 5 Sep 2015 13:15:44 +0200
I don't think we should fix this with a 'revision+1' to explicitly allow many bad ranges, but I do think we should specifically fix the r0 case for empty repositories.
Sent from Mail for Windows 10
From: Julian Foad
On trunk (released versions are unaffected), "svnsync sync" with an
subversion/libsvn_ra/ra_loader.c' line 1198: assertion failed
because start_revision is 1 and end_revision is 0.
The assertion was added in r1665480
One possible fix is to avoid calling it with an empty revision range, like this:
- if (from_latest < last_merged)
That makes the caller return early when there are no revisions to
However, I think we need to continue allowing the historical usage of
- && start_revision <= end_revision
I haven't yet seen where the problem or restriction existed in some RA
The attached patch contains both possible fixes, a log msg and a
(I found this by running tests with the "--dump-load-cross-check"
This is an archived mail posted to the Subversion Dev mailing list.