Ubuntu IRC Bots

Merge lp:~gz/ubuntu-bots/oops_link_1051902 into lp:ubuntu-bots

  1. oops_link_1051902
  2. Merge into devel
Proposed by Martin Packman
Status: Merged
Approved by: Terence Simpson
Approved revision: 298
Merged at revision: 298
Proposed branch: lp:~gz/ubuntu-bots/oops_link_1051902
Merge into: lp:ubuntu-bots
Diff against target: 12 lines (+1/-1)
1 file modified
Bugtracker/plugin.py (+1/-1)
To merge this branch: bzr merge lp:~gz/ubuntu-bots/oops_link_1051902
Reviewer Review Type Date Requested Status
m4v Abstain
Alan Bell Approve
Review via email: mp+124648@code.launchpad.net

Commit message

Update OOPS URL

Description of the change

Since OOPS has been picked up by services other than launchpad, a new site for looking them up was created. The old launchpad specific site still exists for now but new oopses only go to the new site, so the bot should link there instead. This branch switches the address to the new form.

How is deployment handled? Do I need to some extra steps after this change is merged to get the new code live on the ubot5 in #launchpad etc?

To post a comment you must log in.
Revision history for this message
Alan Bell (alanbell) wrote :

simple update, no risk

review: Approve
Revision history for this message
m4v (m4v) wrote :

Famous last words :P

Thanks for the patch Martin, deployment will depend on the bot's owner. Hopefully it will be shortly.

Revision history for this message
m4v (m4v) :
review: Abstain

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Bugtracker/plugin.py'
2--- Bugtracker/plugin.py 2012-07-04 13:21:37 +0000
3+++ Bugtracker/plugin.py 2012-09-17 10:57:26 +0000
4@@ -450,7 +450,7 @@
5 return
6 if not self.is_ok(channel, 'lpoops', oopsid):
7 return
8- irc.reply('https://lp-oops.canonical.com/oops.py/?oopsid=' + oopsid, prefixNick=False)
9+ irc.reply('https://oops.canonical.com/?oopsid=' + oopsid, prefixNick=False)
10
11 def cveSnarfer(self, irc, msg, match):
12 r"(cve[- ]\d{4}[- ]\d{4})"

Subscribers

People subscribed via source and target branches