Re: svn commit: r1574868 - dump with r0 mergeinfo
From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Tue, 11 Mar 2014 15:11:45 +0000 (GMT)
[switching back to plain text]
Branko Čibej wrote:
We're talking about two different warnings.
There is a warning about mergeinfo referring to pre-dump revisions. This is only meaningful, and only checked for, when dumping a range of revisions starting after r1.
There is now the potential to add a new warning that says, "While trying to examine the mergeinfo in order to warn you about references to pre-dump revisions, we failed to parse the mergeinfo, so we can't warn you about pre-dump revisions, but you might want to know that we failed to parse the mergeinfo."
I don't want us to do just that, because I think it would lead some users to believe that we warn about invalid mergeinfo when in fact we would only do so *sometimes*. I think that would lead to confusion and dissatisfaction when they eventually discover that a full dump doesn't warn and a verify doesn't warn.
I think it would be better if we always try to parse the mergeinfo during every 'dump' and during every 'verify', and always warn (on dump) or error (on verify) if it's bad. (We would leave the first warning about references to pre-dump revisions just as it is: only for 'dump' and only when first rev > 1.)
This is an archived mail posted to the Subversion Dev mailing list.