← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/commands.py

  • Committer: Martin Pool
  • Date: 2005-06-17 07:28:53 UTC
  • Revision ID: mbp@sourcefrog.net-20050617072853-a815535b877b1ac7
- weed out all remaining calls to bailout() and remove the function

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/commands.py
20
20
 
21
21
import bzrlib
22
22
from bzrlib.trace import mutter, note, log_error
23
 
from bzrlib.errors import bailout, BzrError, BzrCheckError, BzrCommandError
 
23
from bzrlib.errors import BzrError, BzrCheckError, BzrCommandError
24
24
from bzrlib.osutils import quotefn
25
25
from bzrlib import Branch, Inventory, InventoryEntry, BZRDIR, \
26
26
     format_date
264
264
 
265
265
        for opt in self.takes_options:
266
266
            if not opt in OPTIONS:
267
 
                bailout("Unknown option '%s' returned by external command %s"
268
 
                    % (opt, path))
 
267
                raise BzrError("Unknown option '%s' returned by external command %s"
 
268
                               % (opt, path))
269
269
 
270
270
        # TODO: Is there any way to check takes_args is valid here?
271
271
        self.takes_args = pipe.readline().split()
272
272
 
273
273
        if pipe.close() is not None:
274
 
            bailout("Failed funning '%s --bzr-usage'" % path)
 
274
            raise BzrError("Failed funning '%s --bzr-usage'" % path)
275
275
 
276
276
        pipe = os.popen('%s --bzr-help' % path, 'r')
277
277
        self.__doc__ = pipe.read()
278
278
        if pipe.close() is not None:
279
 
            bailout("Failed funning '%s --bzr-help'" % path)
 
279
            raise BzrError("Failed funning '%s --bzr-help'" % path)
280
280
 
281
281
    def __call__(self, options, arguments):
282
282
        Command.__init__(self, options, arguments)
646
646
        b = Branch(filename)
647
647
        i = b.inventory.path2id(b.relpath(filename))
648
648
        if i == None:
649
 
            bailout("%r is not a versioned file" % filename)
 
649
            raise BzrError("%r is not a versioned file" % filename)
650
650
        else:
651
651
            print i
652
652
 
663
663
        inv = b.inventory
664
664
        fid = inv.path2id(b.relpath(filename))
665
665
        if fid == None:
666
 
            bailout("%r is not a versioned file" % filename)
 
666
            raise BzrError("%r is not a versioned file" % filename)
667
667
        for fip in inv.get_idpath(fid):
668
668
            print fip
669
669
 
1361
1361
                else:
1362
1362
                    optname = a[2:]
1363
1363
                if optname not in OPTIONS:
1364
 
                    bailout('unknown long option %r' % a)
 
1364
                    raise BzrError('unknown long option %r' % a)
1365
1365
            else:
1366
1366
                shortopt = a[1:]
1367
1367
                if shortopt in SHORT_OPTIONS:
1375
1375
                    if shortopt not in SHORT_OPTIONS:
1376
1376
                        # We didn't find the multi-character name, and we
1377
1377
                        # didn't find the single char name
1378
 
                        bailout('unknown short option %r' % a)
 
1378
                        raise BzrError('unknown short option %r' % a)
1379
1379
                    optname = SHORT_OPTIONS[shortopt]
1380
1380
 
1381
1381
                    if a[2:]:
1395
1395
            
1396
1396
            if optname in opts:
1397
1397
                # XXX: Do we ever want to support this, e.g. for -r?
1398
 
                bailout('repeated option %r' % a)
 
1398
                raise BzrError('repeated option %r' % a)
1399
1399
                
1400
1400
            optargfn = OPTIONS[optname]
1401
1401
            if optargfn:
1402
1402
                if optarg == None:
1403
1403
                    if not argv:
1404
 
                        bailout('option %r needs an argument' % a)
 
1404
                        raise BzrError('option %r needs an argument' % a)
1405
1405
                    else:
1406
1406
                        optarg = argv.pop(0)
1407
1407
                opts[optname] = optargfn(optarg)
1408
1408
            else:
1409
1409
                if optarg != None:
1410
 
                    bailout('option %r takes no argument' % optname)
 
1410
                    raise BzrError('option %r takes no argument' % optname)
1411
1411
                opts[optname] = True
1412
1412
        else:
1413
1413
            args.append(a)