Merge lp:~seb128/ubuntu-archive-tools/component-mismatches-mirtitles into lp:ubuntu-archive-tools

[Sebastien Bacher]

component-mismatches: always include the MIR titles

those are useful in the diff emails generated

The title does some useful context, especially for the diff emails generated.

Currently that's an example of summary we get mailed to the release mailing list when only the status is changing (https://lists.ubuntu.com/archives/ubuntu-release/2020-November/005129.html)

' MIR: #1889688 (Fix Released)'

which doesn't give enough information to tell you what request it is about, I personally often end up just copying the bug number to a webbrowser to go see if that's a package my team cares about. Having the title would probably be enough in most cases

[Sebastien Bacher (seb128)]

I've updated only the txt output which is used for the diff generation, unsure if we want the same change in the html (it's not really required on the report since the package name is included there)

[Iain Lane (laney)]

Cheers. I just fixed a minor indentation problem, but it's merged now!

review: Approve

=== modified file 'component-mismatches'
--- component-mismatches	2020-05-28 04:58:54 +0000
+++ component-mismatches	2020-12-10 10:19:44 +0000
@@ -74,13 +74,10 @@
 
     def __str__(self):
         if self.status not in ('Fix Committed', 'Fix Released') and self.assignee:
-            s = "MIR: #%d (%s for %s)" % (self.id, self.status,
-                                          self.assignee.display_name)
+            s = "MIR: #%d (%s for %s) %s" % (self.id, self.status,
+                                          self.assignee.display_name, self.title)
         else:
-            s = "MIR: #%d (%s)" % (self.id, self.status)
-        # no need to repeat the standard title
-        if not self.title.startswith("[MIR]"):
-            s += " %s" % self.title
+            s = "MIR: #%d (%s) %s" % (self.id, self.status, self.title)
         return s
 
     def html(self):