[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: Giovanni Bajo <rasky_at_develer.com>
Date: 2006-04-12 09:52:41 CEST

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.
>
> * contrib/client-side/svnmerge.py
> (get_default_head): Modified to make the error message
> more informative by listing the available heads (relative
> to the repos root).
> ]]]

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).

Giovanni Bajo

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Apr 12 09:52:58 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.