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