plugin: sftp transport

Martin Pool martinpool at gmail.com
Thu Oct 20 01:43:31 BST 2005


On 19/10/05, John Arbash Meinel <john at arbash-meinel.com> wrote:
> Martin Pool wrote:
> > If it's OK with you I'm going to make this a builtin transport.
> > People will still need to have paramiko to actually use it.
>
> I'm concerned that you put sftp support into the core without actually
> adding test cases for it. (This shouldn't be very difficult)

I know, I normally wouldn't have but Robert twisted my arm.

> Also, you have it require paramiko, which means that you can't even run
> bzr anymore.
>
> The exception really should be such that if it can't load the sftp code,
> that it at most gives a warning and continues.

I think that's the current behaviour, but there was an earlier
revision where it did fail.  The current one only loads the transport
when it's needed and so won't complain about paramiko until you try to
use an sftp url.

--
Martin




More information about the bazaar mailing list