/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 apport/source_bzr.py

  • Committer: John Arbash Meinel
  • Date: 2009-06-18 18:18:36 UTC
  • mto: This revision was merged to the branch mainline in revision 4461.
  • Revision ID: john@arbash-meinel.com-20090618181836-biodfkat9a8eyzjz
The new add_inventory_by_delta is returning a CHKInventory when mapping from NULL
Which is completely valid, but 'broke' one of the tests.
So to fix it, changed the test to use CHKInventories on both sides, and add an __eq__
member. The nice thing is that CHKInventory.__eq__ is fairly cheap, since it only
has to check the root keys.

Show diffs side-by-side

added added

removed removed

Lines of Context:
apport/source_bzr.py
1
 
'''apport package hook for Bazaar'''
2
 
 
3
 
# Copyright (c) 2009, 2010 Canonical Ltd.
4
 
# Author: Matt Zimmerman <mdz@canonical.com>
5
 
#         and others
6
 
 
7
 
from apport.hookutils import *
8
 
import os
9
 
 
10
 
bzr_log = os.path.expanduser('~/.bzr.log')
11
 
dot_bzr = os.path.expanduser('~/.bazaar')
12
 
 
13
 
def _add_log_tail(report):
14
 
    # may have already been added in-process
15
 
    if 'BzrLogTail' in report:
16
 
        return
17
 
 
18
 
    bzr_log_lines = open(bzr_log).readlines()
19
 
    bzr_log_lines.reverse()
20
 
 
21
 
    bzr_log_tail = []
22
 
    blanks = 0
23
 
    for line in bzr_log_lines:
24
 
        if line == '\n':
25
 
            blanks += 1
26
 
        bzr_log_tail.append(line)
27
 
        if blanks >= 2: 
28
 
            break
29
 
 
30
 
    bzr_log_tail.reverse()
31
 
    report['BzrLogTail'] = ''.join(bzr_log_tail)
32
 
 
33
 
 
34
 
def add_info(report):
35
 
    _add_log_tail(report)
36
 
    if 'BzrPlugins' not in report:
37
 
        # may already be present in-process
38
 
        report['BzrPlugins'] = command_output(['bzr', 'plugins', '-v'])
39
 
        
40
 
    # by default assume bzr crashes are upstream bugs; this relies on
41
 
    # having a bzr entry under /etc/apport/crashdb.conf.d/
42
 
    report['CrashDB'] = 'bzr'
43
 
 
44
 
    # these may contain some sensitive info (smtp_passwords)
45
 
    # TODO: strip that out and attach the rest
46
 
 
47
 
    #attach_file_if_exists(report,
48
 
    #   os.path.join(dot_bzr, 'bazaar.conf', 'BzrConfig')
49
 
    #attach_file_if_exists(report,
50
 
    #   os.path.join(dot_bzr, 'locations.conf', 'BzrLocations')
51
 
 
52
 
        
53
 
# vim: expandtab shiftwidth=4