Madan U S <[EMAIL PROTECTED]> 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 _______________________________________________ Svnmerge mailing list [email protected] http://www.orcaware.com/mailman/listinfo/svnmerge
