~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/log.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-01-20 18:55:04 UTC
  • mfrom: (4971.2.2 505762)
  • Revision ID: pqm@pqm.ubuntu.com-20100120185504-es1x5ntwauunwxvp
(nmb) Explain bound branches in "branches" help topic

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2009 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
73
73
    repository as _mod_repository,
74
74
    revision as _mod_revision,
75
75
    revisionspec,
 
76
    trace,
76
77
    tsort,
77
 
    i18n,
78
78
    )
79
79
""")
80
80
 
81
81
from bzrlib import (
82
 
    lazy_regex,
83
82
    registry,
84
83
    )
85
84
from bzrlib.osutils import (
86
85
    format_date,
87
86
    format_date_with_offset_in_original_timezone,
88
 
    get_diff_header_encoding,
89
87
    get_terminal_encoding,
 
88
    re_compile_checked,
90
89
    terminal_width,
91
90
    )
92
91
 
106
105
    last_path = None
107
106
    revno = 1
108
107
    for revision_id in branch.revision_history():
109
 
        this_inv = branch.repository.get_inventory(revision_id)
110
 
        if this_inv.has_id(file_id):
 
108
        this_inv = branch.repository.get_revision_inventory(revision_id)
 
109
        if file_id in this_inv:
111
110
            this_ie = this_inv[file_id]
112
111
            this_path = this_inv.id2path(file_id)
113
112
        else:
153
152
             end_revision=None,
154
153
             search=None,
155
154
             limit=None,
156
 
             show_diff=False,
157
 
             match=None):
 
155
             show_diff=False):
158
156
    """Write out human-readable log of commits to this branch.
159
157
 
160
158
    This function is being retained for backwards compatibility but
183
181
        if None or 0.
184
182
 
185
183
    :param show_diff: If True, output a diff after each revision.
186
 
 
187
 
    :param match: Dictionary of search lists to use when matching revision
188
 
      properties.
189
184
    """
190
185
    # Convert old-style parameters to new-style parameters
191
186
    if specific_fileid is not None:
219
214
# make_log_request_dict() below
220
215
_DEFAULT_REQUEST_PARAMS = {
221
216
    'direction': 'reverse',
222
 
    'levels': None,
 
217
    'levels': 1,
223
218
    'generate_tags': True,
224
 
    'exclude_common_ancestry': False,
225
219
    '_match_using_deltas': True,
226
220
    }
227
221
 
228
222
 
229
223
def make_log_request_dict(direction='reverse', specific_fileids=None,
230
 
                          start_revision=None, end_revision=None, limit=None,
231
 
                          message_search=None, levels=None, generate_tags=True,
232
 
                          delta_type=None,
233
 
                          diff_type=None, _match_using_deltas=True,
234
 
                          exclude_common_ancestry=False, match=None,
235
 
                          signature=False,
236
 
                          ):
 
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):
237
227
    """Convenience function for making a logging request dictionary.
238
228
 
239
229
    Using this function may make code slightly safer by ensuring
259
249
      matching commit messages
260
250
 
261
251
    :param levels: the number of levels of revisions to
262
 
      generate; 1 for just the mainline; 0 for all levels, or None for
263
 
      a sensible default.
 
252
      generate; 1 for just the mainline; 0 for all levels.
264
253
 
265
254
    :param generate_tags: If True, include tags for matched revisions.
266
 
`
 
255
 
267
256
    :param delta_type: Either 'full', 'partial' or None.
268
257
      'full' means generate the complete delta - adds/deletes/modifies/etc;
269
258
      'partial' means filter the delta using specific_fileids;
278
267
      algorithm used for matching specific_fileids. This parameter
279
268
      may be removed in the future so bzrlib client code should NOT
280
269
      use it.
281
 
 
282
 
    :param exclude_common_ancestry: Whether -rX..Y should be interpreted as a
283
 
      range operator or as a graph difference.
284
 
 
285
 
    :param signature: show digital signature information
286
 
 
287
 
    :param match: Dictionary of list of search strings to use when filtering
288
 
      revisions. Keys can be 'message', 'author', 'committer', 'bugs' or
289
 
      the empty string to match any of the preceding properties.
290
 
 
291
270
    """
292
 
    # Take care of old style message_search parameter
293
 
    if message_search:
294
 
        if match:
295
 
            if 'message' in match:
296
 
                match['message'].append(message_search)
297
 
            else:
298
 
                match['message'] = [message_search]
299
 
        else:
300
 
            match={ 'message': [message_search] }
301
271
    return {
302
272
        'direction': direction,
303
273
        'specific_fileids': specific_fileids,
304
274
        'start_revision': start_revision,
305
275
        'end_revision': end_revision,
306
276
        'limit': limit,
 
277
        'message_search': message_search,
307
278
        'levels': levels,
308
279
        'generate_tags': generate_tags,
309
280
        'delta_type': delta_type,
310
281
        'diff_type': diff_type,
311
 
        'exclude_common_ancestry': exclude_common_ancestry,
312
 
        'signature': signature,
313
 
        'match': match,
314
282
        # Add 'private' attributes for features that may be deprecated
315
283
        '_match_using_deltas': _match_using_deltas,
316
284
    }
318
286
 
319
287
def _apply_log_request_defaults(rqst):
320
288
    """Apply default values to a request dictionary."""
321
 
    result = _DEFAULT_REQUEST_PARAMS.copy()
 
289
    result = _DEFAULT_REQUEST_PARAMS
322
290
    if rqst:
323
291
        result.update(rqst)
324
292
    return result
325
293
 
326
294
 
327
 
def format_signature_validity(rev_id, repo):
328
 
    """get the signature validity
329
 
 
330
 
    :param rev_id: revision id to validate
331
 
    :param repo: repository of revision
332
 
    :return: human readable string to print to log
333
 
    """
334
 
    from bzrlib import gpg
335
 
 
336
 
    gpg_strategy = gpg.GPGStrategy(None)
337
 
    result = repo.verify_revision(rev_id, gpg_strategy)
338
 
    if result[0] == gpg.SIGNATURE_VALID:
339
 
        return "valid signature from {0}".format(result[1])
340
 
    if result[0] == gpg.SIGNATURE_KEY_MISSING:
341
 
        return "unknown key {0}".format(result[1])
342
 
    if result[0] == gpg.SIGNATURE_NOT_VALID:
343
 
        return "invalid signature!"
344
 
    if result[0] == gpg.SIGNATURE_NOT_SIGNED:
345
 
        return "no signature"
346
 
 
347
 
 
348
295
class LogGenerator(object):
349
296
    """A generator of log revisions."""
350
297
 
357
304
 
358
305
 
359
306
class Logger(object):
360
 
    """An object that generates, formats and displays a log."""
 
307
    """An object the generates, formats and displays a log."""
361
308
 
362
309
    def __init__(self, branch, rqst):
363
310
        """Create a Logger.
395
342
        # Tweak the LogRequest based on what the LogFormatter can handle.
396
343
        # (There's no point generating stuff if the formatter can't display it.)
397
344
        rqst = self.rqst
398
 
        if rqst['levels'] is None or lf.get_levels() > rqst['levels']:
399
 
            # user didn't specify levels, use whatever the LF can handle:
400
 
            rqst['levels'] = lf.get_levels()
401
 
 
 
345
        rqst['levels'] = lf.get_levels()
402
346
        if not getattr(lf, 'supports_tags', False):
403
347
            rqst['generate_tags'] = False
404
348
        if not getattr(lf, 'supports_delta', False):
405
349
            rqst['delta_type'] = None
406
350
        if not getattr(lf, 'supports_diff', False):
407
351
            rqst['diff_type'] = None
408
 
        if not getattr(lf, 'supports_signatures', False):
409
 
            rqst['signature'] = False
410
352
 
411
353
        # Find and print the interesting revisions
412
354
        generator = self._generator_factory(self.branch, rqst)
416
358
 
417
359
    def _generator_factory(self, branch, rqst):
418
360
        """Make the LogGenerator object to use.
419
 
 
 
361
        
420
362
        Subclasses may wish to override this.
421
363
        """
422
364
        return _DefaultLogGenerator(branch, rqst)
446
388
        levels = rqst.get('levels')
447
389
        limit = rqst.get('limit')
448
390
        diff_type = rqst.get('diff_type')
449
 
        show_signature = rqst.get('signature')
450
391
        log_count = 0
451
392
        revision_iterator = self._create_log_revision_iterator()
452
393
        for revs in revision_iterator:
458
399
                    diff = None
459
400
                else:
460
401
                    diff = self._format_diff(rev, rev_id, diff_type)
461
 
                if show_signature:
462
 
                    signature = format_signature_validity(rev_id,
463
 
                                                self.branch.repository)
464
 
                else:
465
 
                    signature = None
466
402
                yield LogRevision(rev, revno, merge_depth, delta,
467
 
                    self.rev_tag_dict.get(rev_id), diff, signature)
 
403
                    self.rev_tag_dict.get(rev_id), diff)
468
404
                if limit:
469
405
                    log_count += 1
470
406
                    if log_count >= limit:
484
420
        else:
485
421
            specific_files = None
486
422
        s = StringIO()
487
 
        path_encoding = get_diff_header_encoding()
488
423
        diff.show_diff_trees(tree_1, tree_2, s, specific_files, old_label='',
489
 
            new_label='', path_encoding=path_encoding)
 
424
            new_label='')
490
425
        return s.getvalue()
491
426
 
492
427
    def _create_log_revision_iterator(self):
516
451
        generate_merge_revisions = rqst.get('levels') != 1
517
452
        delayed_graph_generation = not rqst.get('specific_fileids') and (
518
453
                rqst.get('limit') or self.start_rev_id or self.end_rev_id)
519
 
        view_revisions = _calc_view_revisions(
520
 
            self.branch, self.start_rev_id, self.end_rev_id,
521
 
            rqst.get('direction'),
522
 
            generate_merge_revisions=generate_merge_revisions,
523
 
            delayed_graph_generation=delayed_graph_generation,
524
 
            exclude_common_ancestry=rqst.get('exclude_common_ancestry'))
 
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)
525
457
 
526
458
        # Apply the other filters
527
459
        return make_log_rev_iterator(self.branch, view_revisions,
528
 
            rqst.get('delta_type'), rqst.get('match'),
 
460
            rqst.get('delta_type'), rqst.get('message_search'),
529
461
            file_ids=rqst.get('specific_fileids'),
530
462
            direction=rqst.get('direction'))
531
463
 
534
466
        # Note that we always generate the merge revisions because
535
467
        # filter_revisions_touching_file_id() requires them ...
536
468
        rqst = self.rqst
537
 
        view_revisions = _calc_view_revisions(
538
 
            self.branch, self.start_rev_id, self.end_rev_id,
539
 
            rqst.get('direction'), generate_merge_revisions=True,
540
 
            exclude_common_ancestry=rqst.get('exclude_common_ancestry'))
 
469
        view_revisions = _calc_view_revisions(self.branch, self.start_rev_id,
 
470
            self.end_rev_id, rqst.get('direction'), True)
541
471
        if not isinstance(view_revisions, list):
542
472
            view_revisions = list(view_revisions)
543
473
        view_revisions = _filter_revisions_touching_file_id(self.branch,
544
474
            rqst.get('specific_fileids')[0], view_revisions,
545
475
            include_merges=rqst.get('levels') != 1)
546
476
        return make_log_rev_iterator(self.branch, view_revisions,
547
 
            rqst.get('delta_type'), rqst.get('match'))
 
477
            rqst.get('delta_type'), rqst.get('message_search'))
548
478
 
549
479
 
550
480
def _calc_view_revisions(branch, start_rev_id, end_rev_id, direction,
551
 
                         generate_merge_revisions,
552
 
                         delayed_graph_generation=False,
553
 
                         exclude_common_ancestry=False,
554
 
                         ):
 
481
    generate_merge_revisions, delayed_graph_generation=False):
555
482
    """Calculate the revisions to view.
556
483
 
557
484
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
558
485
             a list of the same tuples.
559
486
    """
560
 
    if (exclude_common_ancestry and start_rev_id == end_rev_id):
561
 
        raise errors.BzrCommandError(
562
 
            '--exclude-common-ancestry requires two different revisions')
563
 
    if direction not in ('reverse', 'forward'):
564
 
        raise ValueError('invalid direction %r' % direction)
565
487
    br_revno, br_rev_id = branch.last_revision_info()
566
488
    if br_revno == 0:
567
489
        return []
568
490
 
569
 
    if (end_rev_id and start_rev_id == end_rev_id
570
 
        and (not generate_merge_revisions
571
 
             or not _has_merges(branch, end_rev_id))):
572
 
        # If a single revision is requested, check we can handle it
573
 
        iter_revs = _generate_one_revision(branch, end_rev_id, br_rev_id,
574
 
                                           br_revno)
575
 
    elif not generate_merge_revisions:
576
 
        # If we only want to see linear revisions, we can iterate ...
577
 
        iter_revs = _generate_flat_revisions(branch, start_rev_id, end_rev_id,
578
 
                                             direction, exclude_common_ancestry)
579
 
        if direction == 'forward':
580
 
            iter_revs = reversed(iter_revs)
 
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)
581
501
    else:
582
 
        iter_revs = _generate_all_revisions(branch, start_rev_id, end_rev_id,
583
 
                                            direction, delayed_graph_generation,
584
 
                                            exclude_common_ancestry)
585
 
        if direction == 'forward':
586
 
            iter_revs = _rebase_merge_depth(reverse_by_depth(list(iter_revs)))
587
 
    return iter_revs
 
502
        return _generate_all_revisions(branch, start_rev_id, end_rev_id,
 
503
            direction, delayed_graph_generation)
588
504
 
589
505
 
590
506
def _generate_one_revision(branch, rev_id, br_rev_id, br_revno):
592
508
        # It's the tip
593
509
        return [(br_rev_id, br_revno, 0)]
594
510
    else:
595
 
        revno_str = _compute_revno_str(branch, rev_id)
 
511
        revno = branch.revision_id_to_dotted_revno(rev_id)
 
512
        revno_str = '.'.join(str(n) for n in revno)
596
513
        return [(rev_id, revno_str, 0)]
597
514
 
598
515
 
599
 
def _generate_flat_revisions(branch, start_rev_id, end_rev_id, direction,
600
 
                             exclude_common_ancestry=False):
601
 
    result = _linear_view_revisions(
602
 
        branch, start_rev_id, end_rev_id,
603
 
        exclude_common_ancestry=exclude_common_ancestry)
 
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)
604
518
    # If a start limit was given and it's not obviously an
605
519
    # ancestor of the end limit, check it before outputting anything
606
520
    if direction == 'forward' or (start_rev_id
610
524
        except _StartNotLinearAncestor:
611
525
            raise errors.BzrCommandError('Start revision not found in'
612
526
                ' left-hand history of end revision.')
 
527
    if direction == 'forward':
 
528
        result = reversed(result)
613
529
    return result
614
530
 
615
531
 
616
532
def _generate_all_revisions(branch, start_rev_id, end_rev_id, direction,
617
 
                            delayed_graph_generation,
618
 
                            exclude_common_ancestry=False):
 
533
    delayed_graph_generation):
619
534
    # On large trees, generating the merge graph can take 30-60 seconds
620
535
    # so we delay doing it until a merge is detected, incrementally
621
536
    # returning initial (non-merge) revisions while we can.
622
 
 
623
 
    # The above is only true for old formats (<= 0.92), for newer formats, a
624
 
    # couple of seconds only should be needed to load the whole graph and the
625
 
    # other graph operations needed are even faster than that -- vila 100201
626
537
    initial_revisions = []
627
538
    if delayed_graph_generation:
628
539
        try:
629
 
            for rev_id, revno, depth in  _linear_view_revisions(
630
 
                branch, start_rev_id, end_rev_id, exclude_common_ancestry):
 
540
            for rev_id, revno, depth in \
 
541
                _linear_view_revisions(branch, start_rev_id, end_rev_id):
631
542
                if _has_merges(branch, rev_id):
632
 
                    # The end_rev_id can be nested down somewhere. We need an
633
 
                    # explicit ancestry check. There is an ambiguity here as we
634
 
                    # may not raise _StartNotLinearAncestor for a revision that
635
 
                    # is an ancestor but not a *linear* one. But since we have
636
 
                    # loaded the graph to do the check (or calculate a dotted
637
 
                    # revno), we may as well accept to show the log...  We need
638
 
                    # the check only if start_rev_id is not None as all
639
 
                    # revisions have _mod_revision.NULL_REVISION as an ancestor
640
 
                    # -- vila 20100319
641
 
                    graph = branch.repository.get_graph()
642
 
                    if (start_rev_id is not None
643
 
                        and not graph.is_ancestor(start_rev_id, end_rev_id)):
644
 
                        raise _StartNotLinearAncestor()
645
 
                    # Since we collected the revisions so far, we need to
646
 
                    # adjust end_rev_id.
647
543
                    end_rev_id = rev_id
648
544
                    break
649
545
                else:
650
546
                    initial_revisions.append((rev_id, revno, depth))
651
547
            else:
652
548
                # No merged revisions found
653
 
                return initial_revisions
 
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)
654
555
        except _StartNotLinearAncestor:
655
556
            # A merge was never detected so the lower revision limit can't
656
557
            # be nested down somewhere
657
558
            raise errors.BzrCommandError('Start revision not found in'
658
559
                ' history of end revision.')
659
560
 
660
 
    # We exit the loop above because we encounter a revision with merges, from
661
 
    # this revision, we need to switch to _graph_view_revisions.
662
 
 
663
561
    # A log including nested merges is required. If the direction is reverse,
664
562
    # we rebase the initial merge depths so that the development line is
665
563
    # shown naturally, i.e. just like it is for linear logging. We can easily
667
565
    # indented at the end seems slightly nicer in that case.
668
566
    view_revisions = chain(iter(initial_revisions),
669
567
        _graph_view_revisions(branch, start_rev_id, end_rev_id,
670
 
                              rebase_initial_depths=(direction == 'reverse'),
671
 
                              exclude_common_ancestry=exclude_common_ancestry))
672
 
    return view_revisions
 
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)
673
577
 
674
578
 
675
579
def _has_merges(branch, rev_id):
678
582
    return len(parents) > 1
679
583
 
680
584
 
681
 
def _compute_revno_str(branch, rev_id):
682
 
    """Compute the revno string from a rev_id.
683
 
 
684
 
    :return: The revno string, or None if the revision is not in the supplied
685
 
        branch.
686
 
    """
687
 
    try:
688
 
        revno = branch.revision_id_to_dotted_revno(rev_id)
689
 
    except errors.NoSuchRevision:
690
 
        # The revision must be outside of this branch
691
 
        return None
692
 
    else:
693
 
        return '.'.join(str(n) for n in revno)
694
 
 
695
 
 
696
585
def _is_obvious_ancestor(branch, start_rev_id, end_rev_id):
697
586
    """Is start_rev_id an obvious ancestor of end_rev_id?"""
698
587
    if start_rev_id and end_rev_id:
699
 
        try:
700
 
            start_dotted = branch.revision_id_to_dotted_revno(start_rev_id)
701
 
            end_dotted = branch.revision_id_to_dotted_revno(end_rev_id)
702
 
        except errors.NoSuchRevision:
703
 
            # one or both is not in the branch; not obvious
704
 
            return False
 
588
        start_dotted = branch.revision_id_to_dotted_revno(start_rev_id)
 
589
        end_dotted = branch.revision_id_to_dotted_revno(end_rev_id)
705
590
        if len(start_dotted) == 1 and len(end_dotted) == 1:
706
591
            # both on mainline
707
592
            return start_dotted[0] <= end_dotted[0]
712
597
        else:
713
598
            # not obvious
714
599
            return False
715
 
    # if either start or end is not specified then we use either the first or
716
 
    # the last revision and *they* are obvious ancestors.
717
600
    return True
718
601
 
719
602
 
720
 
def _linear_view_revisions(branch, start_rev_id, end_rev_id,
721
 
                           exclude_common_ancestry=False):
 
603
def _linear_view_revisions(branch, start_rev_id, end_rev_id):
722
604
    """Calculate a sequence of revisions to view, newest to oldest.
723
605
 
724
606
    :param start_rev_id: the lower revision-id
725
607
    :param end_rev_id: the upper revision-id
726
 
    :param exclude_common_ancestry: Whether the start_rev_id should be part of
727
 
        the iterated revisions.
728
608
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples.
729
609
    :raises _StartNotLinearAncestor: if a start_rev_id is specified but
730
 
        is not found walking the left-hand history
 
610
      is not found walking the left-hand history
731
611
    """
732
612
    br_revno, br_rev_id = branch.last_revision_info()
733
613
    repo = branch.repository
734
 
    graph = repo.get_graph()
735
614
    if start_rev_id is None and end_rev_id is None:
736
615
        cur_revno = br_revno
737
 
        for revision_id in graph.iter_lefthand_ancestry(br_rev_id,
738
 
            (_mod_revision.NULL_REVISION,)):
 
616
        for revision_id in repo.iter_reverse_revision_history(br_rev_id):
739
617
            yield revision_id, str(cur_revno), 0
740
618
            cur_revno -= 1
741
619
    else:
742
620
        if end_rev_id is None:
743
621
            end_rev_id = br_rev_id
744
622
        found_start = start_rev_id is None
745
 
        for revision_id in graph.iter_lefthand_ancestry(end_rev_id,
746
 
                (_mod_revision.NULL_REVISION,)):
747
 
            revno_str = _compute_revno_str(branch, revision_id)
 
623
        for revision_id in repo.iter_reverse_revision_history(end_rev_id):
 
624
            revno = branch.revision_id_to_dotted_revno(revision_id)
 
625
            revno_str = '.'.join(str(n) for n in revno)
748
626
            if not found_start and revision_id == start_rev_id:
749
 
                if not exclude_common_ancestry:
750
 
                    yield revision_id, revno_str, 0
 
627
                yield revision_id, revno_str, 0
751
628
                found_start = True
752
629
                break
753
630
            else:
758
635
 
759
636
 
760
637
def _graph_view_revisions(branch, start_rev_id, end_rev_id,
761
 
                          rebase_initial_depths=True,
762
 
                          exclude_common_ancestry=False):
 
638
    rebase_initial_depths=True):
763
639
    """Calculate revisions to view including merges, newest to oldest.
764
640
 
765
641
    :param branch: the branch
769
645
      revision is found?
770
646
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples.
771
647
    """
772
 
    if exclude_common_ancestry:
773
 
        stop_rule = 'with-merges-without-common-ancestry'
774
 
    else:
775
 
        stop_rule = 'with-merges'
776
648
    view_revisions = branch.iter_merge_sorted_revisions(
777
649
        start_revision_id=end_rev_id, stop_revision_id=start_rev_id,
778
 
        stop_rule=stop_rule)
 
650
        stop_rule="with-merges")
779
651
    if not rebase_initial_depths:
780
652
        for (rev_id, merge_depth, revno, end_of_merge
781
653
             ) in view_revisions:
792
664
                depth_adjustment = merge_depth
793
665
            if depth_adjustment:
794
666
                if merge_depth < depth_adjustment:
795
 
                    # From now on we reduce the depth adjustement, this can be
796
 
                    # surprising for users. The alternative requires two passes
797
 
                    # which breaks the fast display of the first revision
798
 
                    # though.
799
667
                    depth_adjustment = merge_depth
800
668
                merge_depth -= depth_adjustment
801
669
            yield rev_id, '.'.join(map(str, revno)), merge_depth
802
670
 
803
671
 
 
672
def calculate_view_revisions(branch, start_revision, end_revision, direction,
 
673
        specific_fileid, generate_merge_revisions):
 
674
    """Calculate the revisions to view.
 
675
 
 
676
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
 
677
             a list of the same tuples.
 
678
    """
 
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
    start_rev_id, end_rev_id = _get_revision_limits(branch, start_revision,
 
683
        end_revision)
 
684
    view_revisions = list(_calc_view_revisions(branch, start_rev_id, end_rev_id,
 
685
        direction, generate_merge_revisions or specific_fileid))
 
686
    if specific_fileid:
 
687
        view_revisions = _filter_revisions_touching_file_id(branch,
 
688
            specific_fileid, view_revisions,
 
689
            include_merges=generate_merge_revisions)
 
690
    return _rebase_merge_depth(view_revisions)
 
691
 
 
692
 
804
693
def _rebase_merge_depth(view_revisions):
805
694
    """Adjust depths upwards so the top level is 0."""
806
695
    # If either the first or last revision have a merge_depth of 0, we're done
850
739
    return log_rev_iterator
851
740
 
852
741
 
853
 
def _make_search_filter(branch, generate_delta, match, log_rev_iterator):
 
742
def _make_search_filter(branch, generate_delta, search, log_rev_iterator):
854
743
    """Create a filtered iterator of log_rev_iterator matching on a regex.
855
744
 
856
745
    :param branch: The branch being logged.
857
746
    :param generate_delta: Whether to generate a delta for each revision.
858
 
    :param match: A dictionary with properties as keys and lists of strings
859
 
        as values. To match, a revision may match any of the supplied strings
860
 
        within a single property but must match at least one string for each
861
 
        property.
 
747
    :param search: A user text search string.
862
748
    :param log_rev_iterator: An input iterator containing all revisions that
863
749
        could be displayed, in lists.
864
750
    :return: An iterator over lists of ((rev_id, revno, merge_depth), rev,
865
751
        delta).
866
752
    """
867
 
    if match is None:
 
753
    if search is None:
868
754
        return log_rev_iterator
869
 
    searchRE = [(k, [re.compile(x, re.IGNORECASE) for x in v])
870
 
                for (k,v) in match.iteritems()]
871
 
    return _filter_re(searchRE, log_rev_iterator)
872
 
 
873
 
 
874
 
def _filter_re(searchRE, log_rev_iterator):
 
755
    searchRE = re_compile_checked(search, re.IGNORECASE,
 
756
            'log message filter')
 
757
    return _filter_message_re(searchRE, log_rev_iterator)
 
758
 
 
759
 
 
760
def _filter_message_re(searchRE, log_rev_iterator):
875
761
    for revs in log_rev_iterator:
876
 
        new_revs = [rev for rev in revs if _match_filter(searchRE, rev[1])]
877
 
        if new_revs:
878
 
            yield new_revs
879
 
 
880
 
def _match_filter(searchRE, rev):
881
 
    strings = {
882
 
               'message': (rev.message,),
883
 
               'committer': (rev.committer,),
884
 
               'author': (rev.get_apparent_authors()),
885
 
               'bugs': list(rev.iter_bugs())
886
 
               }
887
 
    strings[''] = [item for inner_list in strings.itervalues()
888
 
                   for item in inner_list]
889
 
    for (k,v) in searchRE:
890
 
        if k in strings and not _match_any_filter(strings[k], v):
891
 
            return False
892
 
    return True
893
 
 
894
 
def _match_any_filter(strings, res):
895
 
    return any([filter(None, map(re.search, strings)) for re in res])
 
762
        new_revs = []
 
763
        for (rev_id, revno, merge_depth), rev, delta in revs:
 
764
            if searchRE.search(rev.message):
 
765
                new_revs.append(((rev_id, revno, merge_depth), rev, delta))
 
766
        yield new_revs
 
767
 
896
768
 
897
769
def _make_delta_filter(branch, generate_delta, search, log_rev_iterator,
898
770
    fileids=None, direction='reverse'):
971
843
 
972
844
def _update_fileids(delta, fileids, stop_on):
973
845
    """Update the set of file-ids to search based on file lifecycle events.
974
 
 
 
846
    
975
847
    :param fileids: a set of fileids to update
976
848
    :param stop_on: either 'add' or 'remove' - take file-ids out of the
977
849
      fileids set once their add or remove entry is detected respectively
1018
890
    :return: An iterator over lists of ((rev_id, revno, merge_depth), rev,
1019
891
        delta).
1020
892
    """
 
893
    repository = branch.repository
1021
894
    num = 9
1022
895
    for batch in log_rev_iterator:
1023
896
        batch = iter(batch)
1140
1013
    cur_revno = branch_revno
1141
1014
    rev_nos = {}
1142
1015
    mainline_revs = []
1143
 
    graph = branch.repository.get_graph()
1144
 
    for revision_id in graph.iter_lefthand_ancestry(
1145
 
            branch_last_revision, (_mod_revision.NULL_REVISION,)):
 
1016
    for revision_id in branch.repository.iter_reverse_revision_history(
 
1017
                        branch_last_revision):
1146
1018
        if cur_revno < start_revno:
1147
1019
            # We have gone far enough, but we always add 1 more revision
1148
1020
            rev_nos[revision_id] = cur_revno
1162
1034
    return mainline_revs, rev_nos, start_rev_id, end_rev_id
1163
1035
 
1164
1036
 
 
1037
def _filter_revision_range(view_revisions, start_rev_id, end_rev_id):
 
1038
    """Filter view_revisions based on revision ranges.
 
1039
 
 
1040
    :param view_revisions: A list of (revision_id, dotted_revno, merge_depth)
 
1041
            tuples to be filtered.
 
1042
 
 
1043
    :param start_rev_id: If not NONE specifies the first revision to be logged.
 
1044
            If NONE then all revisions up to the end_rev_id are logged.
 
1045
 
 
1046
    :param end_rev_id: If not NONE specifies the last revision to be logged.
 
1047
            If NONE then all revisions up to the end of the log are logged.
 
1048
 
 
1049
    :return: The filtered view_revisions.
 
1050
    """
 
1051
    # This method is no longer called by the main code path.
 
1052
    # It may be removed soon. IGC 20090127
 
1053
    if start_rev_id or end_rev_id:
 
1054
        revision_ids = [r for r, n, d in view_revisions]
 
1055
        if start_rev_id:
 
1056
            start_index = revision_ids.index(start_rev_id)
 
1057
        else:
 
1058
            start_index = 0
 
1059
        if start_rev_id == end_rev_id:
 
1060
            end_index = start_index
 
1061
        else:
 
1062
            if end_rev_id:
 
1063
                end_index = revision_ids.index(end_rev_id)
 
1064
            else:
 
1065
                end_index = len(view_revisions) - 1
 
1066
        # To include the revisions merged into the last revision,
 
1067
        # extend end_rev_id down to, but not including, the next rev
 
1068
        # with the same or lesser merge_depth
 
1069
        end_merge_depth = view_revisions[end_index][2]
 
1070
        try:
 
1071
            for index in xrange(end_index+1, len(view_revisions)+1):
 
1072
                if view_revisions[index][2] <= end_merge_depth:
 
1073
                    end_index = index - 1
 
1074
                    break
 
1075
        except IndexError:
 
1076
            # if the search falls off the end then log to the end as well
 
1077
            end_index = len(view_revisions) - 1
 
1078
        view_revisions = view_revisions[start_index:end_index+1]
 
1079
    return view_revisions
 
1080
 
 
1081
 
1165
1082
def _filter_revisions_touching_file_id(branch, file_id, view_revisions,
1166
1083
    include_merges=True):
1167
1084
    r"""Return the list of revision ids which touch a given file id.
1170
1087
    This includes the revisions which directly change the file id,
1171
1088
    and the revisions which merge these changes. So if the
1172
1089
    revision graph is::
1173
 
 
1174
1090
        A-.
1175
1091
        |\ \
1176
1092
        B C E
1203
1119
    """
1204
1120
    # Lookup all possible text keys to determine which ones actually modified
1205
1121
    # the file.
1206
 
    graph = branch.repository.get_file_graph()
1207
 
    get_parent_map = graph.get_parent_map
1208
1122
    text_keys = [(file_id, rev_id) for rev_id, revno, depth in view_revisions]
1209
1123
    next_keys = None
1210
1124
    # Looking up keys in batches of 1000 can cut the time in half, as well as
1214
1128
    #       indexing layer. We might consider passing in hints as to the known
1215
1129
    #       access pattern (sparse/clustered, high success rate/low success
1216
1130
    #       rate). This particular access is clustered with a low success rate.
 
1131
    get_parent_map = branch.repository.texts.get_parent_map
1217
1132
    modified_text_revisions = set()
1218
1133
    chunk_size = 1000
1219
1134
    for start in xrange(0, len(text_keys), chunk_size):
1246
1161
    return result
1247
1162
 
1248
1163
 
 
1164
def get_view_revisions(mainline_revs, rev_nos, branch, direction,
 
1165
                       include_merges=True):
 
1166
    """Produce an iterator of revisions to show
 
1167
    :return: an iterator of (revision_id, revno, merge_depth)
 
1168
    (if there is no revno for a revision, None is supplied)
 
1169
    """
 
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
    if not include_merges:
 
1174
        revision_ids = mainline_revs[1:]
 
1175
        if direction == 'reverse':
 
1176
            revision_ids.reverse()
 
1177
        for revision_id in revision_ids:
 
1178
            yield revision_id, str(rev_nos[revision_id]), 0
 
1179
        return
 
1180
    graph = branch.repository.get_graph()
 
1181
    # This asks for all mainline revisions, which means we only have to spider
 
1182
    # sideways, rather than depth history. That said, its still size-of-history
 
1183
    # and should be addressed.
 
1184
    # mainline_revisions always includes an extra revision at the beginning, so
 
1185
    # don't request it.
 
1186
    parent_map = dict(((key, value) for key, value in
 
1187
        graph.iter_ancestry(mainline_revs[1:]) if value is not None))
 
1188
    # filter out ghosts; merge_sort errors on ghosts.
 
1189
    rev_graph = _mod_repository._strip_NULL_ghosts(parent_map)
 
1190
    merge_sorted_revisions = tsort.merge_sort(
 
1191
        rev_graph,
 
1192
        mainline_revs[-1],
 
1193
        mainline_revs,
 
1194
        generate_revno=True)
 
1195
 
 
1196
    if direction == 'forward':
 
1197
        # forward means oldest first.
 
1198
        merge_sorted_revisions = reverse_by_depth(merge_sorted_revisions)
 
1199
    elif direction != 'reverse':
 
1200
        raise ValueError('invalid direction %r' % direction)
 
1201
 
 
1202
    for (sequence, rev_id, merge_depth, revno, end_of_merge
 
1203
         ) in merge_sorted_revisions:
 
1204
        yield rev_id, '.'.join(map(str, revno)), merge_depth
 
1205
 
 
1206
 
1249
1207
def reverse_by_depth(merge_sorted_revisions, _depth=0):
1250
1208
    """Reverse revisions by depth.
1251
1209
 
1286
1244
    """
1287
1245
 
1288
1246
    def __init__(self, rev=None, revno=None, merge_depth=0, delta=None,
1289
 
                 tags=None, diff=None, signature=None):
 
1247
                 tags=None, diff=None):
1290
1248
        self.rev = rev
1291
 
        if revno is None:
1292
 
            self.revno = None
1293
 
        else:
1294
 
            self.revno = str(revno)
 
1249
        self.revno = str(revno)
1295
1250
        self.merge_depth = merge_depth
1296
1251
        self.delta = delta
1297
1252
        self.tags = tags
1298
1253
        self.diff = diff
1299
 
        self.signature = signature
1300
1254
 
1301
1255
 
1302
1256
class LogFormatter(object):
1311
1265
    to indicate which LogRevision attributes it supports:
1312
1266
 
1313
1267
    - supports_delta must be True if this log formatter supports delta.
1314
 
      Otherwise the delta attribute may not be populated.  The 'delta_format'
1315
 
      attribute describes whether the 'short_status' format (1) or the long
1316
 
      one (2) should be used.
 
1268
        Otherwise the delta attribute may not be populated.  The 'delta_format'
 
1269
        attribute describes whether the 'short_status' format (1) or the long
 
1270
        one (2) should be used.
1317
1271
 
1318
1272
    - supports_merge_revisions must be True if this log formatter supports
1319
 
      merge revisions.  If not, then only mainline revisions will be passed
1320
 
      to the formatter.
 
1273
        merge revisions.  If not, then only mainline revisions will be passed
 
1274
        to the formatter.
1321
1275
 
1322
1276
    - preferred_levels is the number of levels this formatter defaults to.
1323
 
      The default value is zero meaning display all levels.
1324
 
      This value is only relevant if supports_merge_revisions is True.
 
1277
        The default value is zero meaning display all levels.
 
1278
        This value is only relevant if supports_merge_revisions is True.
1325
1279
 
1326
1280
    - supports_tags must be True if this log formatter supports tags.
1327
 
      Otherwise the tags attribute may not be populated.
 
1281
        Otherwise the tags attribute may not be populated.
1328
1282
 
1329
1283
    - supports_diff must be True if this log formatter supports diffs.
1330
 
      Otherwise the diff attribute may not be populated.
1331
 
 
1332
 
    - supports_signatures must be True if this log formatter supports GPG
1333
 
      signatures.
 
1284
        Otherwise the diff attribute may not be populated.
1334
1285
 
1335
1286
    Plugins can register functions to show custom revision properties using
1336
1287
    the properties_handler_registry. The registered function
1337
 
    must respect the following interface description::
1338
 
 
 
1288
    must respect the following interface description:
1339
1289
        def my_show_properties(properties_dict):
1340
1290
            # code that returns a dict {'name':'value'} of the properties
1341
1291
            # to be shown
1343
1293
    preferred_levels = 0
1344
1294
 
1345
1295
    def __init__(self, to_file, show_ids=False, show_timezone='original',
1346
 
                 delta_format=None, levels=None, show_advice=False,
1347
 
                 to_exact_file=None, author_list_handler=None):
 
1296
            delta_format=None, levels=None, show_advice=False,
 
1297
            to_exact_file=None):
1348
1298
        """Create a LogFormatter.
1349
1299
 
1350
1300
        :param to_file: the file to output to
1351
 
        :param to_exact_file: if set, gives an output stream to which
 
1301
        :param to_exact_file: if set, gives an output stream to which 
1352
1302
             non-Unicode diffs are written.
1353
1303
        :param show_ids: if True, revision-ids are to be displayed
1354
1304
        :param show_timezone: the timezone to use
1358
1308
          let the log formatter decide.
1359
1309
        :param show_advice: whether to show advice at the end of the
1360
1310
          log or not
1361
 
        :param author_list_handler: callable generating a list of
1362
 
          authors to display for a given revision
1363
1311
        """
1364
1312
        self.to_file = to_file
1365
1313
        # 'exact' stream used to show diff, it should print content 'as is'
1380
1328
        self.levels = levels
1381
1329
        self._show_advice = show_advice
1382
1330
        self._merge_count = 0
1383
 
        self._author_list_handler = author_list_handler
1384
1331
 
1385
1332
    def get_levels(self):
1386
1333
        """Get the number of levels to display or 0 for all."""
1418
1365
        return address
1419
1366
 
1420
1367
    def short_author(self, rev):
1421
 
        return self.authors(rev, 'first', short=True, sep=', ')
1422
 
 
1423
 
    def authors(self, rev, who, short=False, sep=None):
1424
 
        """Generate list of authors, taking --authors option into account.
1425
 
 
1426
 
        The caller has to specify the name of a author list handler,
1427
 
        as provided by the author list registry, using the ``who``
1428
 
        argument.  That name only sets a default, though: when the
1429
 
        user selected a different author list generation using the
1430
 
        ``--authors`` command line switch, as represented by the
1431
 
        ``author_list_handler`` constructor argument, that value takes
1432
 
        precedence.
1433
 
 
1434
 
        :param rev: The revision for which to generate the list of authors.
1435
 
        :param who: Name of the default handler.
1436
 
        :param short: Whether to shorten names to either name or address.
1437
 
        :param sep: What separator to use for automatic concatenation.
1438
 
        """
1439
 
        if self._author_list_handler is not None:
1440
 
            # The user did specify --authors, which overrides the default
1441
 
            author_list_handler = self._author_list_handler
1442
 
        else:
1443
 
            # The user didn't specify --authors, so we use the caller's default
1444
 
            author_list_handler = author_list_registry.get(who)
1445
 
        names = author_list_handler(rev)
1446
 
        if short:
1447
 
            for i in range(len(names)):
1448
 
                name, address = config.parse_username(names[i])
1449
 
                if name:
1450
 
                    names[i] = name
1451
 
                else:
1452
 
                    names[i] = address
1453
 
        if sep is not None:
1454
 
            names = sep.join(names)
1455
 
        return names
 
1368
        name, address = config.parse_username(rev.get_apparent_authors()[0])
 
1369
        if name:
 
1370
            return name
 
1371
        return address
1456
1372
 
1457
1373
    def merge_marker(self, revision):
1458
1374
        """Get the merge marker to include in the output or '' if none."""
1489
1405
        """
1490
1406
        # Revision comes directly from a foreign repository
1491
1407
        if isinstance(rev, foreign.ForeignRevision):
1492
 
            return self._format_properties(
1493
 
                rev.mapping.vcs.show_foreign_revid(rev.foreign_revid))
 
1408
            return rev.mapping.vcs.show_foreign_revid(rev.foreign_revid)
1494
1409
 
1495
1410
        # Imported foreign revision revision ids always contain :
1496
1411
        if not ":" in rev.revision_id:
1528
1443
    supports_delta = True
1529
1444
    supports_tags = True
1530
1445
    supports_diff = True
1531
 
    supports_signatures = True
1532
1446
 
1533
1447
    def __init__(self, *args, **kwargs):
1534
1448
        super(LongLogFormatter, self).__init__(*args, **kwargs)
1554
1468
                self.merge_marker(revision)))
1555
1469
        if revision.tags:
1556
1470
            lines.append('tags: %s' % (', '.join(revision.tags)))
1557
 
        if self.show_ids or revision.revno is None:
 
1471
        if self.show_ids:
1558
1472
            lines.append('revision-id: %s' % (revision.rev.revision_id,))
1559
 
        if self.show_ids:
1560
1473
            for parent_id in revision.rev.parent_ids:
1561
1474
                lines.append('parent: %s' % (parent_id,))
1562
1475
        lines.extend(self.custom_properties(revision.rev))
1563
1476
 
1564
1477
        committer = revision.rev.committer
1565
 
        authors = self.authors(revision.rev, 'all')
 
1478
        authors = revision.rev.get_apparent_authors()
1566
1479
        if authors != [committer]:
1567
1480
            lines.append('author: %s' % (", ".join(authors),))
1568
1481
        lines.append('committer: %s' % (committer,))
1573
1486
 
1574
1487
        lines.append('timestamp: %s' % (self.date_string(revision.rev),))
1575
1488
 
1576
 
        if revision.signature is not None:
1577
 
            lines.append('signature: ' + revision.signature)
1578
 
 
1579
1489
        lines.append('message:')
1580
1490
        if not revision.rev.message:
1581
1491
            lines.append('  (no message)')
1588
1498
        to_file = self.to_file
1589
1499
        to_file.write("%s%s\n" % (indent, ('\n' + indent).join(lines)))
1590
1500
        if revision.delta is not None:
1591
 
            # Use the standard status output to display changes
1592
 
            from bzrlib.delta import report_delta
1593
 
            report_delta(to_file, revision.delta, short_status=False,
1594
 
                         show_ids=self.show_ids, indent=indent)
 
1501
            # We don't respect delta_format for compatibility
 
1502
            revision.delta.show(to_file, self.show_ids, indent=indent,
 
1503
                                short_status=False)
1595
1504
        if revision.diff is not None:
1596
1505
            to_file.write(indent + 'diff:\n')
1597
1506
            to_file.flush()
1628
1537
        indent = '    ' * depth
1629
1538
        revno_width = self.revno_width_by_depth.get(depth)
1630
1539
        if revno_width is None:
1631
 
            if revision.revno is None or revision.revno.find('.') == -1:
 
1540
            if revision.revno.find('.') == -1:
1632
1541
                # mainline revno, e.g. 12345
1633
1542
                revno_width = 5
1634
1543
            else:
1642
1551
        if revision.tags:
1643
1552
            tags = ' {%s}' % (', '.join(revision.tags))
1644
1553
        to_file.write(indent + "%*s %s\t%s%s%s\n" % (revno_width,
1645
 
                revision.revno or "", self.short_author(revision.rev),
 
1554
                revision.revno, self.short_author(revision.rev),
1646
1555
                format_date(revision.rev.timestamp,
1647
1556
                            revision.rev.timezone or 0,
1648
1557
                            self.show_timezone, date_fmt="%Y-%m-%d",
1649
1558
                            show_offset=False),
1650
1559
                tags, self.merge_marker(revision)))
1651
1560
        self.show_properties(revision.rev, indent+offset)
1652
 
        if self.show_ids or revision.revno is None:
 
1561
        if self.show_ids:
1653
1562
            to_file.write(indent + offset + 'revision-id:%s\n'
1654
1563
                          % (revision.rev.revision_id,))
1655
1564
        if not revision.rev.message:
1660
1569
                to_file.write(indent + offset + '%s\n' % (l,))
1661
1570
 
1662
1571
        if revision.delta is not None:
1663
 
            # Use the standard status output to display changes
1664
 
            from bzrlib.delta import report_delta
1665
 
            report_delta(to_file, revision.delta,
1666
 
                         short_status=self.delta_format==1,
1667
 
                         show_ids=self.show_ids, indent=indent + offset)
 
1572
            revision.delta.show(to_file, self.show_ids, indent=indent + offset,
 
1573
                                short_status=self.delta_format==1)
1668
1574
        if revision.diff is not None:
1669
1575
            self.show_diff(self.to_exact_file, revision.diff, '      ')
1670
1576
        to_file.write('\n')
1708
1614
 
1709
1615
    def log_string(self, revno, rev, max_chars, tags=None, prefix=''):
1710
1616
        """Format log info into one string. Truncate tail of string
1711
 
 
1712
 
        :param revno:      revision number or None.
1713
 
                           Revision numbers counts from 1.
1714
 
        :param rev:        revision object
1715
 
        :param max_chars:  maximum length of resulting string
1716
 
        :param tags:       list of tags or None
1717
 
        :param prefix:     string to prefix each line
1718
 
        :return:           formatted truncated string
 
1617
        :param  revno:      revision number or None.
 
1618
                            Revision numbers counts from 1.
 
1619
        :param  rev:        revision object
 
1620
        :param  max_chars:  maximum length of resulting string
 
1621
        :param  tags:       list of tags or None
 
1622
        :param  prefix:     string to prefix each line
 
1623
        :return:            formatted truncated string
1719
1624
        """
1720
1625
        out = []
1721
1626
        if revno:
1722
1627
            # show revno only when is not None
1723
1628
            out.append("%s:" % revno)
1724
 
        if max_chars is not None:
1725
 
            out.append(self.truncate(self.short_author(rev), (max_chars+3)/4))
1726
 
        else:
1727
 
            out.append(self.short_author(rev))
 
1629
        out.append(self.truncate(self.short_author(rev), 20))
1728
1630
        out.append(self.date_string(rev))
1729
1631
        if len(rev.parent_ids) > 1:
1730
1632
            out.append('[merge]')
1749
1651
                               self.show_timezone,
1750
1652
                               date_fmt='%Y-%m-%d',
1751
1653
                               show_offset=False)
1752
 
        committer_str = self.authors(revision.rev, 'first', sep=', ')
1753
 
        committer_str = committer_str.replace(' <', '  <')
 
1654
        committer_str = revision.rev.committer.replace (' <', '  <')
1754
1655
        to_file.write('%s  %s\n\n' % (date_str,committer_str))
1755
1656
 
1756
1657
        if revision.delta is not None and revision.delta.has_changed():
1821
1722
        raise errors.BzrCommandError("unknown log formatter: %r" % name)
1822
1723
 
1823
1724
 
1824
 
def author_list_all(rev):
1825
 
    return rev.get_apparent_authors()[:]
1826
 
 
1827
 
 
1828
 
def author_list_first(rev):
1829
 
    lst = rev.get_apparent_authors()
1830
 
    try:
1831
 
        return [lst[0]]
1832
 
    except IndexError:
1833
 
        return []
1834
 
 
1835
 
 
1836
 
def author_list_committer(rev):
1837
 
    return [rev.committer]
1838
 
 
1839
 
 
1840
 
author_list_registry = registry.Registry()
1841
 
 
1842
 
author_list_registry.register('all', author_list_all,
1843
 
                              'All authors')
1844
 
 
1845
 
author_list_registry.register('first', author_list_first,
1846
 
                              'The first author')
1847
 
 
1848
 
author_list_registry.register('committer', author_list_committer,
1849
 
                              'The committer')
 
1725
def show_one_log(revno, rev, delta, verbose, to_file, show_timezone):
 
1726
    # deprecated; for compatibility
 
1727
    lf = LongLogFormatter(to_file=to_file, show_timezone=show_timezone)
 
1728
    lf.show(revno, rev, delta)
1850
1729
 
1851
1730
 
1852
1731
def show_changed_revisions(branch, old_rh, new_rh, to_file=None,
1917
1796
    old_revisions = set()
1918
1797
    new_history = []
1919
1798
    new_revisions = set()
1920
 
    graph = repository.get_graph()
1921
 
    new_iter = graph.iter_lefthand_ancestry(new_revision_id)
1922
 
    old_iter = graph.iter_lefthand_ancestry(old_revision_id)
 
1799
    new_iter = repository.iter_reverse_revision_history(new_revision_id)
 
1800
    old_iter = repository.iter_reverse_revision_history(old_revision_id)
1923
1801
    stop_revision = None
1924
1802
    do_old = True
1925
1803
    do_new = True
2000
1878
        lf.log_revision(lr)
2001
1879
 
2002
1880
 
2003
 
def _get_info_for_log_files(revisionspec_list, file_list, add_cleanup):
 
1881
def _get_info_for_log_files(revisionspec_list, file_list):
2004
1882
    """Find file-ids and kinds given a list of files and a revision range.
2005
1883
 
2006
1884
    We search for files at the end of the range. If not found there,
2010
1888
    :param file_list: the list of paths given on the command line;
2011
1889
      the first of these can be a branch location or a file path,
2012
1890
      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.
2015
1891
    :return: (branch, info_list, start_rev_info, end_rev_info) where
2016
1892
      info_list is a list of (relative_path, file_id, kind) tuples where
2017
1893
      kind is one of values 'directory', 'file', 'symlink', 'tree-reference'.
2018
1894
      branch will be read-locked.
2019
1895
    """
2020
 
    from builtins import _get_revision_range
 
1896
    from builtins import _get_revision_range, safe_relpath_files
2021
1897
    tree, b, path = bzrdir.BzrDir.open_containing_tree_or_branch(file_list[0])
2022
 
    add_cleanup(b.lock_read().unlock)
 
1898
    b.lock_read()
2023
1899
    # XXX: It's damn messy converting a list of paths to relative paths when
2024
1900
    # those paths might be deleted ones, they might be on a case-insensitive
2025
1901
    # filesystem and/or they might be in silly locations (like another branch).
2029
1905
    # case of running log in a nested directory, assuming paths beyond the
2030
1906
    # first one haven't been deleted ...
2031
1907
    if tree:
2032
 
        relpaths = [path] + tree.safe_relpath_files(file_list[1:])
 
1908
        relpaths = [path] + safe_relpath_files(tree, file_list[1:])
2033
1909
    else:
2034
1910
        relpaths = [path] + file_list[1:]
2035
1911
    info_list = []
2111
1987
        bug_rows = [line.split(' ', 1) for line in bug_lines]
2112
1988
        fixed_bug_urls = [row[0] for row in bug_rows if
2113
1989
                          len(row) > 1 and row[1] == 'fixed']
2114
 
 
 
1990
        
2115
1991
        if fixed_bug_urls:
2116
1992
            return {'fixes bug(s)': ' '.join(fixed_bug_urls)}
2117
1993
    return {}