convert fail silently if cvsps is not installed

marco gaddoni marco.gaddoni at gmail.com
Sun Oct 28 15:44:48 CDT 2007


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.

ciao, marco,

-- 
"Reality continues to ruin my life." - Calvin.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch3.patch
Type: text/x-patch
Size: 1978 bytes
Desc: not available
Url : http://selenic.com/pipermail/mercurial-devel/attachments/20071028/eb311b75/attachment.bin 


More information about the Mercurial-devel mailing list