← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/ssh.py

  • Committer: John Arbash Meinel
  • Date: 2010-08-30 21:23:49 UTC
  • mto: This revision was merged to the branch mainline in revision 5398.
  • Revision ID: john@arbash-meinel.com-20100830212349-figt9yz2cic6hy68
Remove the 'false' invocation.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/transport/ssh.py
131
131
        # is given in BZR_SSH. See https://bugs.launchpad.net/bugs/414743
132
132
        elif 'plink' in version and progname == 'plink':
133
133
            # Checking if "plink" was the executed argument as Windows
134
 
            # sometimes reports 'ssh -V' incorrectly with 'plink' in its
 
134
            # sometimes reports 'ssh -V' incorrectly with 'plink' in it's
135
135
            # version.  See https://bugs.launchpad.net/bzr/+bug/107155
136
136
            trace.mutter("ssh implementation is Putty's plink.")
137
137
            vendor = PLinkSubprocessVendor()
336
336
            self._raise_connection_error(host, port=port, orig_error=e,
337
337
                                         msg='Unable to invoke remote bzr')
338
338
 
339
 
_ssh_connection_errors = (EOFError, OSError, IOError, socket.error)
340
339
if paramiko is not None:
341
340
    vendor = ParamikoVendor()
342
341
    register_ssh_vendor('paramiko', vendor)
343
342
    register_ssh_vendor('none', vendor)
344
343
    register_default_ssh_vendor(vendor)
345
 
    _ssh_connection_errors += (paramiko.SSHException,)
 
344
    _sftp_connection_errors = (EOFError, paramiko.SSHException)
346
345
    del vendor
 
346
else:
 
347
    _sftp_connection_errors = (EOFError,)
347
348
 
348
349
 
349
350
class SubprocessVendor(SSHVendor):
374
375
                                                  subsystem='sftp')
375
376
            sock = self._connect(argv)
376
377
            return SFTPClient(SocketAsChannelAdapter(sock))
377
 
        except _ssh_connection_errors, e:
 
378
        except _sftp_connection_errors, e:
 
379
            self._raise_connection_error(host, port=port, orig_error=e)
 
380
        except (OSError, IOError), e:
 
381
            # If the machine is fast enough, ssh can actually exit
 
382
            # before we try and send it the sftp request, which
 
383
            # raises a Broken Pipe
 
384
            if e.errno not in (errno.EPIPE,):
 
385
                raise
378
386
            self._raise_connection_error(host, port=port, orig_error=e)
379
387
 
380
388
    def connect_ssh(self, username, password, host, port, command):
382
390
            argv = self._get_vendor_specific_argv(username, host, port,
383
391
                                                  command=command)
384
392
            return self._connect(argv)
385
 
        except _ssh_connection_errors, e:
 
393
        except (EOFError), e:
 
394
            self._raise_connection_error(host, port=port, orig_error=e)
 
395
        except (OSError, IOError), e:
 
396
            # If the machine is fast enough, ssh can actually exit
 
397
            # before we try and send it the sftp request, which
 
398
            # raises a Broken Pipe
 
399
            if e.errno not in (errno.EPIPE,):
 
400
                raise
386
401
            self._raise_connection_error(host, port=port, orig_error=e)
387
402
 
388
403
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
630
645
_subproc_weakrefs = set()
631
646
 
632
647
def _close_ssh_proc(proc):
633
 
    """Carefully close stdin/stdout and reap the SSH process.
634
 
 
635
 
    If the pipes are already closed and/or the process has already been
636
 
    wait()ed on, that's ok, and no error is raised.  The goal is to do our best
637
 
    to clean up (whether or not a clean up was already tried).
638
 
    """
639
 
    dotted_names = ['stdin.close', 'stdout.close', 'wait']
640
 
    for dotted_name in dotted_names:
641
 
        attrs = dotted_name.split('.')
642
 
        try:
643
 
            obj = proc
644
 
            for attr in attrs:
645
 
                obj = getattr(obj, attr)
646
 
        except AttributeError:
647
 
            # It's ok for proc.stdin or proc.stdout to be None.
648
 
            continue
649
 
        try:
650
 
            obj()
 
648
    for func in [proc.stdin.close, proc.stdout.close, proc.wait]:
 
649
        try:
 
650
            func()
651
651
        except OSError:
652
 
            # It's ok for the pipe to already be closed, or the process to
653
 
            # already be finished.
654
 
            continue
 
652
            pass
655
653
 
656
654
 
657
655
class SSHConnection(object):