← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/ssh.py

  • Committer: Martin Pool
  • Date: 2008-04-24 07:22:53 UTC
  • mto: This revision was merged to the branch mainline in revision 3415.
  • Revision ID: mbp@sourcefrog.net-20080424072253-opmjij7xfy38w27f
Remove every assert statement from bzrlib!

Depending on the context they are:

 * turned into an explicit if/raise of either AssertionError 
   or something more specific -- particularly where they protect
   programming interfaces, complex invariants, or data file integrity
 * removed, if they're redundant with a later check, not protecting
   a meaningful invariant
 * turned into a selftest method on tests

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/transport/ssh.py
402
402
 
403
403
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
404
404
                                  command=None):
405
 
        assert subsystem is not None or command is not None, (
406
 
            'Must specify a command or subsystem')
407
 
        if subsystem is not None:
408
 
            assert command is None, (
409
 
                'subsystem and command are mutually exclusive')
410
405
        args = ['ssh',
411
406
                '-oForwardX11=no', '-oForwardAgent=no',
412
407
                '-oClearAllForwardings=yes', '-oProtocol=2',
429
424
 
430
425
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
431
426
                                  command=None):
432
 
        assert subsystem is not None or command is not None, (
433
 
            'Must specify a command or subsystem')
434
 
        if subsystem is not None:
435
 
            assert command is None, (
436
 
                'subsystem and command are mutually exclusive')
437
427
        args = ['ssh', '-x']
438
428
        if port is not None:
439
429
            args.extend(['-p', str(port)])
453
443
 
454
444
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
455
445
                                  command=None):
456
 
        assert subsystem is not None or command is not None, (
457
 
            'Must specify a command or subsystem')
458
 
        if subsystem is not None:
459
 
            assert command is None, (
460
 
                'subsystem and command are mutually exclusive')
461
446
        args = ['plink', '-x', '-a', '-ssh', '-2', '-batch']
462
447
        if port is not None:
463
448
            args.extend(['-P', str(port)])