Re: Issue #4476 - Mergeinfo containing r0 makes svnsync and svnadmin dump fail
From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Thu, 4 Dec 2014 17:55:44 +0000
Julian Foad wrote:
I have done this, and proposed it for backport to 1.7 and 1.8.
(My fix adds a new enumeration constant to an enumeration type, for the warning. For back-porting, I renamed this constant to a double-underscore name to indicate it's not for public use in the 1.7/1.8 APIs.)
> I'll come to 'svnsync' later, but basically my current thought is it
I have written a test for this, and hacked up code to do this by textual substitution. It isn't quite right -- for example it would go wrong if a path contained the character sequence ":0", among other issues. I ought to rewrite it in the form of a proper mergeinfo parser but one that is more lenient than the regular parser.
I also noticed that loading from a dump file also modifies mergeinfo in other ways such as removing references to r1. I don't think is a good idea: I think there should be a way to load exactly what is in the dump. But that's a different, though related, issue.
This is an archived mail posted to the Subversion Dev mailing list.