Merge lp:~micahg/ubuntu-transition-tracker/add-pts-link into lp:ubuntu-transition-tracker

Proposed by Micah Gersten on 2012-08-13
Status: Needs review
Proposed branch: lp:~micahg/ubuntu-transition-tracker/add-pts-link
Merge into: lp:ubuntu-transition-tracker
Diff against target: 39 lines (+14/-0)
2 files modified
debian/changelog (+8/-0)
frontends/ben_monitor.ml (+6/-0)
To merge this branch: bzr merge lp:~micahg/ubuntu-transition-tracker/add-pts-link
Reviewer Review Type Date Requested Status
Gianfranco Costamagna Needs Fixing on 2017-08-05
Iain Lane 2012-08-13 Approve on 2012-08-14
Review via email: mp+119287@code.launchpad.net
To post a comment you must log in.
Micah Gersten (micahg) wrote :

Just would like another set of eyes before I push to the branch and upload.

Iain Lane (laney) wrote :

Looks good. We probably ought to rename "pts" for sanity, but that's preexisting.

For info, the precise-backports branch is: lp:~ubuntu-transition-trackers/ubuntu-transition-tracker/precise-backports

review: Approve

I think this needs new work for merging...

review: Needs Fixing

Unmerged revisions

173. By Micah Gersten on 2012-08-13

* Add a PTS link so that we can check if there's a new version in Debian
  more easily
  - update frontends/ben_monitor.ml

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-07-04 19:50:41 +0000
+++ debian/changelog 2012-08-13 00:13:27 +0000
@@ -1,3 +1,11 @@
1ben (0.6.1ubuntu3) UNRELEASED; urgency=low
2
3 * Add a PTS link so that we can check if there's a new version in Debian
4 more easily
5 - update frontends/ben_monitor.ml
6
7 -- Micah Gersten <micahg@ubuntu.com> Sun, 12 Aug 2012 19:10:33 -0500
8
1ben (0.6.1ubuntu2) quantal; urgency=low9ben (0.6.1ubuntu2) quantal; urgency=low
210
3 * Update Vcs-* for Ubuntu11 * Update Vcs-* for Ubuntu
412
=== modified file 'frontends/ben_monitor.ml'
--- frontends/ben_monitor.ml 2012-07-04 18:05:13 +0000
+++ frontends/ben_monitor.ml 2012-08-13 00:13:27 +0000
@@ -538,6 +538,9 @@
538let pts src =538let pts src =
539 a_link (sprintf "http://launchpad.net/ubuntu/+source/%s" (escape src)) src539 a_link (sprintf "http://launchpad.net/ubuntu/+source/%s" (escape src)) src
540540
541let debianpts src =
542 a_link (sprintf "http://packages.qa.debian.org/%s" (escape src)) "PTS"
543
541let buildd src vers =544let buildd src vers =
542 a_link (sprintf "http://launchpad.net/ubuntu/+source/%s/%s" (escape src)545 a_link (sprintf "http://launchpad.net/ubuntu/+source/%s/%s" (escape src)
543 (escape vers)) "build logs"546 (escape vers)) "build logs"
@@ -790,6 +793,9 @@
790 td793 td
791 ~a:[ a_class [ "src"] ]794 ~a:[ a_class [ "src"] ]
792 [ pcdata "[";795 [ pcdata "[";
796 debianpts (escape src);
797 pcdata "] ";
798 pcdata "[";
793 buildd (escape src) version;799 buildd (escape src) version;
794 pcdata "] (";800 pcdata "] (";
795 changelog (sprintf "%s" version) directory (escape src);801 changelog (sprintf "%s" version) directory (escape src);

Subscribers

People subscribed via source and target branches