[merge] handle a SSHException

[John Arbash Meinel]

Stefan (metze) Metzmacher wrote:
> John Arbash Meinel schrieb:
>>> I like the patch, but I think I've seen more of 'EOFError' than
>>> SSHException. It might be because I'm running paramiko 1.5.2 instead of
>>> 1.6.1.
>>>
>>> Actually, in my testing with 1.5.2 I get EOFError, and with paramiko
>>> 1.6.1-dev (revno 371) I get the exception:
>>>
>>>   File "build/bdist.linux-i686/egg/paramiko/sftp_client.py", line 86, in
>>> __init__
>>> NameError: global name 'SSHException' is not defined
> 
> I've send that to the paramiko mailing list, but it didn't get through...

Robey, can you comment/at least patch this for 1.6.2. Do you have a
bug-tracker for stuff like this?

> 
> I've attached it to this mail to
> 
>>> So I'd like to modify the patch to check for at least EOFError and
>>> SSHException. I don't think we want to catch NameError.
> 
> ok, a new patch is attached.
> 
> metze

+1 from me. I'd like to see this in 0.9, so I'll put it in my pending
queue. And if we don't get any negative feedback I'll merge it in a day
or so. (If we get positive, I'll merge it sooner).

John
=:->

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060727/5e279db1/attachment.pgp