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

Re: [PATCH] svn diff --summarize

From: Martin Hauner <martin.hauner_at_gmx.net>
Date: 2006-02-05 15:26:42 CET

Julian Foad wrote:
> Ben Collins-Sussman wrote:
>> On 1/31/06, Julian Foad <julianfoad@btopenworld.com> wrote:
>>
>>>> $ svn diff --summarize
>>>> svn: Summarizing diff can only compare repository to repository
>>>
>>> That might be considered an annoying limitation (of the API).
>>
>> Ha! What's the difference between 'svn diff --summarize' (with no
>> explicit arguments) and 'svn status'? :-)
> [..]

> * "svn status" shows when an Added item is added with history. I'm not
> sure that needs to be part of "diff --summarize" output, but it would be
> nice. This sort of thing will become more important when we support
> true renames.

Generally the summarize api provides the info but from looking at the
code the copyfrom info is only filled by the editor in the add_directory
function and not in the add_file function. Looks like a missing copy
and paste bug. ;)

> * "svn status" shows each individual item deleted when a directory is
> deleted; "svn diff --summarize" just shows a single entry for the
> directory being deleted. That's a problem with the behaviour of "svn
> diff". (At least I believe there's a problem, if only that one might
> sometimes want to see all the deleted items.)

Interesting, does that mean that the delete_entry function in the editor
is not called for subitems of a deleted directory?

-- 
Martin
Subcommander, http://subcommander.tigris.org
a cross platform Win32/Unix/MacOSX subversion gui client & diff/merge tool.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Feb 5 16:19:28 2006

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.