Merge lp:~brian-murray/ubuntu-archive-tools/sru-accept-web-links into lp:ubuntu-archive-tools

Proposed by Brian Murray
Status: Merged
Merged at revision: 471
Proposed branch: lp:~brian-murray/ubuntu-archive-tools/sru-accept-web-links
Merge into: lp:ubuntu-archive-tools
Diff against target: 35 lines (+7/-2) (has conflicts)
2 files modified
sru-accept.py (+2/-2)
sru-report (+5/-0)
Text conflict in sru-report
To merge this branch: bzr merge lp:~brian-murray/ubuntu-archive-tools/sru-accept-web-links
Reviewer Review Type Date Requested Status
Colin Watson Approve
Review via email: mp+108409@code.launchpad.net

Description of the change

sru-accept.py prints out information about the tasks that it modified however those links have api.somethingsomething which won't work in a browser. I changed to using task.web_link which will.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) wrote :

LGTM, modulo conflicts in sru-report (which I'll resolve).

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sru-accept.py'
--- sru-accept.py 2012-04-26 16:38:48 +0000
+++ sru-accept.py 2012-06-01 19:44:18 +0000
@@ -59,13 +59,13 @@
59 match = bug_target_re.search(task.target.self_link)59 match = bug_target_re.search(task.target.self_link)
60 if (not match or60 if (not match or
61 (options.package and match.group('source') != options.package)):61 (options.package and match.group('source') != options.package)):
62 print("Ignoring task %s in bug %s" % (task.target, num))62 print("Ignoring task %s in bug %s" % (task.web_link, num))
63 continue63 continue
64 if (match.group('suite') in options.targets and64 if (match.group('suite') in options.targets and
65 task.status not in ("Invalid", "Won't Fix", "Fix Committed", "Fix Released")):65 task.status not in ("Invalid", "Won't Fix", "Fix Committed", "Fix Released")):
66 task.status = "Fix Committed"66 task.status = "Fix Committed"
67 task.lp_save()67 task.lp_save()
68 print("Success: task %s in bug %s" % (task.target, num))68 print("Success: task %s in bug %s" % (task.web_link, num))
6969
70 bug.subscribe(person=launchpad.people['ubuntu-sru'])70 bug.subscribe(person=launchpad.people['ubuntu-sru'])
71 bug.subscribe(person=launchpad.people['sru-verification'])71 bug.subscribe(person=launchpad.people['sru-verification'])
7272
=== modified file 'sru-report'
--- sru-report 2012-06-01 15:32:20 +0000
+++ sru-report 2012-06-01 19:44:18 +0000
@@ -364,8 +364,13 @@
364def get_queue_count(search_status, release, search_pocket):364def get_queue_count(search_status, release, search_pocket):
365 '''Return number of results of given queue page URL'''365 '''Return number of results of given queue page URL'''
366366
367<<<<<<< TREE
367 count = len(release.getPackageUploads(status=search_status,368 count = len(release.getPackageUploads(status=search_status,
368 archive=archive, pocket=search_pocket))369 archive=archive, pocket=search_pocket))
370=======
371 count = int(release.getPackageUploads(status=search_status,
372 pocket=search_pocket)._wadl_resource.representation['total_size'])
373>>>>>>> MERGE-SOURCE
369 return count374 return count
370375
371376

Subscribers

People subscribed via source and target branches