/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/log.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006, 2007, 2009 Canonical Ltd
 
1
# Copyright (C) 2005-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
83
83
    )
84
84
from bzrlib.osutils import (
85
85
    format_date,
 
86
    format_date_with_offset_in_original_timezone,
86
87
    get_terminal_encoding,
87
88
    re_compile_checked,
88
89
    terminal_width,
89
90
    )
 
91
from bzrlib.symbol_versioning import (
 
92
    deprecated_function,
 
93
    deprecated_in,
 
94
    )
90
95
 
91
96
 
92
97
def find_touching_revisions(branch, file_id):
104
109
    last_path = None
105
110
    revno = 1
106
111
    for revision_id in branch.revision_history():
107
 
        this_inv = branch.repository.get_revision_inventory(revision_id)
 
112
        this_inv = branch.repository.get_inventory(revision_id)
108
113
        if file_id in this_inv:
109
114
            this_ie = this_inv[file_id]
110
115
            this_path = this_inv.id2path(file_id)
215
220
    'direction': 'reverse',
216
221
    'levels': 1,
217
222
    'generate_tags': True,
 
223
    'exclude_common_ancestry': False,
218
224
    '_match_using_deltas': True,
219
225
    }
220
226
 
221
227
 
222
228
def make_log_request_dict(direction='reverse', specific_fileids=None,
223
 
    start_revision=None, end_revision=None, limit=None,
224
 
    message_search=None, levels=1, generate_tags=True, delta_type=None,
225
 
    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
                          ):
226
235
    """Convenience function for making a logging request dictionary.
227
236
 
228
237
    Using this function may make code slightly safer by ensuring
266
275
      algorithm used for matching specific_fileids. This parameter
267
276
      may be removed in the future so bzrlib client code should NOT
268
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.
269
281
    """
270
282
    return {
271
283
        'direction': direction,
278
290
        'generate_tags': generate_tags,
279
291
        'delta_type': delta_type,
280
292
        'diff_type': diff_type,
 
293
        'exclude_common_ancestry': exclude_common_ancestry,
281
294
        # Add 'private' attributes for features that may be deprecated
282
295
        '_match_using_deltas': _match_using_deltas,
283
296
    }
303
316
 
304
317
 
305
318
class Logger(object):
306
 
    """An object the generates, formats and displays a log."""
 
319
    """An object that generates, formats and displays a log."""
307
320
 
308
321
    def __init__(self, branch, rqst):
309
322
        """Create a Logger.
384
397
        :return: An iterator yielding LogRevision objects.
385
398
        """
386
399
        rqst = self.rqst
 
400
        levels = rqst.get('levels')
 
401
        limit = rqst.get('limit')
 
402
        diff_type = rqst.get('diff_type')
387
403
        log_count = 0
388
404
        revision_iterator = self._create_log_revision_iterator()
389
405
        for revs in revision_iterator:
390
406
            for (rev_id, revno, merge_depth), rev, delta in revs:
391
407
                # 0 levels means show everything; merge_depth counts from 0
392
 
                levels = rqst.get('levels')
393
408
                if levels != 0 and merge_depth >= levels:
394
409
                    continue
395
 
                diff = self._format_diff(rev, rev_id)
 
410
                if diff_type is None:
 
411
                    diff = None
 
412
                else:
 
413
                    diff = self._format_diff(rev, rev_id, diff_type)
396
414
                yield LogRevision(rev, revno, merge_depth, delta,
397
415
                    self.rev_tag_dict.get(rev_id), diff)
398
 
                limit = rqst.get('limit')
399
416
                if limit:
400
417
                    log_count += 1
401
418
                    if log_count >= limit:
402
419
                        return
403
420
 
404
 
    def _format_diff(self, rev, rev_id):
405
 
        diff_type = self.rqst.get('diff_type')
406
 
        if diff_type is None:
407
 
            return None
 
421
    def _format_diff(self, rev, rev_id, diff_type):
408
422
        repo = self.branch.repository
409
423
        if len(rev.parent_ids) == 0:
410
424
            ancestor_id = _mod_revision.NULL_REVISION
449
463
        generate_merge_revisions = rqst.get('levels') != 1
450
464
        delayed_graph_generation = not rqst.get('specific_fileids') and (
451
465
                rqst.get('limit') or self.start_rev_id or self.end_rev_id)
452
 
        view_revisions = _calc_view_revisions(self.branch, self.start_rev_id,
453
 
            self.end_rev_id, rqst.get('direction'), generate_merge_revisions,
454
 
            delayed_graph_generation=delayed_graph_generation)
 
466
        view_revisions = _calc_view_revisions(
 
467
            self.branch, self.start_rev_id, self.end_rev_id,
 
468
            rqst.get('direction'),
 
469
            generate_merge_revisions=generate_merge_revisions,
 
470
            delayed_graph_generation=delayed_graph_generation,
 
471
            exclude_common_ancestry=rqst.get('exclude_common_ancestry'))
455
472
 
456
473
        # Apply the other filters
457
474
        return make_log_rev_iterator(self.branch, view_revisions,
464
481
        # Note that we always generate the merge revisions because
465
482
        # filter_revisions_touching_file_id() requires them ...
466
483
        rqst = self.rqst
467
 
        view_revisions = _calc_view_revisions(self.branch, self.start_rev_id,
468
 
            self.end_rev_id, rqst.get('direction'), True)
 
484
        view_revisions = _calc_view_revisions(
 
485
            self.branch, self.start_rev_id, self.end_rev_id,
 
486
            rqst.get('direction'), generate_merge_revisions=True,
 
487
            exclude_common_ancestry=rqst.get('exclude_common_ancestry'))
469
488
        if not isinstance(view_revisions, list):
470
489
            view_revisions = list(view_revisions)
471
490
        view_revisions = _filter_revisions_touching_file_id(self.branch,
476
495
 
477
496
 
478
497
def _calc_view_revisions(branch, start_rev_id, end_rev_id, direction,
479
 
    generate_merge_revisions, delayed_graph_generation=False):
 
498
                         generate_merge_revisions,
 
499
                         delayed_graph_generation=False,
 
500
                         exclude_common_ancestry=False,
 
501
                         ):
480
502
    """Calculate the revisions to view.
481
503
 
482
504
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
483
505
             a list of the same tuples.
484
506
    """
 
507
    if (exclude_common_ancestry and start_rev_id == end_rev_id):
 
508
        raise errors.BzrCommandError(
 
509
            '--exclude-common-ancestry requires two different revisions')
 
510
    if direction not in ('reverse', 'forward'):
 
511
        raise ValueError('invalid direction %r' % direction)
485
512
    br_revno, br_rev_id = branch.last_revision_info()
486
513
    if br_revno == 0:
487
514
        return []
488
515
 
489
 
    # If a single revision is requested, check we can handle it
490
 
    generate_single_revision = (end_rev_id and start_rev_id == end_rev_id and
491
 
        (not generate_merge_revisions or not _has_merges(branch, end_rev_id)))
492
 
    if generate_single_revision:
493
 
        return _generate_one_revision(branch, end_rev_id, br_rev_id, br_revno)
494
 
 
495
 
    # If we only want to see linear revisions, we can iterate ...
496
 
    if not generate_merge_revisions:
497
 
        return _generate_flat_revisions(branch, start_rev_id, end_rev_id,
498
 
            direction)
 
516
    if (end_rev_id and start_rev_id == end_rev_id
 
517
        and (not generate_merge_revisions
 
518
             or not _has_merges(branch, end_rev_id))):
 
519
        # If a single revision is requested, check we can handle it
 
520
        iter_revs = _generate_one_revision(branch, end_rev_id, br_rev_id,
 
521
                                           br_revno)
 
522
    elif not generate_merge_revisions:
 
523
        # If we only want to see linear revisions, we can iterate ...
 
524
        iter_revs = _generate_flat_revisions(branch, start_rev_id, end_rev_id,
 
525
                                             direction)
 
526
        if direction == 'forward':
 
527
            iter_revs = reversed(iter_revs)
499
528
    else:
500
 
        return _generate_all_revisions(branch, start_rev_id, end_rev_id,
501
 
            direction, delayed_graph_generation)
 
529
        iter_revs = _generate_all_revisions(branch, start_rev_id, end_rev_id,
 
530
                                            direction, delayed_graph_generation,
 
531
                                            exclude_common_ancestry)
 
532
        if direction == 'forward':
 
533
            iter_revs = _rebase_merge_depth(reverse_by_depth(list(iter_revs)))
 
534
    return iter_revs
502
535
 
503
536
 
504
537
def _generate_one_revision(branch, rev_id, br_rev_id, br_revno):
522
555
        except _StartNotLinearAncestor:
523
556
            raise errors.BzrCommandError('Start revision not found in'
524
557
                ' left-hand history of end revision.')
525
 
    if direction == 'forward':
526
 
        result = reversed(result)
527
558
    return result
528
559
 
529
560
 
530
561
def _generate_all_revisions(branch, start_rev_id, end_rev_id, direction,
531
 
    delayed_graph_generation):
 
562
                            delayed_graph_generation,
 
563
                            exclude_common_ancestry=False):
532
564
    # On large trees, generating the merge graph can take 30-60 seconds
533
565
    # so we delay doing it until a merge is detected, incrementally
534
566
    # returning initial (non-merge) revisions while we can.
 
567
 
 
568
    # The above is only true for old formats (<= 0.92), for newer formats, a
 
569
    # couple of seconds only should be needed to load the whole graph and the
 
570
    # other graph operations needed are even faster than that -- vila 100201
535
571
    initial_revisions = []
536
572
    if delayed_graph_generation:
537
573
        try:
538
 
            for rev_id, revno, depth in \
539
 
                _linear_view_revisions(branch, start_rev_id, end_rev_id):
 
574
            for rev_id, revno, depth in  _linear_view_revisions(
 
575
                branch, start_rev_id, end_rev_id):
540
576
                if _has_merges(branch, rev_id):
 
577
                    # The end_rev_id can be nested down somewhere. We need an
 
578
                    # explicit ancestry check. There is an ambiguity here as we
 
579
                    # may not raise _StartNotLinearAncestor for a revision that
 
580
                    # is an ancestor but not a *linear* one. But since we have
 
581
                    # loaded the graph to do the check (or calculate a dotted
 
582
                    # revno), we may as well accept to show the log...  We need
 
583
                    # the check only if start_rev_id is not None as all
 
584
                    # revisions have _mod_revision.NULL_REVISION as an ancestor
 
585
                    # -- vila 20100319
 
586
                    graph = branch.repository.get_graph()
 
587
                    if (start_rev_id is not None
 
588
                        and not graph.is_ancestor(start_rev_id, end_rev_id)):
 
589
                        raise _StartNotLinearAncestor()
 
590
                    # Since we collected the revisions so far, we need to
 
591
                    # adjust end_rev_id.
541
592
                    end_rev_id = rev_id
542
593
                    break
543
594
                else:
544
595
                    initial_revisions.append((rev_id, revno, depth))
545
596
            else:
546
597
                # No merged revisions found
547
 
                if direction == 'reverse':
548
 
                    return initial_revisions
549
 
                elif direction == 'forward':
550
 
                    return reversed(initial_revisions)
551
 
                else:
552
 
                    raise ValueError('invalid direction %r' % direction)
 
598
                return initial_revisions
553
599
        except _StartNotLinearAncestor:
554
600
            # A merge was never detected so the lower revision limit can't
555
601
            # be nested down somewhere
556
602
            raise errors.BzrCommandError('Start revision not found in'
557
603
                ' history of end revision.')
558
604
 
 
605
    # We exit the loop above because we encounter a revision with merges, from
 
606
    # this revision, we need to switch to _graph_view_revisions.
 
607
 
559
608
    # A log including nested merges is required. If the direction is reverse,
560
609
    # we rebase the initial merge depths so that the development line is
561
610
    # shown naturally, i.e. just like it is for linear logging. We can easily
563
612
    # indented at the end seems slightly nicer in that case.
564
613
    view_revisions = chain(iter(initial_revisions),
565
614
        _graph_view_revisions(branch, start_rev_id, end_rev_id,
566
 
        rebase_initial_depths=direction == 'reverse'))
567
 
    if direction == 'reverse':
568
 
        return view_revisions
569
 
    elif direction == 'forward':
570
 
        # Forward means oldest first, adjusting for depth.
571
 
        view_revisions = reverse_by_depth(list(view_revisions))
572
 
        return _rebase_merge_depth(view_revisions)
573
 
    else:
574
 
        raise ValueError('invalid direction %r' % direction)
 
615
                              rebase_initial_depths=(direction == 'reverse'),
 
616
                              exclude_common_ancestry=exclude_common_ancestry))
 
617
    return view_revisions
575
618
 
576
619
 
577
620
def _has_merges(branch, rev_id):
595
638
        else:
596
639
            # not obvious
597
640
            return False
 
641
    # if either start or end is not specified then we use either the first or
 
642
    # the last revision and *they* are obvious ancestors.
598
643
    return True
599
644
 
600
645
 
633
678
 
634
679
 
635
680
def _graph_view_revisions(branch, start_rev_id, end_rev_id,
636
 
    rebase_initial_depths=True):
 
681
                          rebase_initial_depths=True,
 
682
                          exclude_common_ancestry=False):
637
683
    """Calculate revisions to view including merges, newest to oldest.
638
684
 
639
685
    :param branch: the branch
643
689
      revision is found?
644
690
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples.
645
691
    """
 
692
    if exclude_common_ancestry:
 
693
        stop_rule = 'with-merges-without-common-ancestry'
 
694
    else:
 
695
        stop_rule = 'with-merges'
646
696
    view_revisions = branch.iter_merge_sorted_revisions(
647
697
        start_revision_id=end_rev_id, stop_revision_id=start_rev_id,
648
 
        stop_rule="with-merges")
 
698
        stop_rule=stop_rule)
649
699
    if not rebase_initial_depths:
650
700
        for (rev_id, merge_depth, revno, end_of_merge
651
701
             ) in view_revisions:
662
712
                depth_adjustment = merge_depth
663
713
            if depth_adjustment:
664
714
                if merge_depth < depth_adjustment:
 
715
                    # From now on we reduce the depth adjustement, this can be
 
716
                    # surprising for users. The alternative requires two passes
 
717
                    # which breaks the fast display of the first revision
 
718
                    # though.
665
719
                    depth_adjustment = merge_depth
666
720
                merge_depth -= depth_adjustment
667
721
            yield rev_id, '.'.join(map(str, revno)), merge_depth
668
722
 
669
723
 
 
724
@deprecated_function(deprecated_in((2, 2, 0)))
670
725
def calculate_view_revisions(branch, start_revision, end_revision, direction,
671
726
        specific_fileid, generate_merge_revisions):
672
727
    """Calculate the revisions to view.
674
729
    :return: An iterator of (revision_id, dotted_revno, merge_depth) tuples OR
675
730
             a list of the same tuples.
676
731
    """
677
 
    # This method is no longer called by the main code path.
678
 
    # It is retained for API compatibility and may be deprecated
679
 
    # soon. IGC 20090116
680
732
    start_rev_id, end_rev_id = _get_revision_limits(branch, start_revision,
681
733
        end_revision)
682
734
    view_revisions = list(_calc_view_revisions(branch, start_rev_id, end_rev_id,
1032
1084
    return mainline_revs, rev_nos, start_rev_id, end_rev_id
1033
1085
 
1034
1086
 
 
1087
@deprecated_function(deprecated_in((2, 2, 0)))
1035
1088
def _filter_revision_range(view_revisions, start_rev_id, end_rev_id):
1036
1089
    """Filter view_revisions based on revision ranges.
1037
1090
 
1046
1099
 
1047
1100
    :return: The filtered view_revisions.
1048
1101
    """
1049
 
    # This method is no longer called by the main code path.
1050
 
    # It may be removed soon. IGC 20090127
1051
1102
    if start_rev_id or end_rev_id:
1052
1103
        revision_ids = [r for r, n, d in view_revisions]
1053
1104
        if start_rev_id:
1159
1210
    return result
1160
1211
 
1161
1212
 
 
1213
@deprecated_function(deprecated_in((2, 2, 0)))
1162
1214
def get_view_revisions(mainline_revs, rev_nos, branch, direction,
1163
1215
                       include_merges=True):
1164
1216
    """Produce an iterator of revisions to show
1165
1217
    :return: an iterator of (revision_id, revno, merge_depth)
1166
1218
    (if there is no revno for a revision, None is supplied)
1167
1219
    """
1168
 
    # This method is no longer called by the main code path.
1169
 
    # It is retained for API compatibility and may be deprecated
1170
 
    # soon. IGC 20090127
1171
1220
    if not include_merges:
1172
1221
        revision_ids = mainline_revs[1:]
1173
1222
        if direction == 'reverse':
1291
1340
    preferred_levels = 0
1292
1341
 
1293
1342
    def __init__(self, to_file, show_ids=False, show_timezone='original',
1294
 
                 delta_format=None, levels=None, show_advice=False):
 
1343
                 delta_format=None, levels=None, show_advice=False,
 
1344
                 to_exact_file=None):
1295
1345
        """Create a LogFormatter.
1296
1346
 
1297
1347
        :param to_file: the file to output to
 
1348
        :param to_exact_file: if set, gives an output stream to which 
 
1349
             non-Unicode diffs are written.
1298
1350
        :param show_ids: if True, revision-ids are to be displayed
1299
1351
        :param show_timezone: the timezone to use
1300
1352
        :param delta_format: the level of delta information to display
1307
1359
        self.to_file = to_file
1308
1360
        # 'exact' stream used to show diff, it should print content 'as is'
1309
1361
        # and should not try to decode/encode it to unicode to avoid bug #328007
1310
 
        self.to_exact_file = getattr(to_file, 'stream', to_file)
 
1362
        if to_exact_file is not None:
 
1363
            self.to_exact_file = to_exact_file
 
1364
        else:
 
1365
            # XXX: somewhat hacky; this assumes it's a codec writer; it's better
 
1366
            # for code that expects to get diffs to pass in the exact file
 
1367
            # stream
 
1368
            self.to_exact_file = getattr(to_file, 'stream', to_file)
1311
1369
        self.show_ids = show_ids
1312
1370
        self.show_timezone = show_timezone
1313
1371
        if delta_format is None:
1367
1425
        else:
1368
1426
            return ''
1369
1427
 
1370
 
    def show_foreign_info(self, rev, indent):
 
1428
    def show_properties(self, revision, indent):
 
1429
        """Displays the custom properties returned by each registered handler.
 
1430
 
 
1431
        If a registered handler raises an error it is propagated.
 
1432
        """
 
1433
        for line in self.custom_properties(revision):
 
1434
            self.to_file.write("%s%s\n" % (indent, line))
 
1435
 
 
1436
    def custom_properties(self, revision):
 
1437
        """Format the custom properties returned by each registered handler.
 
1438
 
 
1439
        If a registered handler raises an error it is propagated.
 
1440
 
 
1441
        :return: a list of formatted lines (excluding trailing newlines)
 
1442
        """
 
1443
        lines = self._foreign_info_properties(revision)
 
1444
        for key, handler in properties_handler_registry.iteritems():
 
1445
            lines.extend(self._format_properties(handler(revision)))
 
1446
        return lines
 
1447
 
 
1448
    def _foreign_info_properties(self, rev):
1371
1449
        """Custom log displayer for foreign revision identifiers.
1372
1450
 
1373
1451
        :param rev: Revision object.
1374
1452
        """
1375
1453
        # Revision comes directly from a foreign repository
1376
1454
        if isinstance(rev, foreign.ForeignRevision):
1377
 
            self._write_properties(indent, rev.mapping.vcs.show_foreign_revid(
1378
 
                rev.foreign_revid))
1379
 
            return
 
1455
            return self._format_properties(
 
1456
                rev.mapping.vcs.show_foreign_revid(rev.foreign_revid))
1380
1457
 
1381
1458
        # Imported foreign revision revision ids always contain :
1382
1459
        if not ":" in rev.revision_id:
1383
 
            return
 
1460
            return []
1384
1461
 
1385
1462
        # Revision was once imported from a foreign repository
1386
1463
        try:
1387
1464
            foreign_revid, mapping = \
1388
1465
                foreign.foreign_vcs_registry.parse_revision_id(rev.revision_id)
1389
1466
        except errors.InvalidRevisionId:
1390
 
            return
 
1467
            return []
1391
1468
 
1392
 
        self._write_properties(indent, 
 
1469
        return self._format_properties(
1393
1470
            mapping.vcs.show_foreign_revid(foreign_revid))
1394
1471
 
1395
 
    def show_properties(self, revision, indent):
1396
 
        """Displays the custom properties returned by each registered handler.
1397
 
 
1398
 
        If a registered handler raises an error it is propagated.
1399
 
        """
1400
 
        for key, handler in properties_handler_registry.iteritems():
1401
 
            self._write_properties(indent, handler(revision))
1402
 
 
1403
 
    def _write_properties(self, indent, properties):
 
1472
    def _format_properties(self, properties):
 
1473
        lines = []
1404
1474
        for key, value in properties.items():
1405
 
            self.to_file.write(indent + key + ': ' + value + '\n')
 
1475
            lines.append(key + ': ' + value)
 
1476
        return lines
1406
1477
 
1407
1478
    def show_diff(self, to_file, diff, indent):
1408
1479
        for l in diff.rstrip().split('\n'):
1409
1480
            to_file.write(indent + '%s\n' % (l,))
1410
1481
 
1411
1482
 
 
1483
# Separator between revisions in long format
 
1484
_LONG_SEP = '-' * 60
 
1485
 
 
1486
 
1412
1487
class LongLogFormatter(LogFormatter):
1413
1488
 
1414
1489
    supports_merge_revisions = True
1417
1492
    supports_tags = True
1418
1493
    supports_diff = True
1419
1494
 
 
1495
    def __init__(self, *args, **kwargs):
 
1496
        super(LongLogFormatter, self).__init__(*args, **kwargs)
 
1497
        if self.show_timezone == 'original':
 
1498
            self.date_string = self._date_string_original_timezone
 
1499
        else:
 
1500
            self.date_string = self._date_string_with_timezone
 
1501
 
 
1502
    def _date_string_with_timezone(self, rev):
 
1503
        return format_date(rev.timestamp, rev.timezone or 0,
 
1504
                           self.show_timezone)
 
1505
 
 
1506
    def _date_string_original_timezone(self, rev):
 
1507
        return format_date_with_offset_in_original_timezone(rev.timestamp,
 
1508
            rev.timezone or 0)
 
1509
 
1420
1510
    def log_revision(self, revision):
1421
1511
        """Log a revision, either merged or not."""
1422
1512
        indent = '    ' * revision.merge_depth
1423
 
        to_file = self.to_file
1424
 
        to_file.write(indent + '-' * 60 + '\n')
 
1513
        lines = [_LONG_SEP]
1425
1514
        if revision.revno is not None:
1426
 
            to_file.write(indent + 'revno: %s%s\n' % (revision.revno,
 
1515
            lines.append('revno: %s%s' % (revision.revno,
1427
1516
                self.merge_marker(revision)))
1428
1517
        if revision.tags:
1429
 
            to_file.write(indent + 'tags: %s\n' % (', '.join(revision.tags)))
 
1518
            lines.append('tags: %s' % (', '.join(revision.tags)))
1430
1519
        if self.show_ids:
1431
 
            to_file.write(indent + 'revision-id: ' + revision.rev.revision_id)
1432
 
            to_file.write('\n')
 
1520
            lines.append('revision-id: %s' % (revision.rev.revision_id,))
1433
1521
            for parent_id in revision.rev.parent_ids:
1434
 
                to_file.write(indent + 'parent: %s\n' % (parent_id,))
1435
 
        self.show_foreign_info(revision.rev, indent)
1436
 
        self.show_properties(revision.rev, indent)
 
1522
                lines.append('parent: %s' % (parent_id,))
 
1523
        lines.extend(self.custom_properties(revision.rev))
1437
1524
 
1438
1525
        committer = revision.rev.committer
1439
1526
        authors = revision.rev.get_apparent_authors()
1440
1527
        if authors != [committer]:
1441
 
            to_file.write(indent + 'author: %s\n' % (", ".join(authors),))
1442
 
        to_file.write(indent + 'committer: %s\n' % (committer,))
 
1528
            lines.append('author: %s' % (", ".join(authors),))
 
1529
        lines.append('committer: %s' % (committer,))
1443
1530
 
1444
1531
        branch_nick = revision.rev.properties.get('branch-nick', None)
1445
1532
        if branch_nick is not None:
1446
 
            to_file.write(indent + 'branch nick: %s\n' % (branch_nick,))
1447
 
 
1448
 
        date_str = format_date(revision.rev.timestamp,
1449
 
                               revision.rev.timezone or 0,
1450
 
                               self.show_timezone)
1451
 
        to_file.write(indent + 'timestamp: %s\n' % (date_str,))
1452
 
 
1453
 
        to_file.write(indent + 'message:\n')
 
1533
            lines.append('branch nick: %s' % (branch_nick,))
 
1534
 
 
1535
        lines.append('timestamp: %s' % (self.date_string(revision.rev),))
 
1536
 
 
1537
        lines.append('message:')
1454
1538
        if not revision.rev.message:
1455
 
            to_file.write(indent + '  (no message)\n')
 
1539
            lines.append('  (no message)')
1456
1540
        else:
1457
1541
            message = revision.rev.message.rstrip('\r\n')
1458
1542
            for l in message.split('\n'):
1459
 
                to_file.write(indent + '  %s\n' % (l,))
 
1543
                lines.append('  %s' % (l,))
 
1544
 
 
1545
        # Dump the output, appending the delta and diff if requested
 
1546
        to_file = self.to_file
 
1547
        to_file.write("%s%s\n" % (indent, ('\n' + indent).join(lines)))
1460
1548
        if revision.delta is not None:
1461
 
            # We don't respect delta_format for compatibility
1462
 
            revision.delta.show(to_file, self.show_ids, indent=indent,
1463
 
                                short_status=False)
 
1549
            # Use the standard status output to display changes
 
1550
            from bzrlib.delta import report_delta
 
1551
            report_delta(to_file, revision.delta, short_status=False, 
 
1552
                         show_ids=self.show_ids, indent=indent)
1464
1553
        if revision.diff is not None:
1465
1554
            to_file.write(indent + 'diff:\n')
 
1555
            to_file.flush()
1466
1556
            # Note: we explicitly don't indent the diff (relative to the
1467
1557
            # revision information) so that the output can be fed to patch -p0
1468
1558
            self.show_diff(self.to_exact_file, revision.diff, indent)
 
1559
            self.to_exact_file.flush()
1469
1560
 
1470
1561
    def get_advice_separator(self):
1471
1562
        """Get the text separating the log from the closing advice."""
1515
1606
                            self.show_timezone, date_fmt="%Y-%m-%d",
1516
1607
                            show_offset=False),
1517
1608
                tags, self.merge_marker(revision)))
1518
 
        self.show_foreign_info(revision.rev, indent+offset)
1519
1609
        self.show_properties(revision.rev, indent+offset)
1520
1610
        if self.show_ids:
1521
1611
            to_file.write(indent + offset + 'revision-id:%s\n'
1528
1618
                to_file.write(indent + offset + '%s\n' % (l,))
1529
1619
 
1530
1620
        if revision.delta is not None:
1531
 
            revision.delta.show(to_file, self.show_ids, indent=indent + offset,
1532
 
                                short_status=self.delta_format==1)
 
1621
            # Use the standard status output to display changes
 
1622
            from bzrlib.delta import report_delta
 
1623
            report_delta(to_file, revision.delta, 
 
1624
                         short_status=self.delta_format==1, 
 
1625
                         show_ids=self.show_ids, indent=indent + offset)
1533
1626
        if revision.diff is not None:
1534
1627
            self.show_diff(self.to_exact_file, revision.diff, '      ')
1535
1628
        to_file.write('\n')
1543
1636
 
1544
1637
    def __init__(self, *args, **kwargs):
1545
1638
        super(LineLogFormatter, self).__init__(*args, **kwargs)
1546
 
        self._max_chars = terminal_width() - 1
 
1639
        width = terminal_width()
 
1640
        if width is not None:
 
1641
            # we need one extra space for terminals that wrap on last char
 
1642
            width = width - 1
 
1643
        self._max_chars = width
1547
1644
 
1548
1645
    def truncate(self, str, max_len):
1549
 
        if len(str) <= max_len:
 
1646
        if max_len is None or len(str) <= max_len:
1550
1647
            return str
1551
 
        return str[:max_len-3]+'...'
 
1648
        return str[:max_len-3] + '...'
1552
1649
 
1553
1650
    def date_string(self, rev):
1554
1651
        return format_date(rev.timestamp, rev.timezone or 0,
1606
1703
                               self.show_timezone,
1607
1704
                               date_fmt='%Y-%m-%d',
1608
1705
                               show_offset=False)
1609
 
        committer_str = revision.rev.committer.replace (' <', '  <')
 
1706
        committer_str = revision.rev.get_apparent_authors()[0].replace (' <', '  <')
1610
1707
        to_file.write('%s  %s\n\n' % (date_str,committer_str))
1611
1708
 
1612
1709
        if revision.delta is not None and revision.delta.has_changed():
1935
2032
 
1936
2033
properties_handler_registry = registry.Registry()
1937
2034
 
 
2035
# Use the properties handlers to print out bug information if available
 
2036
def _bugs_properties_handler(revision):
 
2037
    if revision.properties.has_key('bugs'):
 
2038
        bug_lines = revision.properties['bugs'].split('\n')
 
2039
        bug_rows = [line.split(' ', 1) for line in bug_lines]
 
2040
        fixed_bug_urls = [row[0] for row in bug_rows if
 
2041
                          len(row) > 1 and row[1] == 'fixed']
 
2042
 
 
2043
        if fixed_bug_urls:
 
2044
            return {'fixes bug(s)': ' '.join(fixed_bug_urls)}
 
2045
    return {}
 
2046
 
 
2047
properties_handler_registry.register('bugs_properties_handler',
 
2048
                                     _bugs_properties_handler)
 
2049
 
1938
2050
 
1939
2051
# adapters which revision ids to log are filtered. When log is called, the
1940
2052
# log_rev_iterator is adapted through each of these factory methods.