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