smart transport test failure if paramiko is not insalled
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
Fix Released
|
Medium
|
Martin Pool |
Bug Description
A test fails if Paramiko is not available:
> =======
> ERROR: test_registration (bzrlib.
>
> vvvv[log from bzrlib.
>
> ^^^^[log from bzrlib.
> -------
> Traceback (most recent call last):
> File "/net/ecrins/
> t = get_transport(
> File "/net/ecrins/
> t, last_err = _try_transport_
> File "/net/ecrins/
> return factory(base), None
> File "/net/ecrins/
> klass = getattr(mod, classname)
> AttributeError: 'module' object has no attribute 'SmartSSHTransport'
Changed in bzr: | |
status: | Unconfirmed → Confirmed |
Changed in bzr: | |
status: | Fix Committed → Fix Released |
I can confirm this is a bug.
Apparently Andrew's branch fixes this so I'll leave it for now.
One part of the problem is that the ParamikoNotPresent exception is transport. smart. But at a deeper level, the smart
absorbed by bzrlib.
transport shouldn't depend on paramiko, unless it's unable to find any
other ssh implementation.