convert fail silently if cvsps is not installed

marco gaddoni marco.gaddoni at gmail.com
Sun Oct 28 20:44:48 UTC 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: 1979 bytes
Desc: not available
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-devel/attachments/20071028/eb311b75/attachment-0002.bin>


More information about the Mercurial-devel mailing list