~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/log.py

  • Committer: Martin von Gagern
  • Date: 2011-06-01 12:53:56 UTC
  • mto: This revision was merged to the branch mainline in revision 6009.
  • Revision ID: martin.vgagern@gmx.net-20110601125356-lwozv2vecea6hxfz
Change from no_decorate to classify as name for the argument.

The command line switch remains as --no-classify, to keep backwards
compatibility.  Users are free to include --no-classify in an alias, and
still use --classify to change back.

Show diffs side-by-side

added added

removed removed

Lines of Context:
75
75
    revisionspec,
76
76
    tsort,
77
77
    )
78
 
from bzrlib.i18n import gettext, ngettext
79
78
""")
80
79
 
81
80
from bzrlib import (
82
 
    lazy_regex,
83
81
    registry,
84
82
    )
85
83
from bzrlib.osutils import (
89
87
    get_terminal_encoding,
90
88
    terminal_width,
91
89
    )
 
90
from bzrlib.symbol_versioning import (
 
91
    deprecated_function,
 
92
    deprecated_in,
 
93
    )
92
94
 
93
95
 
94
96
def find_touching_revisions(branch, file_id):
107
109
    revno = 1
108
110
    for revision_id in branch.revision_history():
109
111
        this_inv = branch.repository.get_inventory(revision_id)
110
 
        if this_inv.has_id(file_id):
 
112
        if file_id in this_inv:
111
113
            this_ie = this_inv[file_id]
112
114
            this_path = this_inv.id2path(file_id)
113
115
        else:
153
155
             end_revision=None,
154
156
             search=None,
155
157
             limit=None,
156
 
             show_diff=False,
157
 
             match=None):
 
158
             show_diff=False):
158
159
    """Write out human-readable log of commits to this branch.
159
160
 
160
161
    This function is being retained for backwards compatibility but
183
184
        if None or 0.
184
185
 
185
186
    :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
187
    """
190
188
    # Convert old-style parameters to new-style parameters
191
189
    if specific_fileid is not None:
215
213
    Logger(branch, rqst).show(lf)
216
214
 
217
215
 
218
 
# Note: This needs to be kept in sync with the defaults in
 
216
# Note: This needs to be kept this in sync with the defaults in
219
217
# make_log_request_dict() below
220
218
_DEFAULT_REQUEST_PARAMS = {
221
219
    'direction': 'reverse',
222
 
    'levels': None,
 
220
    'levels': 1,
223
221
    'generate_tags': True,
224
222
    'exclude_common_ancestry': False,
225
223
    '_match_using_deltas': True,
228
226
 
229
227
def make_log_request_dict(direction='reverse', specific_fileids=None,
230
228
                          start_revision=None, end_revision=None, limit=None,
231
 
                          message_search=None, levels=None, generate_tags=True,
 
229
                          message_search=None, levels=1, generate_tags=True,
232
230
                          delta_type=None,
233
231
                          diff_type=None, _match_using_deltas=True,
234
 
                          exclude_common_ancestry=False, match=None,
235
 
                          signature=False, omit_merges=False,
 
232
                          exclude_common_ancestry=False,
236
233
                          ):
237
234
    """Convenience function for making a logging request dictionary.
238
235
 
259
256
      matching commit messages
260
257
 
261
258
    :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.
 
259
      generate; 1 for just the mainline; 0 for all levels.
264
260
 
265
261
    :param generate_tags: If True, include tags for matched revisions.
266
 
`
 
262
 
267
263
    :param delta_type: Either 'full', 'partial' or None.
268
264
      'full' means generate the complete delta - adds/deletes/modifies/etc;
269
265
      'partial' means filter the delta using specific_fileids;
281
277
 
282
278
    :param exclude_common_ancestry: Whether -rX..Y should be interpreted as a
283
279
      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
 
    :param omit_merges: If True, commits with more than one parent are
292
 
      omitted.
293
 
 
294
280
    """
295
 
    # Take care of old style message_search parameter
296
 
    if message_search:
297
 
        if match:
298
 
            if 'message' in match:
299
 
                match['message'].append(message_search)
300
 
            else:
301
 
                match['message'] = [message_search]
302
 
        else:
303
 
            match={ 'message': [message_search] }
304
281
    return {
305
282
        'direction': direction,
306
283
        'specific_fileids': specific_fileids,
307
284
        'start_revision': start_revision,
308
285
        'end_revision': end_revision,
309
286
        'limit': limit,
 
287
        'message_search': message_search,
310
288
        'levels': levels,
311
289
        'generate_tags': generate_tags,
312
290
        'delta_type': delta_type,
313
291
        'diff_type': diff_type,
314
292
        'exclude_common_ancestry': exclude_common_ancestry,
315
 
        'signature': signature,
316
 
        'match': match,
317
 
        'omit_merges': omit_merges,
318
293
        # Add 'private' attributes for features that may be deprecated
319
294
        '_match_using_deltas': _match_using_deltas,
320
295
    }
328
303
    return result
329
304
 
330
305
 
331
 
def format_signature_validity(rev_id, repo):
332
 
    """get the signature validity
333
 
 
334
 
    :param rev_id: revision id to validate
335
 
    :param repo: repository of revision
336
 
    :return: human readable string to print to log
337
 
    """
338
 
    from bzrlib import gpg
339
 
 
340
 
    gpg_strategy = gpg.GPGStrategy(None)
341
 
    result = repo.verify_revision(rev_id, gpg_strategy)
342
 
    if result[0] == gpg.SIGNATURE_VALID:
343
 
        return "valid signature from {0}".format(result[1])
344
 
    if result[0] == gpg.SIGNATURE_KEY_MISSING:
345
 
        return "unknown key {0}".format(result[1])
346
 
    if result[0] == gpg.SIGNATURE_NOT_VALID:
347
 
        return "invalid signature!"
348
 
    if result[0] == gpg.SIGNATURE_NOT_SIGNED:
349
 
        return "no signature"
350
 
 
351
 
 
352
306
class LogGenerator(object):
353
307
    """A generator of log revisions."""
354
308
 
399
353
        # Tweak the LogRequest based on what the LogFormatter can handle.
400
354
        # (There's no point generating stuff if the formatter can't display it.)
401
355
        rqst = self.rqst
402
 
        if rqst['levels'] is None or lf.get_levels() > rqst['levels']:
403
 
            # user didn't specify levels, use whatever the LF can handle:
404
 
            rqst['levels'] = lf.get_levels()
405
 
 
 
356
        rqst['levels'] = lf.get_levels()
406
357
        if not getattr(lf, 'supports_tags', False):
407
358
            rqst['generate_tags'] = False
408
359
        if not getattr(lf, 'supports_delta', False):
409
360
            rqst['delta_type'] = None
410
361
        if not getattr(lf, 'supports_diff', False):
411
362
            rqst['diff_type'] = None
412
 
        if not getattr(lf, 'supports_signatures', False):
413
 
            rqst['signature'] = False
414
363
 
415
364
        # Find and print the interesting revisions
416
365
        generator = self._generator_factory(self.branch, rqst)
420
369
 
421
370
    def _generator_factory(self, branch, rqst):
422
371
        """Make the LogGenerator object to use.
423
 
 
 
372
        
424
373
        Subclasses may wish to override this.
425
374
        """
426
375
        return _DefaultLogGenerator(branch, rqst)
450
399
        levels = rqst.get('levels')
451
400
        limit = rqst.get('limit')
452
401
        diff_type = rqst.get('diff_type')
453
 
        show_signature = rqst.get('signature')
454
 
        omit_merges = rqst.get('omit_merges')
455
402
        log_count = 0
456
403
        revision_iterator = self._create_log_revision_iterator()
457
404
        for revs in revision_iterator:
459
406
                # 0 levels means show everything; merge_depth counts from 0
460
407
                if levels != 0 and merge_depth >= levels:
461
408
                    continue
462
 
                if omit_merges and len(rev.parent_ids) > 1:
463
 
                    continue
464
409
                if diff_type is None:
465
410
                    diff = None
466
411
                else:
467
412
                    diff = self._format_diff(rev, rev_id, diff_type)
468
 
                if show_signature:
469
 
                    signature = format_signature_validity(rev_id,
470
 
                                                self.branch.repository)
471
 
                else:
472
 
                    signature = None
473
413
                yield LogRevision(rev, revno, merge_depth, delta,
474
 
                    self.rev_tag_dict.get(rev_id), diff, signature)
 
414
                    self.rev_tag_dict.get(rev_id), diff)
475
415
                if limit:
476
416
                    log_count += 1
477
417
                    if log_count >= limit:
532
472
 
533
473
        # Apply the other filters
534
474
        return make_log_rev_iterator(self.branch, view_revisions,
535
 
            rqst.get('delta_type'), rqst.get('match'),
 
475
            rqst.get('delta_type'), rqst.get('message_search'),
536
476
            file_ids=rqst.get('specific_fileids'),
537
477
            direction=rqst.get('direction'))
538
478
 
551
491
            rqst.get('specific_fileids')[0], view_revisions,
552
492
            include_merges=rqst.get('levels') != 1)
553
493
        return make_log_rev_iterator(self.branch, view_revisions,
554
 
            rqst.get('delta_type'), rqst.get('match'))
 
494
            rqst.get('delta_type'), rqst.get('message_search'))
555
495
 
556
496
 
557
497
def _calc_view_revisions(branch, start_rev_id, end_rev_id, direction,
565
505
             a list of the same tuples.
566
506
    """
567
507
    if (exclude_common_ancestry and start_rev_id == end_rev_id):
568
 
        raise errors.BzrCommandError(gettext(
569
 
            '--exclude-common-ancestry requires two different revisions'))
 
508
        raise errors.BzrCommandError(
 
509
            '--exclude-common-ancestry requires two different revisions')
570
510
    if direction not in ('reverse', 'forward'):
571
 
        raise ValueError(gettext('invalid direction %r') % direction)
 
511
        raise ValueError('invalid direction %r' % direction)
572
512
    br_revno, br_rev_id = branch.last_revision_info()
573
513
    if br_revno == 0:
574
514
        return []
615
555
        try:
616
556
            result = list(result)
617
557
        except _StartNotLinearAncestor:
618
 
            raise errors.BzrCommandError(gettext('Start revision not found in'
619
 
                ' left-hand history of end revision.'))
 
558
            raise errors.BzrCommandError('Start revision not found in'
 
559
                ' left-hand history of end revision.')
620
560
    return result
621
561
 
622
562
 
661
601
        except _StartNotLinearAncestor:
662
602
            # A merge was never detected so the lower revision limit can't
663
603
            # be nested down somewhere
664
 
            raise errors.BzrCommandError(gettext('Start revision not found in'
665
 
                ' history of end revision.'))
 
604
            raise errors.BzrCommandError('Start revision not found in'
 
605
                ' history of end revision.')
666
606
 
667
607
    # We exit the loop above because we encounter a revision with merges, from
668
608
    # this revision, we need to switch to _graph_view_revisions.
738
678
    """
739
679
    br_revno, br_rev_id = branch.last_revision_info()
740
680
    repo = branch.repository
741
 
    graph = repo.get_graph()
742
681
    if start_rev_id is None and end_rev_id is None:
743
682
        cur_revno = br_revno
744
 
        for revision_id in graph.iter_lefthand_ancestry(br_rev_id,
745
 
            (_mod_revision.NULL_REVISION,)):
 
683
        for revision_id in repo.iter_reverse_revision_history(br_rev_id):
746
684
            yield revision_id, str(cur_revno), 0
747
685
            cur_revno -= 1
748
686
    else:
749
687
        if end_rev_id is None:
750
688
            end_rev_id = br_rev_id
751
689
        found_start = start_rev_id is None
752
 
        for revision_id in graph.iter_lefthand_ancestry(end_rev_id,
753
 
                (_mod_revision.NULL_REVISION,)):
 
690
        for revision_id in repo.iter_reverse_revision_history(end_rev_id):
754
691
            revno_str = _compute_revno_str(branch, revision_id)
755
692
            if not found_start and revision_id == start_rev_id:
756
693
                if not exclude_common_ancestry:
808
745
            yield rev_id, '.'.join(map(str, revno)), merge_depth
809
746
 
810
747
 
 
748
@deprecated_function(deprecated_in((2, 2, 0)))
 
749
def calculate_view_revisions(branch, start_revision, end_revision, direction,
 
750
        specific_fileid, generate_merge_revisions):
 
751
    """Calculate the revisions to view.
 
752
 
 
753
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
 
754
             a list of the same tuples.
 
755
    """
 
756
    start_rev_id, end_rev_id = _get_revision_limits(branch, start_revision,
 
757
        end_revision)
 
758
    view_revisions = list(_calc_view_revisions(branch, start_rev_id, end_rev_id,
 
759
        direction, generate_merge_revisions or specific_fileid))
 
760
    if specific_fileid:
 
761
        view_revisions = _filter_revisions_touching_file_id(branch,
 
762
            specific_fileid, view_revisions,
 
763
            include_merges=generate_merge_revisions)
 
764
    return _rebase_merge_depth(view_revisions)
 
765
 
 
766
 
811
767
def _rebase_merge_depth(view_revisions):
812
768
    """Adjust depths upwards so the top level is 0."""
813
769
    # If either the first or last revision have a merge_depth of 0, we're done
857
813
    return log_rev_iterator
858
814
 
859
815
 
860
 
def _make_search_filter(branch, generate_delta, match, log_rev_iterator):
 
816
def _make_search_filter(branch, generate_delta, search, log_rev_iterator):
861
817
    """Create a filtered iterator of log_rev_iterator matching on a regex.
862
818
 
863
819
    :param branch: The branch being logged.
864
820
    :param generate_delta: Whether to generate a delta for each revision.
865
 
    :param match: A dictionary with properties as keys and lists of strings
866
 
        as values. To match, a revision may match any of the supplied strings
867
 
        within a single property but must match at least one string for each
868
 
        property.
 
821
    :param search: A user text search string.
869
822
    :param log_rev_iterator: An input iterator containing all revisions that
870
823
        could be displayed, in lists.
871
824
    :return: An iterator over lists of ((rev_id, revno, merge_depth), rev,
872
825
        delta).
873
826
    """
874
 
    if match is None:
 
827
    if search is None:
875
828
        return log_rev_iterator
876
 
    searchRE = [(k, [re.compile(x, re.IGNORECASE) for x in v])
877
 
                for (k,v) in match.iteritems()]
878
 
    return _filter_re(searchRE, log_rev_iterator)
879
 
 
880
 
 
881
 
def _filter_re(searchRE, log_rev_iterator):
 
829
    searchRE = re.compile(search, re.IGNORECASE)
 
830
    return _filter_message_re(searchRE, log_rev_iterator)
 
831
 
 
832
 
 
833
def _filter_message_re(searchRE, log_rev_iterator):
882
834
    for revs in log_rev_iterator:
883
 
        new_revs = [rev for rev in revs if _match_filter(searchRE, rev[1])]
884
 
        if new_revs:
885
 
            yield new_revs
886
 
 
887
 
def _match_filter(searchRE, rev):
888
 
    strings = {
889
 
               'message': (rev.message,),
890
 
               'committer': (rev.committer,),
891
 
               'author': (rev.get_apparent_authors()),
892
 
               'bugs': list(rev.iter_bugs())
893
 
               }
894
 
    strings[''] = [item for inner_list in strings.itervalues()
895
 
                   for item in inner_list]
896
 
    for (k,v) in searchRE:
897
 
        if k in strings and not _match_any_filter(strings[k], v):
898
 
            return False
899
 
    return True
900
 
 
901
 
def _match_any_filter(strings, res):
902
 
    return any([filter(None, map(re.search, strings)) for re in res])
 
835
        new_revs = []
 
836
        for (rev_id, revno, merge_depth), rev, delta in revs:
 
837
            if searchRE.search(rev.message):
 
838
                new_revs.append(((rev_id, revno, merge_depth), rev, delta))
 
839
        yield new_revs
 
840
 
903
841
 
904
842
def _make_delta_filter(branch, generate_delta, search, log_rev_iterator,
905
843
    fileids=None, direction='reverse'):
978
916
 
979
917
def _update_fileids(delta, fileids, stop_on):
980
918
    """Update the set of file-ids to search based on file lifecycle events.
981
 
 
 
919
    
982
920
    :param fileids: a set of fileids to update
983
921
    :param stop_on: either 'add' or 'remove' - take file-ids out of the
984
922
      fileids set once their add or remove entry is detected respectively
1025
963
    :return: An iterator over lists of ((rev_id, revno, merge_depth), rev,
1026
964
        delta).
1027
965
    """
 
966
    repository = branch.repository
1028
967
    num = 9
1029
968
    for batch in log_rev_iterator:
1030
969
        batch = iter(batch)
1079
1018
    if branch_revno != 0:
1080
1019
        if (start_rev_id == _mod_revision.NULL_REVISION
1081
1020
            or end_rev_id == _mod_revision.NULL_REVISION):
1082
 
            raise errors.BzrCommandError(gettext('Logging revision 0 is invalid.'))
 
1021
            raise errors.BzrCommandError('Logging revision 0 is invalid.')
1083
1022
        if start_revno > end_revno:
1084
 
            raise errors.BzrCommandError(gettext("Start revision must be "
1085
 
                                         "older than the end revision."))
 
1023
            raise errors.BzrCommandError("Start revision must be older than "
 
1024
                                         "the end revision.")
1086
1025
    return (start_rev_id, end_rev_id)
1087
1026
 
1088
1027
 
1137
1076
 
1138
1077
    if ((start_rev_id == _mod_revision.NULL_REVISION)
1139
1078
        or (end_rev_id == _mod_revision.NULL_REVISION)):
1140
 
        raise errors.BzrCommandError(gettext('Logging revision 0 is invalid.'))
 
1079
        raise errors.BzrCommandError('Logging revision 0 is invalid.')
1141
1080
    if start_revno > end_revno:
1142
 
        raise errors.BzrCommandError(gettext("Start revision must be older "
1143
 
                                     "than the end revision."))
 
1081
        raise errors.BzrCommandError("Start revision must be older than "
 
1082
                                     "the end revision.")
1144
1083
 
1145
1084
    if end_revno < start_revno:
1146
1085
        return None, None, None, None
1147
1086
    cur_revno = branch_revno
1148
1087
    rev_nos = {}
1149
1088
    mainline_revs = []
1150
 
    graph = branch.repository.get_graph()
1151
 
    for revision_id in graph.iter_lefthand_ancestry(
1152
 
            branch_last_revision, (_mod_revision.NULL_REVISION,)):
 
1089
    for revision_id in branch.repository.iter_reverse_revision_history(
 
1090
                        branch_last_revision):
1153
1091
        if cur_revno < start_revno:
1154
1092
            # We have gone far enough, but we always add 1 more revision
1155
1093
            rev_nos[revision_id] = cur_revno
1169
1107
    return mainline_revs, rev_nos, start_rev_id, end_rev_id
1170
1108
 
1171
1109
 
 
1110
@deprecated_function(deprecated_in((2, 2, 0)))
 
1111
def _filter_revision_range(view_revisions, start_rev_id, end_rev_id):
 
1112
    """Filter view_revisions based on revision ranges.
 
1113
 
 
1114
    :param view_revisions: A list of (revision_id, dotted_revno, merge_depth)
 
1115
            tuples to be filtered.
 
1116
 
 
1117
    :param start_rev_id: If not NONE specifies the first revision to be logged.
 
1118
            If NONE then all revisions up to the end_rev_id are logged.
 
1119
 
 
1120
    :param end_rev_id: If not NONE specifies the last revision to be logged.
 
1121
            If NONE then all revisions up to the end of the log are logged.
 
1122
 
 
1123
    :return: The filtered view_revisions.
 
1124
    """
 
1125
    if start_rev_id or end_rev_id:
 
1126
        revision_ids = [r for r, n, d in view_revisions]
 
1127
        if start_rev_id:
 
1128
            start_index = revision_ids.index(start_rev_id)
 
1129
        else:
 
1130
            start_index = 0
 
1131
        if start_rev_id == end_rev_id:
 
1132
            end_index = start_index
 
1133
        else:
 
1134
            if end_rev_id:
 
1135
                end_index = revision_ids.index(end_rev_id)
 
1136
            else:
 
1137
                end_index = len(view_revisions) - 1
 
1138
        # To include the revisions merged into the last revision,
 
1139
        # extend end_rev_id down to, but not including, the next rev
 
1140
        # with the same or lesser merge_depth
 
1141
        end_merge_depth = view_revisions[end_index][2]
 
1142
        try:
 
1143
            for index in xrange(end_index+1, len(view_revisions)+1):
 
1144
                if view_revisions[index][2] <= end_merge_depth:
 
1145
                    end_index = index - 1
 
1146
                    break
 
1147
        except IndexError:
 
1148
            # if the search falls off the end then log to the end as well
 
1149
            end_index = len(view_revisions) - 1
 
1150
        view_revisions = view_revisions[start_index:end_index+1]
 
1151
    return view_revisions
 
1152
 
 
1153
 
1172
1154
def _filter_revisions_touching_file_id(branch, file_id, view_revisions,
1173
1155
    include_merges=True):
1174
1156
    r"""Return the list of revision ids which touch a given file id.
1253
1235
    return result
1254
1236
 
1255
1237
 
 
1238
@deprecated_function(deprecated_in((2, 2, 0)))
 
1239
def get_view_revisions(mainline_revs, rev_nos, branch, direction,
 
1240
                       include_merges=True):
 
1241
    """Produce an iterator of revisions to show
 
1242
    :return: an iterator of (revision_id, revno, merge_depth)
 
1243
    (if there is no revno for a revision, None is supplied)
 
1244
    """
 
1245
    if not include_merges:
 
1246
        revision_ids = mainline_revs[1:]
 
1247
        if direction == 'reverse':
 
1248
            revision_ids.reverse()
 
1249
        for revision_id in revision_ids:
 
1250
            yield revision_id, str(rev_nos[revision_id]), 0
 
1251
        return
 
1252
    graph = branch.repository.get_graph()
 
1253
    # This asks for all mainline revisions, which means we only have to spider
 
1254
    # sideways, rather than depth history. That said, its still size-of-history
 
1255
    # and should be addressed.
 
1256
    # mainline_revisions always includes an extra revision at the beginning, so
 
1257
    # don't request it.
 
1258
    parent_map = dict(((key, value) for key, value in
 
1259
        graph.iter_ancestry(mainline_revs[1:]) if value is not None))
 
1260
    # filter out ghosts; merge_sort errors on ghosts.
 
1261
    rev_graph = _mod_repository._strip_NULL_ghosts(parent_map)
 
1262
    merge_sorted_revisions = tsort.merge_sort(
 
1263
        rev_graph,
 
1264
        mainline_revs[-1],
 
1265
        mainline_revs,
 
1266
        generate_revno=True)
 
1267
 
 
1268
    if direction == 'forward':
 
1269
        # forward means oldest first.
 
1270
        merge_sorted_revisions = reverse_by_depth(merge_sorted_revisions)
 
1271
    elif direction != 'reverse':
 
1272
        raise ValueError('invalid direction %r' % direction)
 
1273
 
 
1274
    for (sequence, rev_id, merge_depth, revno, end_of_merge
 
1275
         ) in merge_sorted_revisions:
 
1276
        yield rev_id, '.'.join(map(str, revno)), merge_depth
 
1277
 
 
1278
 
1256
1279
def reverse_by_depth(merge_sorted_revisions, _depth=0):
1257
1280
    """Reverse revisions by depth.
1258
1281
 
1293
1316
    """
1294
1317
 
1295
1318
    def __init__(self, rev=None, revno=None, merge_depth=0, delta=None,
1296
 
                 tags=None, diff=None, signature=None):
 
1319
                 tags=None, diff=None):
1297
1320
        self.rev = rev
1298
1321
        if revno is None:
1299
1322
            self.revno = None
1303
1326
        self.delta = delta
1304
1327
        self.tags = tags
1305
1328
        self.diff = diff
1306
 
        self.signature = signature
1307
1329
 
1308
1330
 
1309
1331
class LogFormatter(object):
1336
1358
    - supports_diff must be True if this log formatter supports diffs.
1337
1359
      Otherwise the diff attribute may not be populated.
1338
1360
 
1339
 
    - supports_signatures must be True if this log formatter supports GPG
1340
 
      signatures.
1341
 
 
1342
1361
    Plugins can register functions to show custom revision properties using
1343
1362
    the properties_handler_registry. The registered function
1344
1363
    must respect the following interface description::
1355
1374
        """Create a LogFormatter.
1356
1375
 
1357
1376
        :param to_file: the file to output to
1358
 
        :param to_exact_file: if set, gives an output stream to which
 
1377
        :param to_exact_file: if set, gives an output stream to which 
1359
1378
             non-Unicode diffs are written.
1360
1379
        :param show_ids: if True, revision-ids are to be displayed
1361
1380
        :param show_timezone: the timezone to use
1412
1431
            if advice_sep:
1413
1432
                self.to_file.write(advice_sep)
1414
1433
            self.to_file.write(
1415
 
                "Use --include-merged or -n0 to see merged revisions.\n")
 
1434
                "Use --include-merges or -n0 to see merged revisions.\n")
1416
1435
 
1417
1436
    def get_advice_separator(self):
1418
1437
        """Get the text separating the log from the closing advice."""
1535
1554
    supports_delta = True
1536
1555
    supports_tags = True
1537
1556
    supports_diff = True
1538
 
    supports_signatures = True
1539
1557
 
1540
1558
    def __init__(self, *args, **kwargs):
1541
1559
        super(LongLogFormatter, self).__init__(*args, **kwargs)
1580
1598
 
1581
1599
        lines.append('timestamp: %s' % (self.date_string(revision.rev),))
1582
1600
 
1583
 
        if revision.signature is not None:
1584
 
            lines.append('signature: ' + revision.signature)
1585
 
 
1586
1601
        lines.append('message:')
1587
1602
        if not revision.rev.message:
1588
1603
            lines.append('  (no message)')
1597
1612
        if revision.delta is not None:
1598
1613
            # Use the standard status output to display changes
1599
1614
            from bzrlib.delta import report_delta
1600
 
            report_delta(to_file, revision.delta, short_status=False,
 
1615
            report_delta(to_file, revision.delta, short_status=False, 
1601
1616
                         show_ids=self.show_ids, indent=indent)
1602
1617
        if revision.diff is not None:
1603
1618
            to_file.write(indent + 'diff:\n')
1669
1684
        if revision.delta is not None:
1670
1685
            # Use the standard status output to display changes
1671
1686
            from bzrlib.delta import report_delta
1672
 
            report_delta(to_file, revision.delta,
1673
 
                         short_status=self.delta_format==1,
 
1687
            report_delta(to_file, revision.delta, 
 
1688
                         short_status=self.delta_format==1, 
1674
1689
                         show_ids=self.show_ids, indent=indent + offset)
1675
1690
        if revision.diff is not None:
1676
1691
            self.show_diff(self.to_exact_file, revision.diff, '      ')
1825
1840
    try:
1826
1841
        return log_formatter_registry.make_formatter(name, *args, **kwargs)
1827
1842
    except KeyError:
1828
 
        raise errors.BzrCommandError(gettext("unknown log formatter: %r") % name)
 
1843
        raise errors.BzrCommandError("unknown log formatter: %r" % name)
1829
1844
 
1830
1845
 
1831
1846
def author_list_all(rev):
1856
1871
                              'The committer')
1857
1872
 
1858
1873
 
 
1874
def show_one_log(revno, rev, delta, verbose, to_file, show_timezone):
 
1875
    # deprecated; for compatibility
 
1876
    lf = LongLogFormatter(to_file=to_file, show_timezone=show_timezone)
 
1877
    lf.show(revno, rev, delta)
 
1878
 
 
1879
 
1859
1880
def show_changed_revisions(branch, old_rh, new_rh, to_file=None,
1860
1881
                           log_format='long'):
1861
1882
    """Show the change in revision history comparing the old revision history to the new one.
1924
1945
    old_revisions = set()
1925
1946
    new_history = []
1926
1947
    new_revisions = set()
1927
 
    graph = repository.get_graph()
1928
 
    new_iter = graph.iter_lefthand_ancestry(new_revision_id)
1929
 
    old_iter = graph.iter_lefthand_ancestry(old_revision_id)
 
1948
    new_iter = repository.iter_reverse_revision_history(new_revision_id)
 
1949
    old_iter = repository.iter_reverse_revision_history(old_revision_id)
1930
1950
    stop_revision = None
1931
1951
    do_old = True
1932
1952
    do_new = True
2120
2140
                          len(row) > 1 and row[1] == 'fixed']
2121
2141
 
2122
2142
        if fixed_bug_urls:
2123
 
            return {ngettext('fixes bug', 'fixes bugs', len(fixed_bug_urls)):\
2124
 
                    ' '.join(fixed_bug_urls)}
 
2143
            return {'fixes bug(s)': ' '.join(fixed_bug_urls)}
2125
2144
    return {}
2126
2145
 
2127
2146
properties_handler_registry.register('bugs_properties_handler',