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

RE: [PATCH][SVNMERGE] Make error message more informative.

From: Madan U S <madan_at_collab.net>
Date: 2006-04-12 10:10:50 CEST

On Wed, 12 Apr 2006 13:22:41 +0530, Giovanni Bajo <rasky@develer.com> wrote:

> Madan U S <madan@collab.net> wrote:
>
>> Fixed tabs in the patch... thanks djames for pointing out...
>>
>> [[[
>> Make (multiple heads present)error message more informative.
[snip]
>> ]]]
> There's a missing whitespace between "available" and the opening
> parenthesis.
> The list of heads should be two-space indentented (makes things easier
> for
> people parsing our output).
>
> Patch would be fine, but I don't specifically like the fact that you
> list the
> repo-relative version of the path, given that -S/--head does not accept
> repo-relative addresses. We discussed this some days ago. Are you
> planning to
> have this fixed? I would like to either fix that first, or have a
> modified
> version of your patch which displays full URLs (until that patch goes
> in).
>

Thanks for the feedback. Even I was thinking of printing out the absolute url instead of the repos-root relative path.
Pl. find the patch with the changes you had asked for....

[[[
Make (multiple heads present)error message more informative.

* contrib/client-side/svnmerge.py
  (get_default_head): Modified to make the error message
  more informative by listing the available heads.
]]]

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Received on Wed Apr 12 10:13:03 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.