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

[Madan U S]

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

> Madan U S <madan at 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.
]]]
-------------- next part --------------
An HTML attachment was scrubbed...
URL: /pipermail/svnmerge/attachments/20060412/3483a7af/attachment.htm 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: smdiff.txt
Url: /pipermail/svnmerge/attachments/20060412/3483a7af/attachment.txt 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: smlog.txt
Url: /pipermail/svnmerge/attachments/20060412/3483a7af/attachment-0001.txt