~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/commands.py

  • Committer: Vincent Ladeuil
  • Date: 2011-01-12 00:58:05 UTC
  • mfrom: (5582.7.1 675652-bzr-columns-0)
  • mto: This revision was merged to the branch mainline in revision 5598.
  • Revision ID: v.ladeuil+lp@free.fr-20110112005805-tzclc2wogqef8zty
Accept 0 to mean no limit for BZR_COLUMNS

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
27
27
 
28
28
from bzrlib.lazy_import import lazy_import
29
29
lazy_import(globals(), """
 
30
import codecs
30
31
import errno
31
32
import threading
 
33
from warnings import warn
32
34
 
33
35
import bzrlib
34
36
from bzrlib import (
36
38
    cmdline,
37
39
    debug,
38
40
    errors,
39
 
    i18n,
40
41
    option,
41
42
    osutils,
42
43
    trace,
43
44
    ui,
 
45
    win32utils,
44
46
    )
45
47
""")
46
48
 
47
 
from bzrlib.hooks import Hooks
48
 
from bzrlib.i18n import gettext
 
49
from bzrlib.hooks import HookPoint, Hooks
49
50
# Compatibility - Option used to be in commands.
50
51
from bzrlib.option import Option
51
52
from bzrlib.plugin import disable_plugins, load_plugins
164
165
    return cmd[4:].replace('_','-')
165
166
 
166
167
 
 
168
@deprecated_function(deprecated_in((2, 2, 0)))
 
169
def _builtin_commands():
 
170
    """Return a dict of {name: cmd_class} for builtin commands.
 
171
 
 
172
    :deprecated: Use the builtin_command_registry registry instead
 
173
    """
 
174
    # return dict(name: cmd_class)
 
175
    return dict(builtin_command_registry.items())
 
176
 
 
177
 
167
178
def _register_builtin_commands():
168
179
    if builtin_command_registry.keys():
169
180
        # only load once
229
240
    try:
230
241
        return _get_cmd_object(cmd_name, plugins_override)
231
242
    except KeyError:
232
 
        raise errors.BzrCommandError(gettext('unknown command "%s"') % cmd_name)
 
243
        raise errors.BzrCommandError('unknown command "%s"' % cmd_name)
233
244
 
234
245
 
235
246
def _get_cmd_object(cmd_name, plugins_override=True, check_missing=True):
265
276
    # Allow plugins to extend commands
266
277
    for hook in Command.hooks['extend_command']:
267
278
        hook(cmd)
268
 
    if getattr(cmd, 'invoked_as', None) is None:
269
 
        cmd.invoked_as = cmd_name
270
279
    return cmd
271
280
 
272
281
 
349
358
    summary, then a complete description of the command.  A grammar
350
359
    description will be inserted.
351
360
 
352
 
    :cvar aliases: Other accepted names for this command.
353
 
 
354
 
    :cvar takes_args: List of argument forms, marked with whether they are
355
 
        optional, repeated, etc.  Examples::
356
 
 
357
 
            ['to_location', 'from_branch?', 'file*']
358
 
 
359
 
        * 'to_location' is required
360
 
        * 'from_branch' is optional
361
 
        * 'file' can be specified 0 or more times
362
 
 
363
 
    :cvar takes_options: List of options that may be given for this command.
364
 
        These can be either strings, referring to globally-defined options, or
365
 
        option objects.  Retrieve through options().
366
 
 
367
 
    :cvar hidden: If true, this command isn't advertised.  This is typically
 
361
    aliases
 
362
        Other accepted names for this command.
 
363
 
 
364
    takes_args
 
365
        List of argument forms, marked with whether they are optional,
 
366
        repeated, etc.
 
367
 
 
368
                Examples:
 
369
 
 
370
                ['to_location', 'from_branch?', 'file*']
 
371
 
 
372
                'to_location' is required
 
373
                'from_branch' is optional
 
374
                'file' can be specified 0 or more times
 
375
 
 
376
    takes_options
 
377
        List of options that may be given for this command.  These can
 
378
        be either strings, referring to globally-defined options,
 
379
        or option objects.  Retrieve through options().
 
380
 
 
381
    hidden
 
382
        If true, this command isn't advertised.  This is typically
368
383
        for commands intended for expert users.
369
384
 
370
 
    :cvar encoding_type: Command objects will get a 'outf' attribute, which has
371
 
        been setup to properly handle encoding of unicode strings.
372
 
        encoding_type determines what will happen when characters cannot be
373
 
        encoded:
374
 
 
375
 
        * strict - abort if we cannot decode
376
 
        * replace - put in a bogus character (typically '?')
377
 
        * exact - do not encode sys.stdout
378
 
 
379
 
        NOTE: by default on Windows, sys.stdout is opened as a text stream,
380
 
        therefore LF line-endings are converted to CRLF.  When a command uses
381
 
        encoding_type = 'exact', then sys.stdout is forced to be a binary
382
 
        stream, and line-endings will not mangled.
383
 
 
384
 
    :cvar invoked_as:
385
 
        A string indicating the real name under which this command was
386
 
        invoked, before expansion of aliases.
387
 
        (This may be None if the command was constructed and run in-process.)
 
385
    encoding_type
 
386
        Command objects will get a 'outf' attribute, which has been
 
387
        setup to properly handle encoding of unicode strings.
 
388
        encoding_type determines what will happen when characters cannot
 
389
        be encoded
 
390
            strict - abort if we cannot decode
 
391
            replace - put in a bogus character (typically '?')
 
392
            exact - do not encode sys.stdout
 
393
 
 
394
            NOTE: by default on Windows, sys.stdout is opened as a text
 
395
            stream, therefore LF line-endings are converted to CRLF.
 
396
            When a command uses encoding_type = 'exact', then
 
397
            sys.stdout is forced to be a binary stream, and line-endings
 
398
            will not mangled.
388
399
 
389
400
    :cvar hooks: An instance of CommandHooks.
390
 
 
391
 
    :cvar __doc__: The help shown by 'bzr help command' for this command.
 
401
    :ivar __doc__: The help shown by 'bzr help command' for this command.
392
402
        This is set by assigning explicitly to __doc__ so that -OO can
393
403
        be used::
394
404
 
395
 
            class Foo(Command):
396
 
                __doc__ = "My help goes here"
 
405
        class Foo(Command):
 
406
            __doc__ = "My help goes here"
397
407
    """
398
408
    aliases = []
399
409
    takes_args = []
400
410
    takes_options = []
401
411
    encoding_type = 'strict'
402
 
    invoked_as = None
403
 
    l10n = True
404
412
 
405
413
    hidden = False
406
414
 
432
440
        """
433
441
        self._operation.cleanup_now()
434
442
 
 
443
    @deprecated_method(deprecated_in((2, 1, 0)))
 
444
    def _maybe_expand_globs(self, file_list):
 
445
        """Glob expand file_list if the platform does not do that itself.
 
446
 
 
447
        Not used anymore, now that the bzr command-line parser globs on
 
448
        Windows.
 
449
 
 
450
        :return: A possibly empty list of unicode paths.
 
451
 
 
452
        Introduced in bzrlib 0.18.
 
453
        """
 
454
        return file_list
 
455
 
435
456
    def _usage(self):
436
457
        """Return single-line grammar for this command.
437
458
 
465
486
            usage help (e.g. Purpose, Usage, Options) with a
466
487
            message explaining how to obtain full help.
467
488
        """
468
 
        if self.l10n:
469
 
            i18n.install()  # Install i18n only for get_help_text for now.
470
489
        doc = self.help()
471
 
        if doc:
472
 
            # Note: If self.gettext() translates ':Usage:\n', the section will
473
 
            # be shown after "Description" section and we don't want to
474
 
            # translate the usage string.
475
 
            # Though, bzr export-pot don't exports :Usage: section and it must
476
 
            # not be translated.
477
 
            doc = self.gettext(doc)
478
 
        else:
479
 
            doc = gettext("No help for this command.")
 
490
        if not doc:
 
491
            doc = "No help for this command."
480
492
 
481
493
        # Extract the summary (purpose) and sections out from the text
482
494
        purpose,sections,order = self._get_help_parts(doc)
489
501
 
490
502
        # The header is the purpose and usage
491
503
        result = ""
492
 
        result += gettext(':Purpose: %s\n') % (purpose,)
 
504
        result += ':Purpose: %s\n' % purpose
493
505
        if usage.find('\n') >= 0:
494
 
            result += gettext(':Usage:\n%s\n') % (usage,)
 
506
            result += ':Usage:\n%s\n' % usage
495
507
        else:
496
 
            result += gettext(':Usage:   %s\n') % (usage,)
 
508
            result += ':Usage:   %s\n' % usage
497
509
        result += '\n'
498
510
 
499
511
        # Add the options
501
513
        # XXX: optparse implicitly rewraps the help, and not always perfectly,
502
514
        # so we get <https://bugs.launchpad.net/bzr/+bug/249908>.  -- mbp
503
515
        # 20090319
504
 
        parser = option.get_optparser(self.options())
505
 
        options = parser.format_option_help()
506
 
        # FIXME: According to the spec, ReST option lists actually don't
507
 
        # support options like --1.14 so that causes syntax errors (in Sphinx
508
 
        # at least).  As that pattern always appears in the commands that
509
 
        # break, we trap on that and then format that block of 'format' options
510
 
        # as a literal block. We use the most recent format still listed so we
511
 
        # don't have to do that too often -- vila 20110514
512
 
        if not plain and options.find('  --1.14  ') != -1:
 
516
        options = option.get_optparser(self.options()).format_option_help()
 
517
        # XXX: According to the spec, ReST option lists actually don't support 
 
518
        # options like --1.9 so that causes syntax errors (in Sphinx at least).
 
519
        # As that pattern always appears in the commands that break, we trap
 
520
        # on that and then format that block of 'format' options as a literal
 
521
        # block.
 
522
        if not plain and options.find('  --1.9  ') != -1:
513
523
            options = options.replace(' format:\n', ' format::\n\n', 1)
514
524
        if options.startswith('Options:'):
515
 
            result += gettext(':Options:%s') % (options[len('options:'):],)
 
525
            result += ':' + options
 
526
        elif options.startswith('options:'):
 
527
            # Python 2.4 version of optparse
 
528
            result += ':Options:' + options[len('options:'):]
516
529
        else:
517
530
            result += options
518
531
        result += '\n'
523
536
            if sections.has_key(None):
524
537
                text = sections.pop(None)
525
538
                text = '\n  '.join(text.splitlines())
526
 
                result += gettext(':Description:\n  %s\n\n') % (text,)
 
539
                result += ':%s:\n  %s\n\n' % ('Description',text)
527
540
 
528
541
            # Add the custom sections (e.g. Examples). Note that there's no need
529
542
            # to indent these as they must be indented already in the source.
530
543
            if sections:
531
544
                for label in order:
532
 
                    if label in sections:
533
 
                        result += ':%s:\n%s\n' % (label, sections[label])
 
545
                    if sections.has_key(label):
 
546
                        result += ':%s:\n%s\n' % (label,sections[label])
534
547
                result += '\n'
535
548
        else:
536
 
            result += (gettext("See bzr help %s for more details and examples.\n\n")
 
549
            result += ("See bzr help %s for more details and examples.\n\n"
537
550
                % self.name())
538
551
 
539
552
        # Add the aliases, source (plug-in) and see also links, if any
540
553
        if self.aliases:
541
 
            result += gettext(':Aliases:  ')
 
554
            result += ':Aliases:  '
542
555
            result += ', '.join(self.aliases) + '\n'
543
556
        plugin_name = self.plugin_name()
544
557
        if plugin_name is not None:
545
 
            result += gettext(':From:     plugin "%s"\n') % plugin_name
 
558
            result += ':From:     plugin "%s"\n' % plugin_name
546
559
        see_also = self.get_see_also(additional_see_also)
547
560
        if see_also:
548
561
            if not plain and see_also_as_links:
554
567
                        see_also_links.append(item)
555
568
                    else:
556
569
                        # Use a Sphinx link for this entry
557
 
                        link_text = gettext(":doc:`{0} <{1}-help>`").format(
558
 
                                                                    item, item)
 
570
                        link_text = ":doc:`%s <%s-help>`" % (item, item)
559
571
                        see_also_links.append(link_text)
560
572
                see_also = see_also_links
561
 
            result += gettext(':See also: %s') % ', '.join(see_also) + '\n'
 
573
            result += ':See also: '
 
574
            result += ', '.join(see_also) + '\n'
562
575
 
563
576
        # If this will be rendered as plain text, convert it
564
577
        if plain:
645
658
    def run_argv_aliases(self, argv, alias_argv=None):
646
659
        """Parse the command line and run with extra aliases in alias_argv."""
647
660
        args, opts = parse_args(self, argv, alias_argv)
648
 
        self._setup_outf()
649
661
 
650
662
        # Process the standard options
651
663
        if 'help' in opts:  # e.g. bzr add --help
652
 
            self.outf.write(self.get_help_text())
 
664
            sys.stdout.write(self.get_help_text())
653
665
            return 0
654
666
        if 'usage' in opts:  # e.g. bzr add --usage
655
 
            self.outf.write(self.get_help_text(verbose=False))
 
667
            sys.stdout.write(self.get_help_text(verbose=False))
656
668
            return 0
657
669
        trace.set_verbosity_level(option._verbosity_level)
658
670
        if 'verbose' in self.supported_std_options:
663
675
            opts['quiet'] = trace.is_quiet()
664
676
        elif opts.has_key('quiet'):
665
677
            del opts['quiet']
 
678
 
666
679
        # mix arguments and options into one dictionary
667
680
        cmdargs = _match_argform(self.name(), self.takes_args, args)
668
681
        cmdopts = {}
672
685
        all_cmd_args = cmdargs.copy()
673
686
        all_cmd_args.update(cmdopts)
674
687
 
 
688
        self._setup_outf()
 
689
 
675
690
        try:
676
691
            return self.run(**all_cmd_args)
677
692
        finally:
678
693
            # reset it, so that other commands run in the same process won't
679
 
            # inherit state. Before we reset it, log any activity, so that it
680
 
            # gets properly tracked.
681
 
            ui.ui_factory.log_transport_activity(
682
 
                display=('bytes' in debug.debug_flags))
 
694
            # inherit state
683
695
            trace.set_verbosity_level(0)
684
696
 
685
697
    def _setup_run(self):
700
712
                del self._operation
701
713
        self.run = run
702
714
 
 
715
    @deprecated_method(deprecated_in((2, 2, 0)))
 
716
    def run_direct(self, *args, **kwargs):
 
717
        """Deprecated thunk from bzrlib 2.1."""
 
718
        return self.run(*args, **kwargs)
 
719
 
703
720
    def run(self):
704
721
        """Actually run the command.
705
722
 
731
748
            return None
732
749
        return getdoc(self)
733
750
 
734
 
    def gettext(self, message):
735
 
        """Returns the gettext function used to translate this command's help.
736
 
 
737
 
        Commands provided by plugins should override this to use their
738
 
        own i18n system.
739
 
        """
740
 
        return i18n.gettext_per_paragraph(message)
741
 
 
742
751
    def name(self):
743
 
        """Return the canonical name for this command.
744
 
 
745
 
        The name under which it was actually invoked is available in invoked_as.
746
 
        """
747
752
        return _unsquish_command_name(self.__class__.__name__)
748
753
 
749
754
    def plugin_name(self):
767
772
        These are all empty initially, because by default nothing should get
768
773
        notified.
769
774
        """
770
 
        Hooks.__init__(self, "bzrlib.commands", "Command.hooks")
771
 
        self.add_hook('extend_command',
 
775
        Hooks.__init__(self)
 
776
        self.create_hook(HookPoint('extend_command',
772
777
            "Called after creating a command object to allow modifications "
773
778
            "such as adding or removing options, docs etc. Called with the "
774
 
            "new bzrlib.commands.Command object.", (1, 13))
775
 
        self.add_hook('get_command',
 
779
            "new bzrlib.commands.Command object.", (1, 13), None))
 
780
        self.create_hook(HookPoint('get_command',
776
781
            "Called when creating a single command. Called with "
777
782
            "(cmd_or_None, command_name). get_command should either return "
778
783
            "the cmd_or_None parameter, or a replacement Command object that "
779
784
            "should be used for the command. Note that the Command.hooks "
780
785
            "hooks are core infrastructure. Many users will prefer to use "
781
786
            "bzrlib.commands.register_command or plugin_cmds.register_lazy.",
782
 
            (1, 17))
783
 
        self.add_hook('get_missing_command',
 
787
            (1, 17), None))
 
788
        self.create_hook(HookPoint('get_missing_command',
784
789
            "Called when creating a single command if no command could be "
785
790
            "found. Called with (command_name). get_missing_command should "
786
791
            "either return None, or a Command object to be used for the "
787
 
            "command.", (1, 17))
788
 
        self.add_hook('list_commands',
 
792
            "command.", (1, 17), None))
 
793
        self.create_hook(HookPoint('list_commands',
789
794
            "Called when enumerating commands. Called with a set of "
790
795
            "cmd_name strings for all the commands found so far. This set "
791
796
            " is safe to mutate - e.g. to remove a command. "
792
797
            "list_commands should return the updated set of command names.",
793
 
            (1, 17))
 
798
            (1, 17), None))
794
799
 
795
800
Command.hooks = CommandHooks()
796
801
 
815
820
    try:
816
821
        options, args = parser.parse_args(args)
817
822
    except UnicodeEncodeError,e:
818
 
        raise errors.BzrCommandError(
819
 
            gettext('Only ASCII permitted in option names'))
 
823
        raise errors.BzrCommandError('Only ASCII permitted in option names')
820
824
 
821
825
    opts = dict([(k, v) for k, v in options.__dict__.iteritems() if
822
826
                 v is not option.OptionParser.DEFAULT_VALUE])
840
844
                argdict[argname + '_list'] = None
841
845
        elif ap[-1] == '+':
842
846
            if not args:
843
 
                raise errors.BzrCommandError(gettext(
844
 
                      "command {0!r} needs one or more {1}").format(
845
 
                      cmd, argname.upper()))
 
847
                raise errors.BzrCommandError("command %r needs one or more %s"
 
848
                                             % (cmd, argname.upper()))
846
849
            else:
847
850
                argdict[argname + '_list'] = args[:]
848
851
                args = []
849
852
        elif ap[-1] == '$': # all but one
850
853
            if len(args) < 2:
851
 
                raise errors.BzrCommandError(
852
 
                      gettext("command {0!r} needs one or more {1}").format(
853
 
                                             cmd, argname.upper()))
 
854
                raise errors.BzrCommandError("command %r needs one or more %s"
 
855
                                             % (cmd, argname.upper()))
854
856
            argdict[argname + '_list'] = args[:-1]
855
857
            args[:-1] = []
856
858
        else:
857
859
            # just a plain arg
858
860
            argname = ap
859
861
            if not args:
860
 
                raise errors.BzrCommandError(
861
 
                     gettext("command {0!r} requires argument {1}").format(
862
 
                               cmd, argname.upper()))
 
862
                raise errors.BzrCommandError("command %r requires argument %s"
 
863
                               % (cmd, argname.upper()))
863
864
            else:
864
865
                argdict[argname] = args.pop(0)
865
866
 
866
867
    if args:
867
 
        raise errors.BzrCommandError( gettext(
868
 
                              "extra argument to command {0}: {1}").format(
869
 
                                       cmd, args[0]) )
 
868
        raise errors.BzrCommandError("extra argument to command %s: %s"
 
869
                                     % (cmd, args[0]))
870
870
 
871
871
    return argdict
872
872
 
955
955
 
956
956
def apply_lsprofiled(filename, the_callable, *args, **kwargs):
957
957
    from bzrlib.lsprof import profile
958
 
    ret, stats = profile(exception_to_return_code, the_callable,
959
 
                         *args, **kwargs)
 
958
    ret, stats = profile(exception_to_return_code, the_callable, *args, **kwargs)
960
959
    stats.sort()
961
960
    if filename is None:
962
961
        stats.pprint()
963
962
    else:
964
963
        stats.save(filename)
965
 
        trace.note(gettext('Profile data written to "%s".'), filename)
 
964
        trace.note('Profile data written to "%s".', filename)
966
965
    return ret
967
966
 
968
967
 
 
968
@deprecated_function(deprecated_in((2, 2, 0)))
 
969
def shlex_split_unicode(unsplit):
 
970
    return cmdline.split(unsplit)
 
971
 
 
972
 
969
973
def get_alias(cmd, config=None):
970
974
    """Return an expanded alias, or None if no alias exists.
971
975
 
1027
1031
        Specify the number of processes that can be run concurrently (selftest).
1028
1032
    """
1029
1033
    trace.mutter("bazaar version: " + bzrlib.__version__)
1030
 
    argv = _specified_or_unicode_argv(argv)
 
1034
    argv = list(argv)
1031
1035
    trace.mutter("bzr arguments: %r", argv)
1032
1036
 
1033
 
    opt_lsprof = opt_profile = opt_no_plugins = opt_builtin = \
1034
 
            opt_no_l10n = opt_no_aliases = False
 
1037
    opt_lsprof = opt_profile = opt_no_plugins = opt_builtin =  \
 
1038
                opt_no_aliases = False
1035
1039
    opt_lsprof_file = opt_coverage_dir = None
1036
1040
 
1037
1041
    # --no-plugins is handled specially at a very early stage. We need
1040
1044
 
1041
1045
    argv_copy = []
1042
1046
    i = 0
1043
 
    override_config = []
1044
1047
    while i < len(argv):
1045
1048
        a = argv[i]
1046
1049
        if a == '--profile':
1055
1058
            opt_no_plugins = True
1056
1059
        elif a == '--no-aliases':
1057
1060
            opt_no_aliases = True
1058
 
        elif a == '--no-l10n':
1059
 
            opt_no_l10n = True
1060
1061
        elif a == '--builtin':
1061
1062
            opt_builtin = True
1062
1063
        elif a == '--concurrency':
1069
1070
            pass # already handled in startup script Bug #588277
1070
1071
        elif a.startswith('-D'):
1071
1072
            debug.debug_flags.add(a[2:])
1072
 
        elif a.startswith('-O'):
1073
 
            override_config.append(a[2:])
1074
1073
        else:
1075
1074
            argv_copy.append(a)
1076
1075
        i += 1
1077
1076
 
1078
 
    bzrlib.global_state.cmdline_overrides._from_cmdline(override_config)
1079
 
 
1080
1077
    debug.set_debug_flags_from_config()
1081
1078
 
1082
1079
    if not opt_no_plugins:
1102
1099
 
1103
1100
    cmd = argv.pop(0)
1104
1101
    cmd_obj = get_cmd_object(cmd, plugins_override=not opt_builtin)
1105
 
    if opt_no_l10n:
1106
 
        cmd.l10n = False
1107
1102
    run = cmd_obj.run_argv_aliases
1108
1103
    run_argv = [argv, alias_argv]
1109
1104
 
1134
1129
        if 'memory' in debug.debug_flags:
1135
1130
            trace.debug_memory('Process status after command:', short=False)
1136
1131
        option._verbosity_level = saved_verbosity_level
1137
 
        # Reset the overrides 
1138
 
        bzrlib.global_state.cmdline_overrides._reset()
1139
1132
 
1140
1133
 
1141
1134
def display_command(func):
1170
1163
        "bzr plugin commands")
1171
1164
    Command.hooks.install_named_hook("get_command", _get_external_command,
1172
1165
        "bzr external command lookup")
1173
 
    Command.hooks.install_named_hook("get_missing_command",
1174
 
                                     _try_plugin_provider,
1175
 
                                     "bzr plugin-provider-db check")
 
1166
    Command.hooks.install_named_hook("get_missing_command", _try_plugin_provider,
 
1167
        "bzr plugin-provider-db check")
1176
1168
 
1177
1169
 
1178
1170
 
1185
1177
        new_argv = []
1186
1178
        try:
1187
1179
            # ensure all arguments are unicode strings
1188
 
            for a in argv:
 
1180
            for a in argv[1:]:
1189
1181
                if isinstance(a, unicode):
1190
1182
                    new_argv.append(a)
1191
1183
                else:
1207
1199
 
1208
1200
    :return: exit code of bzr command.
1209
1201
    """
1210
 
    if argv is not None:
1211
 
        argv = argv[1:]
 
1202
    argv = _specified_or_unicode_argv(argv)
1212
1203
    _register_builtin_commands()
1213
1204
    ret = run_bzr_catch_errors(argv)
 
1205
    bzrlib.ui.ui_factory.log_transport_activity(
 
1206
        display=('bytes' in debug.debug_flags))
1214
1207
    trace.mutter("return code %d", ret)
1215
1208
    return ret
1216
1209
 
1269
1262
 
1270
1263
 
1271
1264
class Provider(object):
1272
 
    """Generic class to be overriden by plugins"""
 
1265
    '''Generic class to be overriden by plugins'''
1273
1266
 
1274
1267
    def plugin_for_command(self, cmd_name):
1275
 
        """Takes a command and returns the information for that plugin
 
1268
        '''Takes a command and returns the information for that plugin
1276
1269
 
1277
1270
        :return: A dictionary with all the available information
1278
 
            for the requested plugin
1279
 
        """
 
1271
        for the requested plugin
 
1272
        '''
1280
1273
        raise NotImplementedError
1281
1274
 
1282
1275
 
1283
1276
class ProvidersRegistry(registry.Registry):
1284
 
    """This registry exists to allow other providers to exist"""
 
1277
    '''This registry exists to allow other providers to exist'''
1285
1278
 
1286
1279
    def __iter__(self):
1287
1280
        for key, provider in self.iteritems():