2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
1 |
# Copyright (C) 2007 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., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
|
|
16 |
||
2376.4.36
by Jonathan Lange
Provide really basic help topic for our bug tracker support. |
17 |
import textwrap |
2376.4.6
by jml at canonical
Basic docstrings for bugtracker.py |
18 |
|
2376.4.36
by Jonathan Lange
Provide really basic help topic for our bug tracker support. |
19 |
from bzrlib import registry, help_topics |
2376.4.6
by jml at canonical
Basic docstrings for bugtracker.py |
20 |
from bzrlib.lazy_import import lazy_import |
21 |
lazy_import(globals(), """ |
|
2376.4.17
by Jonathan Lange
Use urlutils.join instead of ugly, buggy, string mashing |
22 |
from bzrlib import errors, urlutils
|
2376.4.6
by jml at canonical
Basic docstrings for bugtracker.py |
23 |
""") |
2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
24 |
|
25 |
||
2376.4.7
by jml at canonical
- Add docstrings to tests. |
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,
|
|
2376.4.23
by Jonathan Lange
Change 'tag' to 'abbreviated_tracker_name' |
36 |
Trac). Given an abbreviated name (e.g. 'lp', 'twisted') and a branch with
|
2376.4.7
by jml at canonical
- Add docstrings to tests. |
37 |
configuration information, these tracker types can return an instance capable
|
38 |
of converting bug IDs into URLs.
|
|
39 |
"""
|
|
40 |
||
41 |
||
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
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>'.
|
|
3154.1.1
by Ian Clatworthy
Minor tweaks to bug tracker integration documentation |
46 |
(The only ``status`` value currently supported is ``fixed.``)
|
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
47 |
Support for Launchpad's central bug tracker is built in. For other bug
|
48 |
trackers, configuration is required in advance so that the correct URL
|
|
49 |
can be recorded.
|
|
50 |
||
51 |
In addition to Launchpad, Bazaar directly supports the generation of
|
|
52 |
URLs appropriate for Bugzilla and Trac. If your project uses a different
|
|
3154.1.1
by Ian Clatworthy
Minor tweaks to bug tracker integration documentation |
53 |
bug tracker, it is easy to add support for it.
|
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
54 |
If you use Bugzilla or Trac, then you only need to set a configuration
|
55 |
variable which contains the base URL of the bug tracker. These options
|
|
56 |
can go into ``bazaar.conf``, ``branch.conf`` or into a branch-specific
|
|
57 |
configuration section in ``locations.conf``. You can set up these values
|
|
58 |
for each of the projects you work on.
|
|
59 |
||
60 |
Note: As you provide a short name for each tracker, you can specify one or
|
|
61 |
more bugs in one or more trackers at commit time if you wish.
|
|
62 |
||
63 |
bugzilla_<tracker_abbreviation>_url
|
|
3089.3.14
by Ian Clatworthy
follow-up tweaks to bzr.dev integration |
64 |
-----------------------------------
|
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
65 |
|
66 |
If present, the location of the Bugzilla bug tracker referred to by
|
|
67 |
<tracker_abbreviation>. This option can then be used together with ``bzr commit
|
|
68 |
--fixes`` to mark bugs in that tracker as being fixed by that commit. For
|
|
69 |
example::
|
|
70 |
||
71 |
bugzilla_squid_url = http://www.squid-cache.org/bugs
|
|
72 |
||
73 |
would allow ``bzr commit --fixes squid:1234`` to mark Squid's bug 1234 as
|
|
74 |
fixed.
|
|
75 |
||
76 |
trac_<tracker_abbrevation>_url
|
|
3089.3.14
by Ian Clatworthy
follow-up tweaks to bzr.dev integration |
77 |
------------------------------
|
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
78 |
|
79 |
If present, the location of the Trac instance referred to by
|
|
80 |
<tracker_abbreviation>. This option can then be used together with ``bzr commit
|
|
81 |
--fixes`` to mark bugs in that tracker as being fixed by that commit. For
|
|
82 |
example::
|
|
83 |
||
84 |
trac_twisted_url = http://www.twistedmatrix.com/trac
|
|
85 |
||
86 |
would allow ``bzr commit --fixes twisted:1234`` to mark Twisted's bug 1234 as
|
|
87 |
fixed.
|
|
88 |
||
89 |
bugtracker_<tracker_abbrevation>_url
|
|
3089.3.14
by Ian Clatworthy
follow-up tweaks to bzr.dev integration |
90 |
------------------------------------
|
91 |
||
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
92 |
If present, the location of a generic bug tracker instance referred to by
|
93 |
<tracker_abbreviation>. The location must contain an ``{id}`` placeholder,
|
|
94 |
which will be replaced by a specific bug ID. This option can then be used
|
|
95 |
together with ``bzr commit --fixes`` to mark bugs in that tracker as being
|
|
96 |
fixed by that commit. For example::
|
|
97 |
||
98 |
bugtracker_python_url = http://bugs.python.org/issue{id}
|
|
99 |
||
100 |
would allow ``bzr commit --fixes python:1234`` to mark bug 1234 in Python's
|
|
101 |
Roundup bug tracker as fixed, or::
|
|
102 |
||
103 |
bugtracker_cpan_url = http://rt.cpan.org/Public/Bug/Display.html?id={id}
|
|
104 |
||
105 |
for CPAN's RT bug tracker.
|
|
106 |
"""
|
|
107 |
||
108 |
||
2376.4.23
by Jonathan Lange
Change 'tag' to 'abbreviated_tracker_name' |
109 |
def get_bug_url(abbreviated_bugtracker_name, branch, bug_id): |
110 |
"""Return a URL pointing to the canonical web page of the bug identified by
|
|
111 |
'bug_id'.
|
|
112 |
"""
|
|
113 |
tracker = tracker_registry.get_tracker(abbreviated_bugtracker_name, branch) |
|
114 |
return tracker.get_bug_url(bug_id) |
|
2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
115 |
|
116 |
||
117 |
class TrackerRegistry(registry.Registry): |
|
118 |
"""Registry of bug tracker types."""
|
|
119 |
||
2376.4.23
by Jonathan Lange
Change 'tag' to 'abbreviated_tracker_name' |
120 |
def get_tracker(self, abbreviated_bugtracker_name, branch): |
121 |
"""Return the first registered tracker that understands
|
|
122 |
'abbreviated_bugtracker_name'.
|
|
2376.4.7
by jml at canonical
- Add docstrings to tests. |
123 |
|
124 |
If no such tracker is found, raise KeyError.
|
|
125 |
"""
|
|
2376.4.23
by Jonathan Lange
Change 'tag' to 'abbreviated_tracker_name' |
126 |
for tracker_name in self.keys(): |
127 |
tracker_type = self.get(tracker_name) |
|
128 |
tracker = tracker_type.get(abbreviated_bugtracker_name, branch) |
|
2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
129 |
if tracker is not None: |
130 |
return tracker |
|
2376.4.28
by Jonathan Lange
Focus the tests better and clean up some dodgy bits in UnknownBugTrackerAbbreviation |
131 |
raise errors.UnknownBugTrackerAbbreviation(abbreviated_bugtracker_name, |
132 |
branch) |
|
2376.4.7
by jml at canonical
- Add docstrings to tests. |
133 |
|
2376.4.36
by Jonathan Lange
Provide really basic help topic for our bug tracker support. |
134 |
def help_topic(self, topic): |
3053.8.1
by Ian Clatworthy
Improve bug tracker integration documentation (Ian Clatworthy) |
135 |
return _bugs_help |
2376.4.36
by Jonathan Lange
Provide really basic help topic for our bug tracker support. |
136 |
|
2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
137 |
|
138 |
tracker_registry = TrackerRegistry() |
|
2376.4.6
by jml at canonical
Basic docstrings for bugtracker.py |
139 |
"""Registry of bug trackers."""
|
2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
140 |
|
2376.4.15
by Jonathan Lange
Whitespace cleanup |
141 |
|
2376.4.40
by Jonathan Lange
Redo the hierarchy of bug trackers to reduce duplication. |
142 |
class BugTracker(object): |
143 |
"""Base class for bug trackers."""
|
|
144 |
||
145 |
def check_bug_id(self, bug_id): |
|
146 |
"""Check that the bug_id is valid.
|
|
147 |
||
148 |
The base implementation assumes that all bug_ids are valid.
|
|
149 |
"""
|
|
150 |
||
151 |
def get_bug_url(self, bug_id): |
|
152 |
"""Return the URL for bug_id. Raise an error if bug ID is malformed."""
|
|
153 |
self.check_bug_id(bug_id) |
|
154 |
return self._get_bug_url(bug_id) |
|
155 |
||
156 |
def _get_bug_url(self, bug_id): |
|
157 |
"""Given a validated bug_id, return the bug's web page's URL."""
|
|
158 |
||
159 |
||
160 |
class IntegerBugTracker(BugTracker): |
|
161 |
"""A bug tracker that only allows integer bug IDs."""
|
|
162 |
||
163 |
def check_bug_id(self, bug_id): |
|
164 |
try: |
|
165 |
int(bug_id) |
|
166 |
except ValueError: |
|
167 |
raise errors.MalformedBugIdentifier(bug_id, "Must be an integer") |
|
168 |
||
169 |
||
170 |
class UniqueIntegerBugTracker(IntegerBugTracker): |
|
2376.4.19
by Jonathan Lange
Rename SimpleBugTracker to UniqueBugTracker |
171 |
"""A style of bug tracker that exists in one place only, such as Launchpad.
|
2376.4.15
by Jonathan Lange
Whitespace cleanup |
172 |
|
2376.4.41
by Jonathan Lange
Update UniqueIntegerBugTracker docstring for new API |
173 |
If you have one of these trackers then register an instance passing in an
|
3270.5.3
by James Westby
No longer add an extra class to accomoadate gnome. |
174 |
abbreviated name for the bug tracker and a base URL. The bug ids are
|
175 |
appended directly to the URL.
|
|
2376.5.1
by James Westby
Add a superclass for easy bug trackers. Also add bugs.debian.org as deb: |
176 |
"""
|
177 |
||
2376.4.25
by Jonathan Lange
Make singleton bug tracker thing work via instances. |
178 |
def __init__(self, abbreviated_bugtracker_name, base_url): |
179 |
self.abbreviation = abbreviated_bugtracker_name |
|
180 |
self.base_url = base_url |
|
181 |
||
182 |
def get(self, abbreviated_bugtracker_name, branch): |
|
2376.4.23
by Jonathan Lange
Change 'tag' to 'abbreviated_tracker_name' |
183 |
"""Returns the tracker if the abbreviation matches. Returns None
|
184 |
otherwise."""
|
|
2376.4.25
by Jonathan Lange
Make singleton bug tracker thing work via instances. |
185 |
if abbreviated_bugtracker_name != self.abbreviation: |
2376.5.1
by James Westby
Add a superclass for easy bug trackers. Also add bugs.debian.org as deb: |
186 |
return None |
2376.4.25
by Jonathan Lange
Make singleton bug tracker thing work via instances. |
187 |
return self |
2376.5.1
by James Westby
Add a superclass for easy bug trackers. Also add bugs.debian.org as deb: |
188 |
|
2376.4.40
by Jonathan Lange
Redo the hierarchy of bug trackers to reduce duplication. |
189 |
def _get_bug_url(self, bug_id): |
2376.5.1
by James Westby
Add a superclass for easy bug trackers. Also add bugs.debian.org as deb: |
190 |
"""Return the URL for bug_id."""
|
3270.5.3
by James Westby
No longer add an extra class to accomoadate gnome. |
191 |
return self.base_url + bug_id |
2376.5.1
by James Westby
Add a superclass for easy bug trackers. Also add bugs.debian.org as deb: |
192 |
|
193 |
||
2376.4.25
by Jonathan Lange
Make singleton bug tracker thing work via instances. |
194 |
tracker_registry.register( |
195 |
'launchpad', UniqueIntegerBugTracker('lp', 'https://launchpad.net/bugs/')) |
|
196 |
||
197 |
||
198 |
tracker_registry.register( |
|
2376.4.29
by Jonathan Lange
Tests for builtin trackers. |
199 |
'debian', UniqueIntegerBugTracker('deb', 'http://bugs.debian.org/')) |
2376.4.4
by jml at canonical
Beginnings of generic bug-tracker plugin system. |
200 |
|
2376.4.15
by Jonathan Lange
Whitespace cleanup |
201 |
|
3270.5.3
by James Westby
No longer add an extra class to accomoadate gnome. |
202 |
tracker_registry.register('gnome', |
203 |
UniqueIntegerBugTracker('gnome', 'http://bugzilla.gnome.org/show_bug.cgi?id=')) |
|
204 |
||
205 |
||
2376.4.40
by Jonathan Lange
Redo the hierarchy of bug trackers to reduce duplication. |
206 |
class URLParametrizedIntegerBugTracker(IntegerBugTracker): |
207 |
"""A type of bug tracker that can be found on a variety of different sites,
|
|
208 |
and thus needs to have the base URL configured.
|
|
209 |
||
210 |
Looks for a config setting in the form '<type_name>_<abbreviation>_url'.
|
|
211 |
`type_name` is the name of the type of tracker (e.g. 'bugzilla' or 'trac')
|
|
212 |
and `abbreviation` is a short name for the particular instance (e.g.
|
|
213 |
'squid' or 'apache').
|
|
214 |
"""
|
|
215 |
||
2376.4.42
by Jonathan Lange
Parametrize URLParametrizedIntegerBugTracker even further so we don't need to |
216 |
def get(self, abbreviation, branch): |
2376.4.40
by Jonathan Lange
Redo the hierarchy of bug trackers to reduce duplication. |
217 |
config = branch.get_config() |
218 |
url = config.get_user_option( |
|
2376.4.42
by Jonathan Lange
Parametrize URLParametrizedIntegerBugTracker even further so we don't need to |
219 |
"%s_%s_url" % (self.type_name, abbreviation)) |
2376.4.40
by Jonathan Lange
Redo the hierarchy of bug trackers to reduce duplication. |
220 |
if url is None: |
221 |
return None |
|
222 |
self._base_url = url |
|
2376.4.42
by Jonathan Lange
Parametrize URLParametrizedIntegerBugTracker even further so we don't need to |
223 |
return self |
224 |
||
225 |
def __init__(self, type_name, bug_area): |
|
226 |
self.type_name = type_name |
|
227 |
self._bug_area = bug_area |
|
2376.4.40
by Jonathan Lange
Redo the hierarchy of bug trackers to reduce duplication. |
228 |
|
229 |
def _get_bug_url(self, bug_id): |
|
2376.4.6
by jml at canonical
Basic docstrings for bugtracker.py |
230 |
"""Return a URL for a bug on this Trac instance."""
|
2376.4.42
by Jonathan Lange
Parametrize URLParametrizedIntegerBugTracker even further so we don't need to |
231 |
return urlutils.join(self._base_url, self._bug_area) + str(bug_id) |
232 |
||
233 |
||
234 |
tracker_registry.register( |
|
235 |
'trac', URLParametrizedIntegerBugTracker('trac', 'ticket/')) |
|
236 |
||
237 |
tracker_registry.register( |
|
238 |
'bugzilla', |
|
239 |
URLParametrizedIntegerBugTracker('bugzilla', 'show_bug.cgi?id=')) |
|
3035.3.1
by Lukáš Lalinský
Generic bug tracker configuration. |
240 |
|
241 |
||
242 |
class GenericBugTracker(URLParametrizedIntegerBugTracker): |
|
243 |
"""Generic bug tracker specified by an URL template."""
|
|
244 |
||
245 |
def __init__(self): |
|
3035.3.2
by Lukáš Lalinský
Add tests for InvalidBugTrackerURL. |
246 |
super(GenericBugTracker, self).__init__('bugtracker', None) |
247 |
||
248 |
def get(self, abbreviation, branch): |
|
249 |
self._abbreviation = abbreviation |
|
250 |
return super(GenericBugTracker, self).get(abbreviation, branch) |
|
3035.3.1
by Lukáš Lalinský
Generic bug tracker configuration. |
251 |
|
252 |
def _get_bug_url(self, bug_id): |
|
253 |
"""Given a validated bug_id, return the bug's web page's URL."""
|
|
254 |
if '{id}' not in self._base_url: |
|
255 |
raise errors.InvalidBugTrackerURL(self._abbreviation, |
|
256 |
self._base_url) |
|
257 |
return self._base_url.replace('{id}', str(bug_id)) |
|
258 |
||
259 |
||
260 |
tracker_registry.register('generic', GenericBugTracker()) |