[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Add XFail test case for Issue 3867

From: Paul Burba <ptburba_at_gmail.com>
Date: Wed, 18 May 2011 10:33:43 -0400

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
> (http://subversion.tigris.org/issues/show_bug.cgi?id=3867)
> Issue Summary:
> reintegrate merges create mergeinfo for non-existent paths
> Regards,
> 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 :-)

Received on 2011-05-18 16:34:12 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.