convert fail silently if cvsps is not installed

Patrick Mézard pmezard at gmail.com
Tue Oct 30 21:19:09 UTC 2007


marco gaddoni a écrit :
> On 10/28/07, Patrick Mézard <pmezard at gmail.com> wrote:
>> marco gaddoni a écrit :
>> * Would util.find_exe() be enough to do this ?
>> * The cvs converter also requires cvs client.
>> * The abort message could be changed from "foobar executable not found. cant convert" to "required foobar executable not found".
>>
>> Except for these, this patch is definitely a good idea.
>>
> 
> Hello,
> 
> thank you for your suggestions.
> version 3 of the patch:
>  - use util.find_exe to check for executables
>  - test both cvsps and cvs for the cvs converter
>  - grammatic corrected
> 
> tested only in linux.

A different patch was pushed in crew as f0a3918abd42.

Thank you for reporting and working on this !

--
Patrick Mézard



More information about the Mercurial-devel mailing list