~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/bugtracker.py

  • Committer: John Arbash Meinel
  • Date: 2013-05-19 14:29:37 UTC
  • mfrom: (6437.63.9 2.5)
  • mto: (6437.63.10 2.5)
  • mto: This revision was merged to the branch mainline in revision 6575.
  • Revision ID: john@arbash-meinel.com-20130519142937-21ykz2n2y2f22za9
Merge in the actual 2.5 branch. It seems I failed before

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2007-2010 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
from __future__ import absolute_import
 
18
 
 
19
from bzrlib import registry
 
20
from bzrlib.lazy_import import lazy_import
 
21
lazy_import(globals(), """
 
22
from bzrlib import errors, urlutils
 
23
""")
 
24
 
 
25
 
 
26
"""Provides a shorthand for referring to bugs on a variety of bug trackers.
 
27
 
 
28
'commit --fixes' stores references to bugs as a <bug_url> -> <bug_status>
 
29
mapping in the properties for that revision.
 
30
 
 
31
However, it's inconvenient to type out full URLs for bugs on the command line,
 
32
particularly given that many users will be using only a single bug tracker per
 
33
branch.
 
34
 
 
35
Thus, this module provides a registry of types of bug tracker (e.g. Launchpad,
 
36
Trac). Given an abbreviated name (e.g. 'lp', 'twisted') and a branch with
 
37
configuration information, these tracker types can return an instance capable
 
38
of converting bug IDs into URLs.
 
39
"""
 
40
 
 
41
 
 
42
_bugs_help = \
 
43
"""When making a commit, metadata about bugs fixed by that change can be
 
44
recorded by using the ``--fixes`` option. For each bug marked as fixed, an
 
45
entry is included in the 'bugs' revision property stating '<url> <status>'.
 
46
(The only ``status`` value currently supported is ``fixed.``)
 
47
 
 
48
The ``--fixes`` option allows you to specify a bug tracker and a bug identifier
 
49
rather than a full URL. This looks like::
 
50
 
 
51
    bzr commit --fixes <tracker>:<id>
 
52
 
 
53
or::
 
54
 
 
55
    bzr commit --fixes <id>
 
56
 
 
57
where "<tracker>" is an identifier for the bug tracker, and "<id>" is the
 
58
identifier for that bug within the bugtracker, usually the bug number.
 
59
If "<tracker>" is not specified the ``bugtracker`` set in the branch
 
60
or global configuration is used.
 
61
 
 
62
Bazaar knows about a few bug trackers that have many users. If
 
63
you use one of these bug trackers then there is no setup required to
 
64
use this feature, you just need to know the tracker identifier to use.
 
65
These are the bugtrackers that are built in:
 
66
 
 
67
  ============================ ============ ============
 
68
  URL                          Abbreviation Example
 
69
  ============================ ============ ============
 
70
  https://bugs.launchpad.net/  lp           lp:12345
 
71
  http://bugs.debian.org/      deb          deb:12345
 
72
  http://bugzilla.gnome.org/   gnome        gnome:12345
 
73
  ============================ ============ ============
 
74
 
 
75
For the bug trackers not listed above configuration is required.
 
76
Support for generating the URLs for any project using Bugzilla or Trac
 
77
is built in, along with a template mechanism for other bugtrackers with
 
78
simple URL schemes. If your bug tracker can't be described by one
 
79
of the schemes described below then you can write a plugin to support
 
80
it.
 
81
 
 
82
If you use Bugzilla or Trac, then you only need to set a configuration
 
83
variable which contains the base URL of the bug tracker. These options
 
84
can go into ``bazaar.conf``, ``branch.conf`` or into a branch-specific
 
85
configuration section in ``locations.conf``.  You can set up these values
 
86
for each of the projects you work on.
 
87
 
 
88
Note: As you provide a short name for each tracker, you can specify one or
 
89
more bugs in one or more trackers at commit time if you wish.
 
90
 
 
91
Launchpad
 
92
---------
 
93
 
 
94
Use ``bzr commit --fixes lp:2`` to record that this commit fixes bug 2.
 
95
 
 
96
bugzilla_<tracker>_url
 
97
----------------------
 
98
 
 
99
If present, the location of the Bugzilla bug tracker referred to by
 
100
<tracker>. This option can then be used together with ``bzr commit
 
101
--fixes`` to mark bugs in that tracker as being fixed by that commit. For
 
102
example::
 
103
 
 
104
    bugzilla_squid_url = http://bugs.squid-cache.org
 
105
 
 
106
would allow ``bzr commit --fixes squid:1234`` to mark Squid's bug 1234 as
 
107
fixed.
 
108
 
 
109
trac_<tracker>_url
 
110
------------------
 
111
 
 
112
If present, the location of the Trac instance referred to by
 
113
<tracker>. This option can then be used together with ``bzr commit
 
114
--fixes`` to mark bugs in that tracker as being fixed by that commit. For
 
115
example::
 
116
 
 
117
    trac_twisted_url = http://www.twistedmatrix.com/trac
 
118
 
 
119
would allow ``bzr commit --fixes twisted:1234`` to mark Twisted's bug 1234 as
 
120
fixed.
 
121
 
 
122
bugtracker_<tracker>_url
 
123
------------------------
 
124
 
 
125
If present, the location of a generic bug tracker instance referred to by
 
126
<tracker>. The location must contain an ``{id}`` placeholder,
 
127
which will be replaced by a specific bug ID. This option can then be used
 
128
together with ``bzr commit --fixes`` to mark bugs in that tracker as being
 
129
fixed by that commit. For example::
 
130
 
 
131
    bugtracker_python_url = http://bugs.python.org/issue{id}
 
132
 
 
133
would allow ``bzr commit --fixes python:1234`` to mark bug 1234 in Python's
 
134
Roundup bug tracker as fixed, or::
 
135
 
 
136
    bugtracker_cpan_url = http://rt.cpan.org/Public/Bug/Display.html?id={id}
 
137
 
 
138
would allow ``bzr commit --fixes cpan:1234`` to mark bug 1234 in CPAN's
 
139
RT bug tracker as fixed, or::
 
140
 
 
141
    bugtracker_hudson_url = http://issues.hudson-ci.org/browse/{id}
 
142
 
 
143
would allow ``bzr commit --fixes hudson:HUDSON-1234`` to mark bug HUDSON-1234
 
144
in Hudson's JIRA bug tracker as fixed.
 
145
"""
 
146
 
 
147
 
 
148
def get_bug_url(abbreviated_bugtracker_name, branch, bug_id):
 
149
    """Return a URL pointing to the canonical web page of the bug identified by
 
150
    'bug_id'.
 
151
    """
 
152
    tracker = tracker_registry.get_tracker(abbreviated_bugtracker_name, branch)
 
153
    return tracker.get_bug_url(bug_id)
 
154
 
 
155
 
 
156
class TrackerRegistry(registry.Registry):
 
157
    """Registry of bug tracker types."""
 
158
 
 
159
    def get_tracker(self, abbreviated_bugtracker_name, branch):
 
160
        """Return the first registered tracker that understands
 
161
        'abbreviated_bugtracker_name'.
 
162
 
 
163
        If no such tracker is found, raise KeyError.
 
164
        """
 
165
        for tracker_name in self.keys():
 
166
            tracker_type = self.get(tracker_name)
 
167
            tracker = tracker_type.get(abbreviated_bugtracker_name, branch)
 
168
            if tracker is not None:
 
169
                return tracker
 
170
        raise errors.UnknownBugTrackerAbbreviation(abbreviated_bugtracker_name,
 
171
                                                   branch)
 
172
 
 
173
    def help_topic(self, topic):
 
174
        return _bugs_help
 
175
 
 
176
 
 
177
tracker_registry = TrackerRegistry()
 
178
"""Registry of bug trackers."""
 
179
 
 
180
 
 
181
class BugTracker(object):
 
182
    """Base class for bug trackers."""
 
183
 
 
184
    def check_bug_id(self, bug_id):
 
185
        """Check that the bug_id is valid.
 
186
 
 
187
        The base implementation assumes that all bug_ids are valid.
 
188
        """
 
189
 
 
190
    def get_bug_url(self, bug_id):
 
191
        """Return the URL for bug_id. Raise an error if bug ID is malformed."""
 
192
        self.check_bug_id(bug_id)
 
193
        return self._get_bug_url(bug_id)
 
194
 
 
195
    def _get_bug_url(self, bug_id):
 
196
        """Given a validated bug_id, return the bug's web page's URL."""
 
197
 
 
198
 
 
199
class IntegerBugTracker(BugTracker):
 
200
    """A bug tracker that only allows integer bug IDs."""
 
201
 
 
202
    def check_bug_id(self, bug_id):
 
203
        try:
 
204
            int(bug_id)
 
205
        except ValueError:
 
206
            raise errors.MalformedBugIdentifier(bug_id, "Must be an integer")
 
207
 
 
208
 
 
209
class UniqueIntegerBugTracker(IntegerBugTracker):
 
210
    """A style of bug tracker that exists in one place only, such as Launchpad.
 
211
 
 
212
    If you have one of these trackers then register an instance passing in an
 
213
    abbreviated name for the bug tracker and a base URL. The bug ids are
 
214
    appended directly to the URL.
 
215
    """
 
216
 
 
217
    def __init__(self, abbreviated_bugtracker_name, base_url):
 
218
        self.abbreviation = abbreviated_bugtracker_name
 
219
        self.base_url = base_url
 
220
 
 
221
    def get(self, abbreviated_bugtracker_name, branch):
 
222
        """Returns the tracker if the abbreviation matches. Returns None
 
223
        otherwise."""
 
224
        if abbreviated_bugtracker_name != self.abbreviation:
 
225
            return None
 
226
        return self
 
227
 
 
228
    def _get_bug_url(self, bug_id):
 
229
        """Return the URL for bug_id."""
 
230
        return self.base_url + bug_id
 
231
 
 
232
 
 
233
tracker_registry.register(
 
234
    'launchpad', UniqueIntegerBugTracker('lp', 'https://launchpad.net/bugs/'))
 
235
 
 
236
 
 
237
tracker_registry.register(
 
238
    'debian', UniqueIntegerBugTracker('deb', 'http://bugs.debian.org/'))
 
239
 
 
240
 
 
241
tracker_registry.register('gnome',
 
242
    UniqueIntegerBugTracker('gnome',
 
243
                            'http://bugzilla.gnome.org/show_bug.cgi?id='))
 
244
 
 
245
 
 
246
class URLParametrizedBugTracker(BugTracker):
 
247
    """A type of bug tracker that can be found on a variety of different sites,
 
248
    and thus needs to have the base URL configured.
 
249
 
 
250
    Looks for a config setting in the form '<type_name>_<abbreviation>_url'.
 
251
    `type_name` is the name of the type of tracker and `abbreviation`
 
252
    is a short name for the particular instance.
 
253
    """
 
254
 
 
255
    def get(self, abbreviation, branch):
 
256
        config = branch.get_config()
 
257
        url = config.get_user_option(
 
258
            "%s_%s_url" % (self.type_name, abbreviation), expand=False)
 
259
        if url is None:
 
260
            return None
 
261
        self._base_url = url
 
262
        return self
 
263
 
 
264
    def __init__(self, type_name, bug_area):
 
265
        self.type_name = type_name
 
266
        self._bug_area = bug_area
 
267
 
 
268
    def _get_bug_url(self, bug_id):
 
269
        """Return a URL for a bug on this Trac instance."""
 
270
        return urlutils.join(self._base_url, self._bug_area) + str(bug_id)
 
271
 
 
272
 
 
273
class URLParametrizedIntegerBugTracker(IntegerBugTracker,
 
274
                                       URLParametrizedBugTracker):
 
275
    """A type of bug tracker that  only allows integer bug IDs.
 
276
 
 
277
    This can be found on a variety of different sites, and thus needs to have
 
278
    the base URL configured.
 
279
 
 
280
    Looks for a config setting in the form '<type_name>_<abbreviation>_url'.
 
281
    `type_name` is the name of the type of tracker (e.g. 'bugzilla' or 'trac')
 
282
    and `abbreviation` is a short name for the particular instance (e.g.
 
283
    'squid' or 'apache').
 
284
    """
 
285
 
 
286
tracker_registry.register(
 
287
    'trac', URLParametrizedIntegerBugTracker('trac', 'ticket/'))
 
288
 
 
289
tracker_registry.register(
 
290
    'bugzilla',
 
291
    URLParametrizedIntegerBugTracker('bugzilla', 'show_bug.cgi?id='))
 
292
 
 
293
 
 
294
class GenericBugTracker(URLParametrizedBugTracker):
 
295
    """Generic bug tracker specified by an URL template."""
 
296
 
 
297
    def __init__(self):
 
298
        super(GenericBugTracker, self).__init__('bugtracker', None)
 
299
 
 
300
    def get(self, abbreviation, branch):
 
301
        self._abbreviation = abbreviation
 
302
        return super(GenericBugTracker, self).get(abbreviation, branch)
 
303
 
 
304
    def _get_bug_url(self, bug_id):
 
305
        """Given a validated bug_id, return the bug's web page's URL."""
 
306
        if '{id}' not in self._base_url:
 
307
            raise errors.InvalidBugTrackerURL(self._abbreviation,
 
308
                                              self._base_url)
 
309
        return self._base_url.replace('{id}', str(bug_id))
 
310
 
 
311
 
 
312
tracker_registry.register('generic', GenericBugTracker())
 
313
 
 
314
 
 
315
FIXED = 'fixed'
 
316
 
 
317
ALLOWED_BUG_STATUSES = set([FIXED])
 
318
 
 
319
 
 
320
def encode_fixes_bug_urls(bug_urls):
 
321
    """Get the revision property value for a commit that fixes bugs.
 
322
 
 
323
    :param bug_urls: An iterable of escaped URLs to bugs. These normally
 
324
        come from `get_bug_url`.
 
325
    :return: A string that will be set as the 'bugs' property of a revision
 
326
        as part of a commit.
 
327
    """
 
328
    return '\n'.join(('%s %s' % (url, FIXED)) for url in bug_urls)