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

Re: [PATCH] svn command - blame - multiple targets

From: Stefan Sperling <stsp_at_elego.de>
Date: Thu, 17 Feb 2011 14:17:55 +0100

On Thu, Feb 17, 2011 at 06:30:05PM +0530, Noorul Islam K M wrote:
>
> This patch is a followup of the following thread. All tests pass with
> this patch.
>
> http://svn.haxx.se/dev/archive-2011-01/0210.shtml
>
> Log
> [[[
>
> Make 'svn blame' continue processing targets after printing warning if one
> or more of the targets is a non-existent URL or wc-entry. Also return a
> non-zero error code and print an error message at the end in those
> situations.
>
> * subversion/svn/blame-cmd.c
> (svn_cl__blame): If one of the targets is a non-existent URL or
> wc-entry, don't bail out. Just warn and move on to the next
> target. Also return a non-zero error code and print an error message
> at the end in those situations.
>
> * subversion/tests/cmdline/blame_tests.py
> (blame_non_existent_wc_target, blame_non_existent_url_target):
> New tests.
> (test_list): Add reference to new tests.
>
> Patch by: Noorul Islam K M <noorul{_AT_}collab.net>
> ]]]
>

Thanks, r1071602. I've stripped the introductory part of the log
message. If it just repeats what the specific entries already say,
it's not necessary. An introduction is more important for larger changes
that contain a lot of entries, so that the gist of the change can be
captured quickly without reading through all the entries.
Received on 2011-02-17 14:18:35 CET

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.