~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/commands.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-09-01 08:02:42 UTC
  • mfrom: (5390.3.3 faster-revert-593560)
  • Revision ID: pqm@pqm.ubuntu.com-20100901080242-esg62ody4frwmy66
(spiv) Avoid repeatedly calling self.target.all_file_ids() in
 InterTree.iter_changes. (Andrew Bennetts)

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
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
from __future__ import absolute_import
18
17
 
19
18
# TODO: Define arguments by objects, rather than just using names.
20
19
# Those objects can specify the expected type of the argument, which
28
27
 
29
28
from bzrlib.lazy_import import lazy_import
30
29
lazy_import(globals(), """
 
30
import codecs
31
31
import errno
32
32
import threading
 
33
from warnings import warn
33
34
 
34
35
import bzrlib
35
36
from bzrlib import (
36
 
    config,
37
37
    cleanup,
38
38
    cmdline,
39
39
    debug,
40
40
    errors,
41
 
    i18n,
42
41
    option,
43
42
    osutils,
44
43
    trace,
45
44
    ui,
 
45
    win32utils,
46
46
    )
47
47
""")
48
48
 
49
 
from bzrlib.hooks import Hooks
50
 
from bzrlib.i18n import gettext
 
49
from bzrlib.hooks import HookPoint, Hooks
51
50
# Compatibility - Option used to be in commands.
52
51
from bzrlib.option import Option
53
52
from bzrlib.plugin import disable_plugins, load_plugins
54
53
from bzrlib import registry
 
54
from bzrlib.symbol_versioning import (
 
55
    deprecated_function,
 
56
    deprecated_in,
 
57
    deprecated_method,
 
58
    )
55
59
 
56
60
 
57
61
class CommandInfo(object):
161
165
    return cmd[4:].replace('_','-')
162
166
 
163
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
 
164
178
def _register_builtin_commands():
165
179
    if builtin_command_registry.keys():
166
180
        # only load once
226
240
    try:
227
241
        return _get_cmd_object(cmd_name, plugins_override)
228
242
    except KeyError:
229
 
        raise errors.BzrCommandError(gettext('unknown command "%s"') % cmd_name)
 
243
        raise errors.BzrCommandError('unknown command "%s"' % cmd_name)
230
244
 
231
245
 
232
246
def _get_cmd_object(cmd_name, plugins_override=True, check_missing=True):
262
276
    # Allow plugins to extend commands
263
277
    for hook in Command.hooks['extend_command']:
264
278
        hook(cmd)
265
 
    if getattr(cmd, 'invoked_as', None) is None:
266
 
        cmd.invoked_as = cmd_name
267
279
    return cmd
268
280
 
269
281
 
346
358
    summary, then a complete description of the command.  A grammar
347
359
    description will be inserted.
348
360
 
349
 
    :cvar aliases: Other accepted names for this command.
350
 
 
351
 
    :cvar takes_args: List of argument forms, marked with whether they are
352
 
        optional, repeated, etc.  Examples::
353
 
 
354
 
            ['to_location', 'from_branch?', 'file*']
355
 
 
356
 
        * 'to_location' is required
357
 
        * 'from_branch' is optional
358
 
        * 'file' can be specified 0 or more times
359
 
 
360
 
    :cvar takes_options: List of options that may be given for this command.
361
 
        These can be either strings, referring to globally-defined options, or
362
 
        option objects.  Retrieve through options().
363
 
 
364
 
    :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
365
383
        for commands intended for expert users.
366
384
 
367
 
    :cvar encoding_type: Command objects will get a 'outf' attribute, which has
368
 
        been setup to properly handle encoding of unicode strings.
369
 
        encoding_type determines what will happen when characters cannot be
370
 
        encoded:
371
 
 
372
 
        * strict - abort if we cannot decode
373
 
        * replace - put in a bogus character (typically '?')
374
 
        * exact - do not encode sys.stdout
375
 
 
376
 
        NOTE: by default on Windows, sys.stdout is opened as a text stream,
377
 
        therefore LF line-endings are converted to CRLF.  When a command uses
378
 
        encoding_type = 'exact', then sys.stdout is forced to be a binary
379
 
        stream, and line-endings will not mangled.
380
 
 
381
 
    :cvar invoked_as:
382
 
        A string indicating the real name under which this command was
383
 
        invoked, before expansion of aliases.
384
 
        (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.
385
399
 
386
400
    :cvar hooks: An instance of CommandHooks.
387
 
 
388
 
    :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.
389
402
        This is set by assigning explicitly to __doc__ so that -OO can
390
403
        be used::
391
404
 
392
 
            class Foo(Command):
393
 
                __doc__ = "My help goes here"
 
405
        class Foo(Command):
 
406
            __doc__ = "My help goes here"
394
407
    """
395
408
    aliases = []
396
409
    takes_args = []
397
410
    takes_options = []
398
411
    encoding_type = 'strict'
399
 
    invoked_as = None
400
 
    l10n = True
401
412
 
402
413
    hidden = False
403
414
 
429
440
        """
430
441
        self._operation.cleanup_now()
431
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
 
432
456
    def _usage(self):
433
457
        """Return single-line grammar for this command.
434
458
 
462
486
            usage help (e.g. Purpose, Usage, Options) with a
463
487
            message explaining how to obtain full help.
464
488
        """
465
 
        if self.l10n:
466
 
            i18n.install()  # Install i18n only for get_help_text for now.
467
489
        doc = self.help()
468
 
        if doc:
469
 
            # Note: If self.gettext() translates ':Usage:\n', the section will
470
 
            # be shown after "Description" section and we don't want to
471
 
            # translate the usage string.
472
 
            # Though, bzr export-pot don't exports :Usage: section and it must
473
 
            # not be translated.
474
 
            doc = self.gettext(doc)
475
 
        else:
476
 
            doc = gettext("No help for this command.")
 
490
        if not doc:
 
491
            doc = "No help for this command."
477
492
 
478
493
        # Extract the summary (purpose) and sections out from the text
479
494
        purpose,sections,order = self._get_help_parts(doc)
486
501
 
487
502
        # The header is the purpose and usage
488
503
        result = ""
489
 
        result += gettext(':Purpose: %s\n') % (purpose,)
 
504
        result += ':Purpose: %s\n' % purpose
490
505
        if usage.find('\n') >= 0:
491
 
            result += gettext(':Usage:\n%s\n') % (usage,)
 
506
            result += ':Usage:\n%s\n' % usage
492
507
        else:
493
 
            result += gettext(':Usage:   %s\n') % (usage,)
 
508
            result += ':Usage:   %s\n' % usage
494
509
        result += '\n'
495
510
 
496
511
        # Add the options
498
513
        # XXX: optparse implicitly rewraps the help, and not always perfectly,
499
514
        # so we get <https://bugs.launchpad.net/bzr/+bug/249908>.  -- mbp
500
515
        # 20090319
501
 
        parser = option.get_optparser(self.options())
502
 
        options = parser.format_option_help()
503
 
        # FIXME: According to the spec, ReST option lists actually don't
504
 
        # support options like --1.14 so that causes syntax errors (in Sphinx
505
 
        # at least).  As that pattern always appears in the commands that
506
 
        # break, we trap on that and then format that block of 'format' options
507
 
        # as a literal block. We use the most recent format still listed so we
508
 
        # don't have to do that too often -- vila 20110514
509
 
        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:
510
523
            options = options.replace(' format:\n', ' format::\n\n', 1)
511
524
        if options.startswith('Options:'):
512
 
            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:'):]
513
529
        else:
514
530
            result += options
515
531
        result += '\n'
520
536
            if sections.has_key(None):
521
537
                text = sections.pop(None)
522
538
                text = '\n  '.join(text.splitlines())
523
 
                result += gettext(':Description:\n  %s\n\n') % (text,)
 
539
                result += ':%s:\n  %s\n\n' % ('Description',text)
524
540
 
525
541
            # Add the custom sections (e.g. Examples). Note that there's no need
526
542
            # to indent these as they must be indented already in the source.
527
543
            if sections:
528
544
                for label in order:
529
 
                    if label in sections:
530
 
                        result += ':%s:\n%s\n' % (label, sections[label])
 
545
                    if sections.has_key(label):
 
546
                        result += ':%s:\n%s\n' % (label,sections[label])
531
547
                result += '\n'
532
548
        else:
533
 
            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"
534
550
                % self.name())
535
551
 
536
552
        # Add the aliases, source (plug-in) and see also links, if any
537
553
        if self.aliases:
538
 
            result += gettext(':Aliases:  ')
 
554
            result += ':Aliases:  '
539
555
            result += ', '.join(self.aliases) + '\n'
540
556
        plugin_name = self.plugin_name()
541
557
        if plugin_name is not None:
542
 
            result += gettext(':From:     plugin "%s"\n') % plugin_name
 
558
            result += ':From:     plugin "%s"\n' % plugin_name
543
559
        see_also = self.get_see_also(additional_see_also)
544
560
        if see_also:
545
561
            if not plain and see_also_as_links:
551
567
                        see_also_links.append(item)
552
568
                    else:
553
569
                        # Use a Sphinx link for this entry
554
 
                        link_text = gettext(":doc:`{0} <{1}-help>`").format(
555
 
                                                                    item, item)
 
570
                        link_text = ":doc:`%s <%s-help>`" % (item, item)
556
571
                        see_also_links.append(link_text)
557
572
                see_also = see_also_links
558
 
            result += gettext(':See also: %s') % ', '.join(see_also) + '\n'
 
573
            result += ':See also: '
 
574
            result += ', '.join(see_also) + '\n'
559
575
 
560
576
        # If this will be rendered as plain text, convert it
561
577
        if plain:
642
658
    def run_argv_aliases(self, argv, alias_argv=None):
643
659
        """Parse the command line and run with extra aliases in alias_argv."""
644
660
        args, opts = parse_args(self, argv, alias_argv)
645
 
        self._setup_outf()
646
661
 
647
662
        # Process the standard options
648
663
        if 'help' in opts:  # e.g. bzr add --help
649
 
            self.outf.write(self.get_help_text())
 
664
            sys.stdout.write(self.get_help_text())
650
665
            return 0
651
666
        if 'usage' in opts:  # e.g. bzr add --usage
652
 
            self.outf.write(self.get_help_text(verbose=False))
 
667
            sys.stdout.write(self.get_help_text(verbose=False))
653
668
            return 0
654
669
        trace.set_verbosity_level(option._verbosity_level)
655
670
        if 'verbose' in self.supported_std_options:
660
675
            opts['quiet'] = trace.is_quiet()
661
676
        elif opts.has_key('quiet'):
662
677
            del opts['quiet']
 
678
 
663
679
        # mix arguments and options into one dictionary
664
680
        cmdargs = _match_argform(self.name(), self.takes_args, args)
665
681
        cmdopts = {}
669
685
        all_cmd_args = cmdargs.copy()
670
686
        all_cmd_args.update(cmdopts)
671
687
 
672
 
        try:
673
 
            return self.run(**all_cmd_args)
674
 
        finally:
675
 
            # reset it, so that other commands run in the same process won't
676
 
            # inherit state. Before we reset it, log any activity, so that it
677
 
            # gets properly tracked.
678
 
            ui.ui_factory.log_transport_activity(
679
 
                display=('bytes' in debug.debug_flags))
680
 
            trace.set_verbosity_level(0)
 
688
        self._setup_outf()
 
689
 
 
690
        return self.run(**all_cmd_args)
681
691
 
682
692
    def _setup_run(self):
683
693
        """Wrap the defined run method on self with a cleanup.
690
700
        """
691
701
        class_run = self.run
692
702
        def run(*args, **kwargs):
693
 
            for hook in Command.hooks['pre_command']:
694
 
                hook(self)
695
703
            self._operation = cleanup.OperationWithCleanups(class_run)
696
704
            try:
697
705
                return self._operation.run_simple(*args, **kwargs)
698
706
            finally:
699
707
                del self._operation
700
 
                for hook in Command.hooks['post_command']:
701
 
                    hook(self)
702
708
        self.run = run
703
709
 
 
710
    @deprecated_method(deprecated_in((2, 2, 0)))
 
711
    def run_direct(self, *args, **kwargs):
 
712
        """Deprecated thunk from bzrlib 2.1."""
 
713
        return self.run(*args, **kwargs)
 
714
 
704
715
    def run(self):
705
716
        """Actually run the command.
706
717
 
732
743
            return None
733
744
        return getdoc(self)
734
745
 
735
 
    def gettext(self, message):
736
 
        """Returns the gettext function used to translate this command's help.
737
 
 
738
 
        Commands provided by plugins should override this to use their
739
 
        own i18n system.
740
 
        """
741
 
        return i18n.gettext_per_paragraph(message)
742
 
 
743
746
    def name(self):
744
 
        """Return the canonical name for this command.
745
 
 
746
 
        The name under which it was actually invoked is available in invoked_as.
747
 
        """
748
747
        return _unsquish_command_name(self.__class__.__name__)
749
748
 
750
749
    def plugin_name(self):
768
767
        These are all empty initially, because by default nothing should get
769
768
        notified.
770
769
        """
771
 
        Hooks.__init__(self, "bzrlib.commands", "Command.hooks")
772
 
        self.add_hook('extend_command',
 
770
        Hooks.__init__(self)
 
771
        self.create_hook(HookPoint('extend_command',
773
772
            "Called after creating a command object to allow modifications "
774
773
            "such as adding or removing options, docs etc. Called with the "
775
 
            "new bzrlib.commands.Command object.", (1, 13))
776
 
        self.add_hook('get_command',
 
774
            "new bzrlib.commands.Command object.", (1, 13), None))
 
775
        self.create_hook(HookPoint('get_command',
777
776
            "Called when creating a single command. Called with "
778
777
            "(cmd_or_None, command_name). get_command should either return "
779
778
            "the cmd_or_None parameter, or a replacement Command object that "
780
779
            "should be used for the command. Note that the Command.hooks "
781
780
            "hooks are core infrastructure. Many users will prefer to use "
782
781
            "bzrlib.commands.register_command or plugin_cmds.register_lazy.",
783
 
            (1, 17))
784
 
        self.add_hook('get_missing_command',
 
782
            (1, 17), None))
 
783
        self.create_hook(HookPoint('get_missing_command',
785
784
            "Called when creating a single command if no command could be "
786
785
            "found. Called with (command_name). get_missing_command should "
787
786
            "either return None, or a Command object to be used for the "
788
 
            "command.", (1, 17))
789
 
        self.add_hook('list_commands',
 
787
            "command.", (1, 17), None))
 
788
        self.create_hook(HookPoint('list_commands',
790
789
            "Called when enumerating commands. Called with a set of "
791
790
            "cmd_name strings for all the commands found so far. This set "
792
791
            " is safe to mutate - e.g. to remove a command. "
793
792
            "list_commands should return the updated set of command names.",
794
 
            (1, 17))
795
 
        self.add_hook('pre_command',
796
 
            "Called prior to executing a command. Called with the command "
797
 
            "object.", (2, 6))
798
 
        self.add_hook('post_command',
799
 
            "Called after executing a command. Called with the command "
800
 
            "object.", (2, 6))
 
793
            (1, 17), None))
801
794
 
802
795
Command.hooks = CommandHooks()
803
796
 
817
810
    else:
818
811
        args = argv
819
812
 
820
 
    # for python 2.5 and later, optparse raises this exception if a non-ascii
821
 
    # option name is given.  See http://bugs.python.org/issue2931
822
 
    try:
823
 
        options, args = parser.parse_args(args)
824
 
    except UnicodeEncodeError,e:
825
 
        raise errors.BzrCommandError(
826
 
            gettext('Only ASCII permitted in option names'))
827
 
 
 
813
    options, args = parser.parse_args(args)
828
814
    opts = dict([(k, v) for k, v in options.__dict__.iteritems() if
829
815
                 v is not option.OptionParser.DEFAULT_VALUE])
830
816
    return args, opts
847
833
                argdict[argname + '_list'] = None
848
834
        elif ap[-1] == '+':
849
835
            if not args:
850
 
                raise errors.BzrCommandError(gettext(
851
 
                      "command {0!r} needs one or more {1}").format(
852
 
                      cmd, argname.upper()))
 
836
                raise errors.BzrCommandError("command %r needs one or more %s"
 
837
                                             % (cmd, argname.upper()))
853
838
            else:
854
839
                argdict[argname + '_list'] = args[:]
855
840
                args = []
856
841
        elif ap[-1] == '$': # all but one
857
842
            if len(args) < 2:
858
 
                raise errors.BzrCommandError(
859
 
                      gettext("command {0!r} needs one or more {1}").format(
860
 
                                             cmd, argname.upper()))
 
843
                raise errors.BzrCommandError("command %r needs one or more %s"
 
844
                                             % (cmd, argname.upper()))
861
845
            argdict[argname + '_list'] = args[:-1]
862
846
            args[:-1] = []
863
847
        else:
864
848
            # just a plain arg
865
849
            argname = ap
866
850
            if not args:
867
 
                raise errors.BzrCommandError(
868
 
                     gettext("command {0!r} requires argument {1}").format(
869
 
                               cmd, argname.upper()))
 
851
                raise errors.BzrCommandError("command %r requires argument %s"
 
852
                               % (cmd, argname.upper()))
870
853
            else:
871
854
                argdict[argname] = args.pop(0)
872
855
 
873
856
    if args:
874
 
        raise errors.BzrCommandError( gettext(
875
 
                              "extra argument to command {0}: {1}").format(
876
 
                                       cmd, args[0]) )
 
857
        raise errors.BzrCommandError("extra argument to command %s: %s"
 
858
                                     % (cmd, args[0]))
877
859
 
878
860
    return argdict
879
861
 
935
917
        exitcode = trace.report_exception(exc_info, sys.stderr)
936
918
        if os.environ.get('BZR_PDB'):
937
919
            print '**** entering debugger'
 
920
            tb = exc_info[2]
938
921
            import pdb
939
 
            pdb.post_mortem(exc_info[2])
 
922
            if sys.version_info[:2] < (2, 6):
 
923
                # XXX: we want to do
 
924
                #    pdb.post_mortem(tb)
 
925
                # but because pdb.post_mortem gives bad results for tracebacks
 
926
                # from inside generators, we do it manually.
 
927
                # (http://bugs.python.org/issue4150, fixed in Python 2.6)
 
928
 
 
929
                # Setup pdb on the traceback
 
930
                p = pdb.Pdb()
 
931
                p.reset()
 
932
                p.setup(tb.tb_frame, tb)
 
933
                # Point the debugger at the deepest frame of the stack
 
934
                p.curindex = len(p.stack) - 1
 
935
                p.curframe = p.stack[p.curindex][0]
 
936
                # Start the pdb prompt.
 
937
                p.print_stack_entry(p.stack[p.curindex])
 
938
                p.execRcLines()
 
939
                p.cmdloop()
 
940
            else:
 
941
                pdb.post_mortem(tb)
940
942
        return exitcode
941
943
 
942
944
 
943
945
def apply_lsprofiled(filename, the_callable, *args, **kwargs):
944
946
    from bzrlib.lsprof import profile
945
 
    ret, stats = profile(exception_to_return_code, the_callable,
946
 
                         *args, **kwargs)
 
947
    ret, stats = profile(exception_to_return_code, the_callable, *args, **kwargs)
947
948
    stats.sort()
948
949
    if filename is None:
949
950
        stats.pprint()
950
951
    else:
951
952
        stats.save(filename)
952
 
        trace.note(gettext('Profile data written to "%s".'), filename)
 
953
        trace.note('Profile data written to "%s".', filename)
953
954
    return ret
954
955
 
955
956
 
 
957
@deprecated_function(deprecated_in((2, 2, 0)))
 
958
def shlex_split_unicode(unsplit):
 
959
    return cmdline.split(unsplit)
 
960
 
 
961
 
956
962
def get_alias(cmd, config=None):
957
963
    """Return an expanded alias, or None if no alias exists.
958
964
 
1014
1020
        Specify the number of processes that can be run concurrently (selftest).
1015
1021
    """
1016
1022
    trace.mutter("bazaar version: " + bzrlib.__version__)
1017
 
    argv = _specified_or_unicode_argv(argv)
 
1023
    argv = list(argv)
1018
1024
    trace.mutter("bzr arguments: %r", argv)
1019
1025
 
1020
 
    opt_lsprof = opt_profile = opt_no_plugins = opt_builtin = \
1021
 
            opt_no_l10n = opt_no_aliases = False
 
1026
    opt_lsprof = opt_profile = opt_no_plugins = opt_builtin =  \
 
1027
                opt_no_aliases = False
1022
1028
    opt_lsprof_file = opt_coverage_dir = None
1023
1029
 
1024
1030
    # --no-plugins is handled specially at a very early stage. We need
1027
1033
 
1028
1034
    argv_copy = []
1029
1035
    i = 0
1030
 
    override_config = []
1031
1036
    while i < len(argv):
1032
1037
        a = argv[i]
1033
1038
        if a == '--profile':
1042
1047
            opt_no_plugins = True
1043
1048
        elif a == '--no-aliases':
1044
1049
            opt_no_aliases = True
1045
 
        elif a == '--no-l10n':
1046
 
            opt_no_l10n = True
1047
1050
        elif a == '--builtin':
1048
1051
            opt_builtin = True
1049
1052
        elif a == '--concurrency':
1056
1059
            pass # already handled in startup script Bug #588277
1057
1060
        elif a.startswith('-D'):
1058
1061
            debug.debug_flags.add(a[2:])
1059
 
        elif a.startswith('-O'):
1060
 
            override_config.append(a[2:])
1061
1062
        else:
1062
1063
            argv_copy.append(a)
1063
1064
        i += 1
1064
1065
 
1065
 
    if bzrlib.global_state is None:
1066
 
        # FIXME: Workaround for users that imported bzrlib but didn't call
1067
 
        # bzrlib.initialize -- vila 2012-01-19
1068
 
        cmdline_overrides = config.CommandLineStore()
1069
 
    else:
1070
 
        cmdline_overrides = bzrlib.global_state.cmdline_overrides
1071
 
    cmdline_overrides._from_cmdline(override_config)
1072
 
 
1073
1066
    debug.set_debug_flags_from_config()
1074
1067
 
1075
1068
    if not opt_no_plugins:
1095
1088
 
1096
1089
    cmd = argv.pop(0)
1097
1090
    cmd_obj = get_cmd_object(cmd, plugins_override=not opt_builtin)
1098
 
    if opt_no_l10n:
1099
 
        cmd.l10n = False
1100
1091
    run = cmd_obj.run_argv_aliases
1101
1092
    run_argv = [argv, alias_argv]
1102
1093
 
1127
1118
        if 'memory' in debug.debug_flags:
1128
1119
            trace.debug_memory('Process status after command:', short=False)
1129
1120
        option._verbosity_level = saved_verbosity_level
1130
 
        # Reset the overrides 
1131
 
        cmdline_overrides._reset()
1132
1121
 
1133
1122
 
1134
1123
def display_command(func):
1163
1152
        "bzr plugin commands")
1164
1153
    Command.hooks.install_named_hook("get_command", _get_external_command,
1165
1154
        "bzr external command lookup")
1166
 
    Command.hooks.install_named_hook("get_missing_command",
1167
 
                                     _try_plugin_provider,
1168
 
                                     "bzr plugin-provider-db check")
 
1155
    Command.hooks.install_named_hook("get_missing_command", _try_plugin_provider,
 
1156
        "bzr plugin-provider-db check")
1169
1157
 
1170
1158
 
1171
1159
 
1178
1166
        new_argv = []
1179
1167
        try:
1180
1168
            # ensure all arguments are unicode strings
1181
 
            for a in argv:
 
1169
            for a in argv[1:]:
1182
1170
                if isinstance(a, unicode):
1183
1171
                    new_argv.append(a)
1184
1172
                else:
1200
1188
 
1201
1189
    :return: exit code of bzr command.
1202
1190
    """
1203
 
    if argv is not None:
1204
 
        argv = argv[1:]
 
1191
    argv = _specified_or_unicode_argv(argv)
1205
1192
    _register_builtin_commands()
1206
1193
    ret = run_bzr_catch_errors(argv)
 
1194
    bzrlib.ui.ui_factory.log_transport_activity(
 
1195
        display=('bytes' in debug.debug_flags))
1207
1196
    trace.mutter("return code %d", ret)
1208
1197
    return ret
1209
1198
 
1262
1251
 
1263
1252
 
1264
1253
class Provider(object):
1265
 
    """Generic class to be overriden by plugins"""
 
1254
    '''Generic class to be overriden by plugins'''
1266
1255
 
1267
1256
    def plugin_for_command(self, cmd_name):
1268
 
        """Takes a command and returns the information for that plugin
 
1257
        '''Takes a command and returns the information for that plugin
1269
1258
 
1270
1259
        :return: A dictionary with all the available information
1271
 
            for the requested plugin
1272
 
        """
 
1260
        for the requested plugin
 
1261
        '''
1273
1262
        raise NotImplementedError
1274
1263
 
1275
1264
 
1276
1265
class ProvidersRegistry(registry.Registry):
1277
 
    """This registry exists to allow other providers to exist"""
 
1266
    '''This registry exists to allow other providers to exist'''
1278
1267
 
1279
1268
    def __iter__(self):
1280
1269
        for key, provider in self.iteritems():