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

Re: [PATCH] print conflict statistics after update/merge

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sat, 18 Oct 2008 22:18:11 +0200 (Jerusalem Standard Time)

Stefan Sperling wrote on Sat, 18 Oct 2008 at 19:11 +0100:
> On Sat, Oct 18, 2008 at 06:07:19PM +0200, Daniel Shahaf wrote:
> > Stefan Sperling wrote on Sat, 18 Oct 2008 at 16:04 +0100:
> > > Conflict stats for externals seem to work fine:
> >
> > Yes. It prints just one summary at the end (after updating both the
> > external and non-external items).
>
> No it doesn't. This is the output when the external has no conflicts:
>
> $ svn update
> U gamma/delta
> C gamma
> U alpha
> C .
>
> Fetching external item into 'ext'
> U ext/gamma/delta
> U ext/alpha
> Updated external to revision 3.
> Summary of conflicts:
> Tree conflicts: 2

What do you mean by "No"? It printed the status for non-externals
(starting with "gamma/delta"), then the status of externals (starting
with "ext/gamma/delta"), and then (at the end ) the summary stats.

>
> Updated to revision 3.
>
> Attached is a patch which makes svn report conflict stats for
> externals separately. I think this is easier to parse:
>
...
>
> When both have conflicts, it looks like this:
>
> $ svn up
> U gamma/delta
> C gamma
> Conflict discovered in 'alpha'.
> Select: (p) postpone, (df) diff-full, (e) edit,
> (mc) mine-conflict, (tc) theirs-conflict,
> (s) show all options: p
> C alpha
>
> Fetching external item into 'ext'
> U ext/gamma/delta
> C ext/gamma
> Conflict discovered in 'ext/alpha'.
> Select: (p) postpone, (df) diff-full, (e) edit,
> (mc) mine-conflict, (tc) theirs-conflict,
> (s) show all options: p
> C ext/alpha
> Updated external to revision 3.
> Summary of conflicts in external item:
> Text conflicts: 1
> Tree conflicts: 1
>
> Updated to revision 3.
> Summary of conflicts:
> Text conflicts: 1
> Tree conflicts: 1
>
> OK?

Looks good.

Daniel

---------------------------------------------------------------------
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-18 22:18:26 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.