/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 tools/check-newsbugs.py

  • Committer: Robert Collins
  • Date: 2009-09-01 23:43:40 UTC
  • mto: (4634.6.14 2.0)
  • mto: This revision was merged to the branch mainline in revision 4666.
  • Revision ID: robertc@robertcollins.net-20090901234340-vc52aosyg23iaotw
revno 4639 in bzr.dev introduced a bug in the conversion logic for 'IDS'.

This bug is present in the InterDifferingSerializer code path, which is
used for local conversions (e.g. bzr upgrade on local disk, or bzr
push/pull between different formats, on local disks).

This revision is present in the 2.0rc1 release.

The bug is simple: A variable used to inform the repository where a
delta should be applied from was aliased in a for loop, leaving its
value indeterminate.

In *some* cases this will cause later consistency checks to error, but
not in all cases, as we don't do a manifest check during conversion.

No commits as yet, but getting this out is a priority.

Show diffs side-by-side

added added

removed removed

Lines of Context:
tools/check-newsbugs.py
 
1
#!/usr/bin/python
 
2
# Simple script that will check which bugs mentioned in NEWS 
 
3
# are not yet marked Fix Released in Launchpad
 
4
 
 
5
import getopt, re, sys
 
6
try:
 
7
    from launchpadbugs import connector
 
8
except ImportError:
 
9
    print "Please install launchpadbugs from lp:python-launchpad-bugs"
 
10
    sys.exit(1)
 
11
 
 
12
options, args = getopt.gnu_getopt(sys.argv, "l", ["launchpad"])
 
13
options = dict(options)
 
14
 
 
15
if len(args) == 1:
 
16
    print "Usage: check-newsbugs [--launchpad] NEWS"
 
17
    print "Options:"
 
18
    print "--launchpad     Print out Launchpad mail commands for closing bugs "
 
19
    print "                that are already fixed."
 
20
    sys.exit(1)
 
21
 
 
22
 
 
23
def report_notmarked(bug, task, section):
 
24
    print 
 
25
    print "Bug %d was mentioned in NEWS but is not marked fix released:" % (bug.bugnumber, )
 
26
    print "Launchpad title: %s" % bug.title
 
27
    print "NEWS summary: "
 
28
    print section
 
29
    if "--launchpad" in options or "-l" in options:
 
30
        print "  bug %d" % bug.bugnumber
 
31
        print "  affects bzr"
 
32
        print "  status fixreleased"
 
33
 
 
34
 
 
35
def read_news_bugnos(path):
 
36
    """Read the bug numbers closed by a particular NEWS file
 
37
 
 
38
    :param path: Path to the NEWS file
 
39
    :return: list of bug numbers that were closed.
 
40
    """
 
41
    # Pattern to find bug numbers
 
42
    bug_pattern = re.compile("\#([0-9]+)")
 
43
    ret = set()
 
44
    f = open(path, 'r')
 
45
    try:
 
46
        section = ""
 
47
        for l in f.readlines():
 
48
            if l.strip() == "":
 
49
                try:
 
50
                    parenthesed = section.rsplit("(", 1)[1]
 
51
                except IndexError:
 
52
                    parenthesed = ""
 
53
                # Empty line, next section begins
 
54
                for bugno in [int(m) for m in bug_pattern.findall(parenthesed)]:
 
55
                    ret.add((bugno, section))
 
56
                section = ""
 
57
            else:
 
58
                section += l
 
59
        return ret
 
60
    finally:
 
61
        f.close()
 
62
 
 
63
open_bug = connector.ConnectBug("TEXT")
 
64
 
 
65
bugnos = read_news_bugnos(args[1])
 
66
for bugno, section in bugnos:
 
67
    bug = open_bug(url="https://bugs.launchpad.net/bzr/+bug/%d" % bugno)
 
68
    found_bzr = False
 
69
    for task in bug.infotable:
 
70
        if task.affects == "bzr":
 
71
            found_bzr = True
 
72
            if task.status != "Fix Released":
 
73
                report_notmarked(bug, task, section)
 
74
    if not found_bzr:
 
75
        print "Bug %d was mentioned in NEWS but is not marked as affecting bzr" % bugno