convert fail silently if cvsps is not installed

Giorgos Keramidas keramida at ceid.upatras.gr
Sun Oct 28 14:15:55 CDT 2007


On 2007-10-28 18:56, marco gaddoni <marco.gaddoni at gmail.com> wrote:
> hello,
> yes you are right.
> see the updated patch.
> i have copied (adding a bug) that function from
> run-tests.py.
> shall i update that function too?

While there, can we also fix the typos?  "cant" is one of them:

> +
> +        if not find_program('cvsps'):
> +            raise util.Abort('cvsps executable not found. cant convert.')

Please either fix "cant" here (i.e. by using "Cannot convert", or drop
the multi-sentence format for the error, i.e.:

	raise util.Abort('cvsps executable not found')



More information about the Mercurial-devel mailing list