~bzr/ubuntu/lucid/bzr/beta-ppa

« back to all changes in this revision

Viewing changes to bzrlib/log.py

  • Committer: Martin Pool
  • Date: 2010-08-18 04:26:39 UTC
  • mfrom: (129.1.8 packaging-karmic)
  • Revision ID: mbp@sourcefrog.net-20100818042639-mjoxtngyjwiu05fo
* PPA rebuild for lucid.
* PPA rebuild for karmic.
* PPA rebuild onto jaunty.
* New upstream release.

Show diffs side-by-side

added added

removed removed

Lines of Context:
70
70
    diff,
71
71
    errors,
72
72
    foreign,
 
73
    osutils,
73
74
    repository as _mod_repository,
74
75
    revision as _mod_revision,
75
76
    revisionspec,
85
86
    format_date,
86
87
    format_date_with_offset_in_original_timezone,
87
88
    get_terminal_encoding,
88
 
    re_compile_checked,
89
89
    terminal_width,
90
90
    )
 
91
from bzrlib.symbol_versioning import (
 
92
    deprecated_function,
 
93
    deprecated_in,
 
94
    )
91
95
 
92
96
 
93
97
def find_touching_revisions(branch, file_id):
105
109
    last_path = None
106
110
    revno = 1
107
111
    for revision_id in branch.revision_history():
108
 
        this_inv = branch.repository.get_revision_inventory(revision_id)
 
112
        this_inv = branch.repository.get_inventory(revision_id)
109
113
        if file_id in this_inv:
110
114
            this_ie = this_inv[file_id]
111
115
            this_path = this_inv.id2path(file_id)
216
220
    'direction': 'reverse',
217
221
    'levels': 1,
218
222
    'generate_tags': True,
 
223
    'exclude_common_ancestry': False,
219
224
    '_match_using_deltas': True,
220
225
    }
221
226
 
222
227
 
223
228
def make_log_request_dict(direction='reverse', specific_fileids=None,
224
 
    start_revision=None, end_revision=None, limit=None,
225
 
    message_search=None, levels=1, generate_tags=True, delta_type=None,
226
 
    diff_type=None, _match_using_deltas=True):
 
229
                          start_revision=None, end_revision=None, limit=None,
 
230
                          message_search=None, levels=1, generate_tags=True,
 
231
                          delta_type=None,
 
232
                          diff_type=None, _match_using_deltas=True,
 
233
                          exclude_common_ancestry=False,
 
234
                          ):
227
235
    """Convenience function for making a logging request dictionary.
228
236
 
229
237
    Using this function may make code slightly safer by ensuring
267
275
      algorithm used for matching specific_fileids. This parameter
268
276
      may be removed in the future so bzrlib client code should NOT
269
277
      use it.
 
278
 
 
279
    :param exclude_common_ancestry: Whether -rX..Y should be interpreted as a
 
280
      range operator or as a graph difference.
270
281
    """
271
282
    return {
272
283
        'direction': direction,
279
290
        'generate_tags': generate_tags,
280
291
        'delta_type': delta_type,
281
292
        'diff_type': diff_type,
 
293
        'exclude_common_ancestry': exclude_common_ancestry,
282
294
        # Add 'private' attributes for features that may be deprecated
283
295
        '_match_using_deltas': _match_using_deltas,
284
296
    }
304
316
 
305
317
 
306
318
class Logger(object):
307
 
    """An object the generates, formats and displays a log."""
 
319
    """An object that generates, formats and displays a log."""
308
320
 
309
321
    def __init__(self, branch, rqst):
310
322
        """Create a Logger.
420
432
        else:
421
433
            specific_files = None
422
434
        s = StringIO()
 
435
        path_encoding = osutils.get_diff_header_encoding()
423
436
        diff.show_diff_trees(tree_1, tree_2, s, specific_files, old_label='',
424
 
            new_label='')
 
437
            new_label='', path_encoding=path_encoding)
425
438
        return s.getvalue()
426
439
 
427
440
    def _create_log_revision_iterator(self):
451
464
        generate_merge_revisions = rqst.get('levels') != 1
452
465
        delayed_graph_generation = not rqst.get('specific_fileids') and (
453
466
                rqst.get('limit') or self.start_rev_id or self.end_rev_id)
454
 
        view_revisions = _calc_view_revisions(self.branch, self.start_rev_id,
455
 
            self.end_rev_id, rqst.get('direction'), generate_merge_revisions,
456
 
            delayed_graph_generation=delayed_graph_generation)
 
467
        view_revisions = _calc_view_revisions(
 
468
            self.branch, self.start_rev_id, self.end_rev_id,
 
469
            rqst.get('direction'),
 
470
            generate_merge_revisions=generate_merge_revisions,
 
471
            delayed_graph_generation=delayed_graph_generation,
 
472
            exclude_common_ancestry=rqst.get('exclude_common_ancestry'))
457
473
 
458
474
        # Apply the other filters
459
475
        return make_log_rev_iterator(self.branch, view_revisions,
466
482
        # Note that we always generate the merge revisions because
467
483
        # filter_revisions_touching_file_id() requires them ...
468
484
        rqst = self.rqst
469
 
        view_revisions = _calc_view_revisions(self.branch, self.start_rev_id,
470
 
            self.end_rev_id, rqst.get('direction'), True)
 
485
        view_revisions = _calc_view_revisions(
 
486
            self.branch, self.start_rev_id, self.end_rev_id,
 
487
            rqst.get('direction'), generate_merge_revisions=True,
 
488
            exclude_common_ancestry=rqst.get('exclude_common_ancestry'))
471
489
        if not isinstance(view_revisions, list):
472
490
            view_revisions = list(view_revisions)
473
491
        view_revisions = _filter_revisions_touching_file_id(self.branch,
478
496
 
479
497
 
480
498
def _calc_view_revisions(branch, start_rev_id, end_rev_id, direction,
481
 
    generate_merge_revisions, delayed_graph_generation=False):
 
499
                         generate_merge_revisions,
 
500
                         delayed_graph_generation=False,
 
501
                         exclude_common_ancestry=False,
 
502
                         ):
482
503
    """Calculate the revisions to view.
483
504
 
484
505
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
485
506
             a list of the same tuples.
486
507
    """
 
508
    if (exclude_common_ancestry and start_rev_id == end_rev_id):
 
509
        raise errors.BzrCommandError(
 
510
            '--exclude-common-ancestry requires two different revisions')
 
511
    if direction not in ('reverse', 'forward'):
 
512
        raise ValueError('invalid direction %r' % direction)
487
513
    br_revno, br_rev_id = branch.last_revision_info()
488
514
    if br_revno == 0:
489
515
        return []
490
516
 
491
 
    # If a single revision is requested, check we can handle it
492
 
    generate_single_revision = (end_rev_id and start_rev_id == end_rev_id and
493
 
        (not generate_merge_revisions or not _has_merges(branch, end_rev_id)))
494
 
    if generate_single_revision:
495
 
        return _generate_one_revision(branch, end_rev_id, br_rev_id, br_revno)
496
 
 
497
 
    # If we only want to see linear revisions, we can iterate ...
498
 
    if not generate_merge_revisions:
499
 
        return _generate_flat_revisions(branch, start_rev_id, end_rev_id,
500
 
            direction)
 
517
    if (end_rev_id and start_rev_id == end_rev_id
 
518
        and (not generate_merge_revisions
 
519
             or not _has_merges(branch, end_rev_id))):
 
520
        # If a single revision is requested, check we can handle it
 
521
        iter_revs = _generate_one_revision(branch, end_rev_id, br_rev_id,
 
522
                                           br_revno)
 
523
    elif not generate_merge_revisions:
 
524
        # If we only want to see linear revisions, we can iterate ...
 
525
        iter_revs = _generate_flat_revisions(branch, start_rev_id, end_rev_id,
 
526
                                             direction, exclude_common_ancestry)
 
527
        if direction == 'forward':
 
528
            iter_revs = reversed(iter_revs)
501
529
    else:
502
 
        return _generate_all_revisions(branch, start_rev_id, end_rev_id,
503
 
            direction, delayed_graph_generation)
 
530
        iter_revs = _generate_all_revisions(branch, start_rev_id, end_rev_id,
 
531
                                            direction, delayed_graph_generation,
 
532
                                            exclude_common_ancestry)
 
533
        if direction == 'forward':
 
534
            iter_revs = _rebase_merge_depth(reverse_by_depth(list(iter_revs)))
 
535
    return iter_revs
504
536
 
505
537
 
506
538
def _generate_one_revision(branch, rev_id, br_rev_id, br_revno):
513
545
        return [(rev_id, revno_str, 0)]
514
546
 
515
547
 
516
 
def _generate_flat_revisions(branch, start_rev_id, end_rev_id, direction):
517
 
    result = _linear_view_revisions(branch, start_rev_id, end_rev_id)
 
548
def _generate_flat_revisions(branch, start_rev_id, end_rev_id, direction,
 
549
                             exclude_common_ancestry=False):
 
550
    result = _linear_view_revisions(
 
551
        branch, start_rev_id, end_rev_id,
 
552
        exclude_common_ancestry=exclude_common_ancestry)
518
553
    # If a start limit was given and it's not obviously an
519
554
    # ancestor of the end limit, check it before outputting anything
520
555
    if direction == 'forward' or (start_rev_id
524
559
        except _StartNotLinearAncestor:
525
560
            raise errors.BzrCommandError('Start revision not found in'
526
561
                ' left-hand history of end revision.')
527
 
    if direction == 'forward':
528
 
        result = reversed(result)
529
562
    return result
530
563
 
531
564
 
532
565
def _generate_all_revisions(branch, start_rev_id, end_rev_id, direction,
533
 
    delayed_graph_generation):
 
566
                            delayed_graph_generation,
 
567
                            exclude_common_ancestry=False):
534
568
    # On large trees, generating the merge graph can take 30-60 seconds
535
569
    # so we delay doing it until a merge is detected, incrementally
536
570
    # returning initial (non-merge) revisions while we can.
 
571
 
 
572
    # The above is only true for old formats (<= 0.92), for newer formats, a
 
573
    # couple of seconds only should be needed to load the whole graph and the
 
574
    # other graph operations needed are even faster than that -- vila 100201
537
575
    initial_revisions = []
538
576
    if delayed_graph_generation:
539
577
        try:
540
 
            for rev_id, revno, depth in \
541
 
                _linear_view_revisions(branch, start_rev_id, end_rev_id):
 
578
            for rev_id, revno, depth in  _linear_view_revisions(
 
579
                branch, start_rev_id, end_rev_id, exclude_common_ancestry):
542
580
                if _has_merges(branch, rev_id):
 
581
                    # The end_rev_id can be nested down somewhere. We need an
 
582
                    # explicit ancestry check. There is an ambiguity here as we
 
583
                    # may not raise _StartNotLinearAncestor for a revision that
 
584
                    # is an ancestor but not a *linear* one. But since we have
 
585
                    # loaded the graph to do the check (or calculate a dotted
 
586
                    # revno), we may as well accept to show the log...  We need
 
587
                    # the check only if start_rev_id is not None as all
 
588
                    # revisions have _mod_revision.NULL_REVISION as an ancestor
 
589
                    # -- vila 20100319
 
590
                    graph = branch.repository.get_graph()
 
591
                    if (start_rev_id is not None
 
592
                        and not graph.is_ancestor(start_rev_id, end_rev_id)):
 
593
                        raise _StartNotLinearAncestor()
 
594
                    # Since we collected the revisions so far, we need to
 
595
                    # adjust end_rev_id.
543
596
                    end_rev_id = rev_id
544
597
                    break
545
598
                else:
546
599
                    initial_revisions.append((rev_id, revno, depth))
547
600
            else:
548
601
                # No merged revisions found
549
 
                if direction == 'reverse':
550
 
                    return initial_revisions
551
 
                elif direction == 'forward':
552
 
                    return reversed(initial_revisions)
553
 
                else:
554
 
                    raise ValueError('invalid direction %r' % direction)
 
602
                return initial_revisions
555
603
        except _StartNotLinearAncestor:
556
604
            # A merge was never detected so the lower revision limit can't
557
605
            # be nested down somewhere
558
606
            raise errors.BzrCommandError('Start revision not found in'
559
607
                ' history of end revision.')
560
608
 
 
609
    # We exit the loop above because we encounter a revision with merges, from
 
610
    # this revision, we need to switch to _graph_view_revisions.
 
611
 
561
612
    # A log including nested merges is required. If the direction is reverse,
562
613
    # we rebase the initial merge depths so that the development line is
563
614
    # shown naturally, i.e. just like it is for linear logging. We can easily
565
616
    # indented at the end seems slightly nicer in that case.
566
617
    view_revisions = chain(iter(initial_revisions),
567
618
        _graph_view_revisions(branch, start_rev_id, end_rev_id,
568
 
        rebase_initial_depths=direction == 'reverse'))
569
 
    if direction == 'reverse':
570
 
        return view_revisions
571
 
    elif direction == 'forward':
572
 
        # Forward means oldest first, adjusting for depth.
573
 
        view_revisions = reverse_by_depth(list(view_revisions))
574
 
        return _rebase_merge_depth(view_revisions)
575
 
    else:
576
 
        raise ValueError('invalid direction %r' % direction)
 
619
                              rebase_initial_depths=(direction == 'reverse'),
 
620
                              exclude_common_ancestry=exclude_common_ancestry))
 
621
    return view_revisions
577
622
 
578
623
 
579
624
def _has_merges(branch, rev_id):
597
642
        else:
598
643
            # not obvious
599
644
            return False
 
645
    # if either start or end is not specified then we use either the first or
 
646
    # the last revision and *they* are obvious ancestors.
600
647
    return True
601
648
 
602
649
 
603
 
def _linear_view_revisions(branch, start_rev_id, end_rev_id):
 
650
def _linear_view_revisions(branch, start_rev_id, end_rev_id,
 
651
                           exclude_common_ancestry=False):
604
652
    """Calculate a sequence of revisions to view, newest to oldest.
605
653
 
606
654
    :param start_rev_id: the lower revision-id
607
655
    :param end_rev_id: the upper revision-id
 
656
    :param exclude_common_ancestry: Whether the start_rev_id should be part of
 
657
        the iterated revisions.
608
658
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples.
609
659
    :raises _StartNotLinearAncestor: if a start_rev_id is specified but
610
 
      is not found walking the left-hand history
 
660
        is not found walking the left-hand history
611
661
    """
612
662
    br_revno, br_rev_id = branch.last_revision_info()
613
663
    repo = branch.repository
624
674
            revno = branch.revision_id_to_dotted_revno(revision_id)
625
675
            revno_str = '.'.join(str(n) for n in revno)
626
676
            if not found_start and revision_id == start_rev_id:
627
 
                yield revision_id, revno_str, 0
 
677
                if not exclude_common_ancestry:
 
678
                    yield revision_id, revno_str, 0
628
679
                found_start = True
629
680
                break
630
681
            else:
635
686
 
636
687
 
637
688
def _graph_view_revisions(branch, start_rev_id, end_rev_id,
638
 
    rebase_initial_depths=True):
 
689
                          rebase_initial_depths=True,
 
690
                          exclude_common_ancestry=False):
639
691
    """Calculate revisions to view including merges, newest to oldest.
640
692
 
641
693
    :param branch: the branch
645
697
      revision is found?
646
698
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples.
647
699
    """
 
700
    if exclude_common_ancestry:
 
701
        stop_rule = 'with-merges-without-common-ancestry'
 
702
    else:
 
703
        stop_rule = 'with-merges'
648
704
    view_revisions = branch.iter_merge_sorted_revisions(
649
705
        start_revision_id=end_rev_id, stop_revision_id=start_rev_id,
650
 
        stop_rule="with-merges")
 
706
        stop_rule=stop_rule)
651
707
    if not rebase_initial_depths:
652
708
        for (rev_id, merge_depth, revno, end_of_merge
653
709
             ) in view_revisions:
664
720
                depth_adjustment = merge_depth
665
721
            if depth_adjustment:
666
722
                if merge_depth < depth_adjustment:
 
723
                    # From now on we reduce the depth adjustement, this can be
 
724
                    # surprising for users. The alternative requires two passes
 
725
                    # which breaks the fast display of the first revision
 
726
                    # though.
667
727
                    depth_adjustment = merge_depth
668
728
                merge_depth -= depth_adjustment
669
729
            yield rev_id, '.'.join(map(str, revno)), merge_depth
670
730
 
671
731
 
 
732
@deprecated_function(deprecated_in((2, 2, 0)))
672
733
def calculate_view_revisions(branch, start_revision, end_revision, direction,
673
734
        specific_fileid, generate_merge_revisions):
674
735
    """Calculate the revisions to view.
676
737
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
677
738
             a list of the same tuples.
678
739
    """
679
 
    # This method is no longer called by the main code path.
680
 
    # It is retained for API compatibility and may be deprecated
681
 
    # soon. IGC 20090116
682
740
    start_rev_id, end_rev_id = _get_revision_limits(branch, start_revision,
683
741
        end_revision)
684
742
    view_revisions = list(_calc_view_revisions(branch, start_rev_id, end_rev_id,
752
810
    """
753
811
    if search is None:
754
812
        return log_rev_iterator
755
 
    searchRE = re_compile_checked(search, re.IGNORECASE,
756
 
            'log message filter')
 
813
    searchRE = re.compile(search, re.IGNORECASE)
757
814
    return _filter_message_re(searchRE, log_rev_iterator)
758
815
 
759
816
 
1034
1091
    return mainline_revs, rev_nos, start_rev_id, end_rev_id
1035
1092
 
1036
1093
 
 
1094
@deprecated_function(deprecated_in((2, 2, 0)))
1037
1095
def _filter_revision_range(view_revisions, start_rev_id, end_rev_id):
1038
1096
    """Filter view_revisions based on revision ranges.
1039
1097
 
1048
1106
 
1049
1107
    :return: The filtered view_revisions.
1050
1108
    """
1051
 
    # This method is no longer called by the main code path.
1052
 
    # It may be removed soon. IGC 20090127
1053
1109
    if start_rev_id or end_rev_id:
1054
1110
        revision_ids = [r for r, n, d in view_revisions]
1055
1111
        if start_rev_id:
1161
1217
    return result
1162
1218
 
1163
1219
 
 
1220
@deprecated_function(deprecated_in((2, 2, 0)))
1164
1221
def get_view_revisions(mainline_revs, rev_nos, branch, direction,
1165
1222
                       include_merges=True):
1166
1223
    """Produce an iterator of revisions to show
1167
1224
    :return: an iterator of (revision_id, revno, merge_depth)
1168
1225
    (if there is no revno for a revision, None is supplied)
1169
1226
    """
1170
 
    # This method is no longer called by the main code path.
1171
 
    # It is retained for API compatibility and may be deprecated
1172
 
    # soon. IGC 20090127
1173
1227
    if not include_merges:
1174
1228
        revision_ids = mainline_revs[1:]
1175
1229
        if direction == 'reverse':
1293
1347
    preferred_levels = 0
1294
1348
 
1295
1349
    def __init__(self, to_file, show_ids=False, show_timezone='original',
1296
 
            delta_format=None, levels=None, show_advice=False,
1297
 
            to_exact_file=None):
 
1350
                 delta_format=None, levels=None, show_advice=False,
 
1351
                 to_exact_file=None, author_list_handler=None):
1298
1352
        """Create a LogFormatter.
1299
1353
 
1300
1354
        :param to_file: the file to output to
1308
1362
          let the log formatter decide.
1309
1363
        :param show_advice: whether to show advice at the end of the
1310
1364
          log or not
 
1365
        :param author_list_handler: callable generating a list of
 
1366
          authors to display for a given revision
1311
1367
        """
1312
1368
        self.to_file = to_file
1313
1369
        # 'exact' stream used to show diff, it should print content 'as is'
1328
1384
        self.levels = levels
1329
1385
        self._show_advice = show_advice
1330
1386
        self._merge_count = 0
 
1387
        self._author_list_handler = author_list_handler
1331
1388
 
1332
1389
    def get_levels(self):
1333
1390
        """Get the number of levels to display or 0 for all."""
1365
1422
        return address
1366
1423
 
1367
1424
    def short_author(self, rev):
1368
 
        name, address = config.parse_username(rev.get_apparent_authors()[0])
1369
 
        if name:
1370
 
            return name
1371
 
        return address
 
1425
        return self.authors(rev, 'first', short=True, sep=', ')
 
1426
 
 
1427
    def authors(self, rev, who, short=False, sep=None):
 
1428
        """Generate list of authors, taking --authors option into account.
 
1429
 
 
1430
        The caller has to specify the name of a author list handler,
 
1431
        as provided by the author list registry, using the ``who``
 
1432
        argument.  That name only sets a default, though: when the
 
1433
        user selected a different author list generation using the
 
1434
        ``--authors`` command line switch, as represented by the
 
1435
        ``author_list_handler`` constructor argument, that value takes
 
1436
        precedence.
 
1437
 
 
1438
        :param rev: The revision for which to generate the list of authors.
 
1439
        :param who: Name of the default handler.
 
1440
        :param short: Whether to shorten names to either name or address.
 
1441
        :param sep: What separator to use for automatic concatenation.
 
1442
        """
 
1443
        if self._author_list_handler is not None:
 
1444
            # The user did specify --authors, which overrides the default
 
1445
            author_list_handler = self._author_list_handler
 
1446
        else:
 
1447
            # The user didn't specify --authors, so we use the caller's default
 
1448
            author_list_handler = author_list_registry.get(who)
 
1449
        names = author_list_handler(rev)
 
1450
        if short:
 
1451
            for i in range(len(names)):
 
1452
                name, address = config.parse_username(names[i])
 
1453
                if name:
 
1454
                    names[i] = name
 
1455
                else:
 
1456
                    names[i] = address
 
1457
        if sep is not None:
 
1458
            names = sep.join(names)
 
1459
        return names
1372
1460
 
1373
1461
    def merge_marker(self, revision):
1374
1462
        """Get the merge marker to include in the output or '' if none."""
1405
1493
        """
1406
1494
        # Revision comes directly from a foreign repository
1407
1495
        if isinstance(rev, foreign.ForeignRevision):
1408
 
            return self._format_properties(rev.mapping.vcs.show_foreign_revid(rev.foreign_revid))
 
1496
            return self._format_properties(
 
1497
                rev.mapping.vcs.show_foreign_revid(rev.foreign_revid))
1409
1498
 
1410
1499
        # Imported foreign revision revision ids always contain :
1411
1500
        if not ":" in rev.revision_id:
1475
1564
        lines.extend(self.custom_properties(revision.rev))
1476
1565
 
1477
1566
        committer = revision.rev.committer
1478
 
        authors = revision.rev.get_apparent_authors()
 
1567
        authors = self.authors(revision.rev, 'all')
1479
1568
        if authors != [committer]:
1480
1569
            lines.append('author: %s' % (", ".join(authors),))
1481
1570
        lines.append('committer: %s' % (committer,))
1498
1587
        to_file = self.to_file
1499
1588
        to_file.write("%s%s\n" % (indent, ('\n' + indent).join(lines)))
1500
1589
        if revision.delta is not None:
1501
 
            # We don't respect delta_format for compatibility
1502
 
            revision.delta.show(to_file, self.show_ids, indent=indent,
1503
 
                                short_status=False)
 
1590
            # Use the standard status output to display changes
 
1591
            from bzrlib.delta import report_delta
 
1592
            report_delta(to_file, revision.delta, short_status=False, 
 
1593
                         show_ids=self.show_ids, indent=indent)
1504
1594
        if revision.diff is not None:
1505
1595
            to_file.write(indent + 'diff:\n')
1506
1596
            to_file.flush()
1569
1659
                to_file.write(indent + offset + '%s\n' % (l,))
1570
1660
 
1571
1661
        if revision.delta is not None:
1572
 
            revision.delta.show(to_file, self.show_ids, indent=indent + offset,
1573
 
                                short_status=self.delta_format==1)
 
1662
            # Use the standard status output to display changes
 
1663
            from bzrlib.delta import report_delta
 
1664
            report_delta(to_file, revision.delta, 
 
1665
                         short_status=self.delta_format==1, 
 
1666
                         show_ids=self.show_ids, indent=indent + offset)
1574
1667
        if revision.diff is not None:
1575
1668
            self.show_diff(self.to_exact_file, revision.diff, '      ')
1576
1669
        to_file.write('\n')
1651
1744
                               self.show_timezone,
1652
1745
                               date_fmt='%Y-%m-%d',
1653
1746
                               show_offset=False)
1654
 
        committer_str = revision.rev.committer.replace (' <', '  <')
 
1747
        committer_str = self.authors(revision.rev, 'first', sep=', ')
 
1748
        committer_str = committer_str.replace(' <', '  <')
1655
1749
        to_file.write('%s  %s\n\n' % (date_str,committer_str))
1656
1750
 
1657
1751
        if revision.delta is not None and revision.delta.has_changed():
1722
1816
        raise errors.BzrCommandError("unknown log formatter: %r" % name)
1723
1817
 
1724
1818
 
 
1819
def author_list_all(rev):
 
1820
    return rev.get_apparent_authors()[:]
 
1821
 
 
1822
 
 
1823
def author_list_first(rev):
 
1824
    lst = rev.get_apparent_authors()
 
1825
    try:
 
1826
        return [lst[0]]
 
1827
    except IndexError:
 
1828
        return []
 
1829
 
 
1830
 
 
1831
def author_list_committer(rev):
 
1832
    return [rev.committer]
 
1833
 
 
1834
 
 
1835
author_list_registry = registry.Registry()
 
1836
 
 
1837
author_list_registry.register('all', author_list_all,
 
1838
                              'All authors')
 
1839
 
 
1840
author_list_registry.register('first', author_list_first,
 
1841
                              'The first author')
 
1842
 
 
1843
author_list_registry.register('committer', author_list_committer,
 
1844
                              'The committer')
 
1845
 
 
1846
 
1725
1847
def show_one_log(revno, rev, delta, verbose, to_file, show_timezone):
1726
1848
    # deprecated; for compatibility
1727
1849
    lf = LongLogFormatter(to_file=to_file, show_timezone=show_timezone)
1878
2000
        lf.log_revision(lr)
1879
2001
 
1880
2002
 
1881
 
def _get_info_for_log_files(revisionspec_list, file_list):
 
2003
def _get_info_for_log_files(revisionspec_list, file_list, add_cleanup):
1882
2004
    """Find file-ids and kinds given a list of files and a revision range.
1883
2005
 
1884
2006
    We search for files at the end of the range. If not found there,
1888
2010
    :param file_list: the list of paths given on the command line;
1889
2011
      the first of these can be a branch location or a file path,
1890
2012
      the remainder must be file paths
 
2013
    :param add_cleanup: When the branch returned is read locked,
 
2014
      an unlock call will be queued to the cleanup.
1891
2015
    :return: (branch, info_list, start_rev_info, end_rev_info) where
1892
2016
      info_list is a list of (relative_path, file_id, kind) tuples where
1893
2017
      kind is one of values 'directory', 'file', 'symlink', 'tree-reference'.
1895
2019
    """
1896
2020
    from builtins import _get_revision_range, safe_relpath_files
1897
2021
    tree, b, path = bzrdir.BzrDir.open_containing_tree_or_branch(file_list[0])
1898
 
    b.lock_read()
 
2022
    add_cleanup(b.lock_read().unlock)
1899
2023
    # XXX: It's damn messy converting a list of paths to relative paths when
1900
2024
    # those paths might be deleted ones, they might be on a case-insensitive
1901
2025
    # filesystem and/or they might be in silly locations (like another branch).
1987
2111
        bug_rows = [line.split(' ', 1) for line in bug_lines]
1988
2112
        fixed_bug_urls = [row[0] for row in bug_rows if
1989
2113
                          len(row) > 1 and row[1] == 'fixed']
1990
 
        
 
2114
 
1991
2115
        if fixed_bug_urls:
1992
2116
            return {'fixes bug(s)': ' '.join(fixed_bug_urls)}
1993
2117
    return {}