[PATCH] sshrepo: add support for paramiko transport engine

[Steve Borho]

On Fri, Apr 2, 2010 at 1:33 PM, Dirkjan Ochtman <dirkjan at ochtman.nl> wrote:
> On Thu, Apr 1, 2010 at 07:47, Steve Borho <steve at borho.org> wrote:
>> # HG changeset patch
>> # User Steve Borho <steve at borho.org>
>> # Date 1270100782 18000
>> # Node ID 692bd14317b35db4e856d222269d3c23320fced6
>> # Parent  fa7a14277cefdfda5b331fa9f9cddbfab8dd8eca
>> sshrepo: add support for paramiko transport engine
>>
>> This is incomplete, looking for comment and/or aid.  It could at least use some
>> cleanup and improved error handling.  I'm not at all sure readerr() won't get
>> stuck if there's actual stderr output.  But this at least passes simple sniff
>> tests.
>
> I definitely like the idea of supporting a paramiko transport, but it
> rather looks like it wants to be its own (ssh?)repository subclass.

I think I agree.

I've found that paramiko did not resolve the performance problems we
have with SSH on Windows (see the bug report earlier in the thread),
so I've moved on to other potential solutions.

Someone could probably package this as an extension that wraps sshrepo
with just a bit more work, but it's not something I'll be doing any
time soon.

Cheers,

--
Steve Borho