Re: Issue 3895, rejecting invalid svn:mergeinfo on commit

Stefan Sperling <stsp_at_elego.de> writes:

> We should allow invalid mergeinfo to exit the repository, e.g. in a dump
> file. But input paths, such as svnadmin load, should refuse invalid mergeinfo.

We have a similar problem with invalid svn:log (non-LF line-ends). In
that case load fails with an error but sync corrects the value and
completes the load. Simply refusing to sync was deemed insufficient.

> Maybe we should provide a way to fix invalid mergeinfo in dump files.
> One existing but no officially supported method is svndumptool
> http://svn.borg.ch/svndumptool/

I've never used it, but I suppose it would work. Is that a good enough
solution? Invalid mergeinfo is harder to fix than an invalid log
message, because the validity rules are much more complex. Do we advise
users to delete it? To analyse the failure and tweak it to be valid?

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com