← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/ssh.py

  • Committer: Patch Queue Manager
  • Date: 2011-12-05 10:59:56 UTC
  • mfrom: (6331.5.3 bad_ssh_no_subprocess)
  • Revision ID: pqm@pqm.ubuntu.com-20111205105956-2sswur26splv04w8
(gz) Avoid using run_bzr_subprocess in a bt.test_sftp_transport test (Martin
 Packman)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/transport/ssh.py
352
352
class SubprocessVendor(SSHVendor):
353
353
    """Abstract base class for vendors that use pipes to a subprocess."""
354
354
 
 
355
    # In general stderr should be inherited from the parent process so prompts
 
356
    # are visible on the terminal. This can be overriden to another file for
 
357
    # tests, but beware of using PIPE which may hang due to not being read.
 
358
    _stderr_target = None
 
359
 
355
360
    def _connect(self, argv):
356
361
        # Attempt to make a socketpair to use as stdin/stdout for the SSH
357
362
        # subprocess.  We prefer sockets to pipes because they support
368
373
        else:
369
374
            stdin = stdout = subproc_sock
370
375
        proc = subprocess.Popen(argv, stdin=stdin, stdout=stdout,
 
376
                                stderr=self._stderr_target,
371
377
                                **os_specific_subprocess_params())
372
378
        if subproc_sock is not None:
373
379
            subproc_sock.close()