launchpad-reviewers team mailing list archive

[Jonathan Riddell <jriddell@xxxxxxxxxx>]

Jonathan Riddell has proposed merging lp:~jr/launchpad/797688-packaging-branches-label into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #797688 in Launchpad itself: "Ubuntu Packaging Branches Should be Labeled Clearer"
  https://bugs.launchpad.net/launchpad/+bug/797688

For more details, see:
https://code.launchpad.net/~jr/launchpad/797688-packaging-branches-label/+merge/66146

Places a message on packaging branch pages to inform the user these branches are for distro specific changes, links to upstream if known.

-- 
https://code.launchpad.net/~jr/launchpad/797688-packaging-branches-label/+merge/66146
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jr/launchpad/797688-packaging-branches-label into lp:launchpad.

=== modified file 'lib/lp/code/templates/distributionsourcepackage-branches-grouped.pt'
--- lib/lp/code/templates/distributionsourcepackage-branches-grouped.pt	2010-08-23 03:30:54 +0000
+++ lib/lp/code/templates/distributionsourcepackage-branches-grouped.pt	2011-06-28 13:42:14 +0000
@@ -10,6 +10,24 @@
 
 <div metal:fill-slot="main">
 
+  <p>
+    <strong>This page is for a packaging branch for a program in
+    <tal:name replace="context/distribution/displayname"/>.</strong> Changes
+    should be made here to
+    <tal:name replace="context/distribution/displayname"/> packaging or
+    for changes specific to
+    <tal:name replace="context/distribution/displayname"/>'s
+    needs. Changes to the program in general should be made on the
+    upstream branch
+
+    <tal:upstream condition="context/upstream_product">
+        <a tal:replace="structure context/upstream_product/fmt:link" />.
+    </tal:upstream>
+    <tal:no-upstream condition="not: context/upstream_product">
+        which is not currently known to Launchpad.
+    </tal:no-upstream>
+  </p>
+
   <tal:no-branches condition="not: view/branch_count">
     <div id="branch-summary">
     There are no branches for the <tal:name replace="context/title"/>