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