Re: [PATCH] Add XFail test case for Issue 3867
On Wednesday 18 May 2011 08:03 PM, Paul Burba wrote:
> On Wed, May 18, 2011 at 9:15 AM, Arwin Arni<arwin_at_collab.net> wrote:
>> Hi All,
>> I have attached a test case for Issue 3867
>> Issue Summary:
>> reintegrate merges create mergeinfo for non-existent paths
>> Arwin Arni
> Thanks Arwin. I committed your test in r1124283. I made a few minor
> tweaks to it:
> 1) Removed some of the pre-branching "trunk" edits, since one or two
> is sufficient to demonstrate the bug.
> 2) Removed the additional "trunk" edits after you made the branch,
> since these aren't relevant to the issue. Instead I made a single
> edit on the "branch" and then synced the "branch" with "trunk", so we
> are following the common use case of reintegrate.
> 3) Added some short informative log messages to the text edit commits
> rather than simply "m".
> 4) In the log message I added this leading line:
> "Add a test for issue #3867 'reintegrate merges create mergeinfo for
> non-existent paths'"
> I find it useful to have the summary of the issue spelled out in the
> log message. This helps the log message stand on it's own a bit
> better (for those of use who haven't memorized all of the issue
> numbers :-)
Thanks Paul, and sorry about that slipshod patch. I know it could've
Received on 2011-05-18 16:49:37 CEST
This is an archived mail posted to the Subversion Dev