Hi Trevor,
> There should also be some way to remove targets.
Thanks, fixed as:
if target:
current_target = bug_target(self.bd, bug)
if current_target:
remove_target(self.bd, bug)
if target != "None":
add_target(self.bd, bug, target)
else:
add_target(self.bd, bug, target)
I was mildly surprised that bug_target() throws an exception on a bug
with more than one target -- is that a limitation of targets, or just
of bug_target()? If the former, we should probably make add_target()
refuse to add a target to a bug that already has one..
Will push this patchset along with a merge of your tree to gitorious,
unless you have any more improvements on this patch. Thanks for the
review!
- Chris.
--
Chris Ball <[email protected]>
One Laptop Per Child
_______________________________________________
Be-devel mailing list
[email protected]
http://void.printf.net/cgi-bin/mailman/listinfo/be-devel
