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

Re: svn commit: r33938 - in trunk: . notes/tree-conflicts subversion/include subversion/include/private subversion/libsvn_auth_kwallet subversion/libsvn_client subversion/libsvn_subr subversion/libsvn_wc subversion/svn subversion/tests/cmdline subversion/tests/cmdline/svntest subversion/tests/libsvn_subr tools/buildbot/slaves/win32-xp-VS2005 tools/server-side www

From: Stefan Sperling <stsp_at_elego.de>
Date: Wed, 29 Oct 2008 16:07:37 +0000

On Wed, Oct 29, 2008 at 02:18:00AM +0100, Neels J. Hofmeyr wrote:
> sbutler_at_tigris.org wrote:
> > Modified: trunk/subversion/tests/cmdline/merge_tests.py
> > URL: http://svn.collab.net/viewvc/svn/trunk/subversion/tests/cmdline/merge_tests.py?pathrev=33938&r1=33937&r2=33938
> > ==============================================================================
> > --- trunk/subversion/tests/cmdline/merge_tests.py Tue Oct 28 08:56:45 2008 (r33937)
> > +++ trunk/subversion/tests/cmdline/merge_tests.py Tue Oct 28 09:22:25 2008 (r33938)
> > @@ -12256,7 +12256,7 @@ def del_differing_file(sbox):
> > "--- Merging r2 into '%s':\n" % dir_G2,
> > "C %s\n" % dir_G2,
> > "Summary of conflicts:\n",
> > - " Tree conflicts: 1\n",
> > + " Text conflicts: 1\n",
> > " Skipped paths: 1\n"])
> >
> > svn_merge(s_rev_pi, source, target, [
> > @@ -12264,7 +12264,7 @@ def del_differing_file(sbox):
> > "--- Merging r3 into '%s':\n" % dir_G2,
> > "C %s\n" % dir_G2,
> > "Summary of conflicts:\n",
> > - " Tree conflicts: 1\n",
> > + " Text conflicts: 1\n",
> > " Skipped paths: 1\n"])
> >
> >
> > @@ -12287,7 +12287,7 @@ def del_differing_file(sbox):
> > "--- Merging r2 into '%s':\n" % dir_G3,
> > "C %s\n" % dir_G3,
> > "Summary of conflicts:\n",
> > - " Tree conflicts: 1\n",
> > + " Text conflicts: 1\n",
> > " Skipped paths: 1\n"])
> >
> > svn_merge(s_rev_pi, source, target, [
> > @@ -12295,7 +12295,7 @@ def del_differing_file(sbox):
> > "--- Merging r3 into '%s':\n" % dir_G3,
> > "C %s\n" % dir_G3,
> > "Summary of conflicts:\n",
> > - " Tree conflicts: 1\n",
> > + " Text conflicts: 1\n",
> > " Skipped paths: 1\n"])
> >
> > os.chdir(saved_cwd)
>
>
> This change feels very wrong to me. I'd rather XFAIL these tests until merge
> notification is fixed. It says "Text conflicts", but these are tree
> conflicts!!

+1, please let's mark these XFAIL. It makes more sense for these
tests to reflect what the output should be.

As an aside: I took a brief look over the rest of the diff,
looks fine to me. Great work lads :)

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-10-29 17:07:57 CET

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