Code review comment for lp:~moritzm/duplicity/duplicity

Revision history for this message
edso (ed.so) wrote :

how about a similar solution like ssh? we periodically see users having issues with paramiko. it's good to have the alternative if the default fails for what reason ever.

never came around to do so, because i never needed it, like the patch does planned to establish lftp as a ftp backend, although as an alternative one. switching would have been done via url scheme eg.

lftp+ftp://
ncftp+ftp://

while at it i could streamline this switching for ssh as well eg.

paramiko+sftp://
pexpect+sftp://

retiring the --ssh-backend switch.

what do you guys think? ..ede

On 18.10.2014 22:42, Kenneth Loafman wrote:
> Let's see if we need to. NcFTP has not been updated in a long time and
> does not support IPv6. We can always restore the old version.
>
>
> On Sat, Oct 18, 2014 at 3:21 PM, edso <email address hidden> wrote:
>
>> On 18.10.2014 21:45, <email address hidden> wrote:
>>> The proposal to merge lp:~moritzm/duplicity/duplicity into lp:duplicity
>> has been updated.
>>>
>>> Status: Needs review => Merged
>>>
>>> For more details, see:
>>> https://code.launchpad.net/~moritzm/duplicity/duplicity/+merge/238157
>>>
>>
>> i on't like this!.. it removes a perfectly working backend, namely ncftp.
>> can we rather keep both and let the user select which one to choose? i
>> could live with making lftp the default.
>>
>> ..ede
>>

« Back to merge proposal