Merge lp:~brian-murray/ubuntu-archive-tools/sru-notification-casper into lp:ubuntu-archive-tools

[Brian Murray]

Instead of spreading FUD[1] about how to test an SRU of casper, let's use a comment that says to test the latest daily Live CD.

[1] https://bugs.launchpad.net/ubuntu/+source/casper/+bug/1619188/comments/79

Also switch the default release to yakkety and stop looking up the package_name when we already know it.

=== modified file 'sru-review'
--- sru-review	2016-10-07 21:38:16 +0000
+++ sru-review	2016-10-31 22:05:32 +0000
@@ -319,19 +319,24 @@
                  'a few hours in the -proposed repository.\n\n' % (
                      release))
 
-    text += ('Please help us by testing this new package.  See '
-             'https://wiki.ubuntu.com/Testing/EnableProposed for '
-             'documentation how to enable and use -proposed.  Your feedback '
-             'will aid us getting this update out to other Ubuntu users.\n\n'
-             'If this package fixes the bug for you, please add a comment to '
-             'this bug, mentioning the version of the package you tested, and '
-             'change the tag from verification-needed to verification-done. '
-             'If it does not fix the bug for you, please add a comment '
-             'stating that, and change the tag to verification-failed.  In '
-             'either case, details of your testing will help us make a better '
-             'decision.\n\n'
-             'Further information regarding the verification process can be '
-             'found at '
+    text += ('Please help us by testing this new package.  ')
+
+    if sourcepkg == 'casper':
+        text += ('To properly test it you will need to obtain and boot '
+                 'a daily build of a Live CD for %s.' % (release))
+    else:
+        text += ('See https://wiki.ubuntu.com/Testing/EnableProposed for '
+                 'documentation on how to enable and use -proposed.')
+
+    text += ('Your feedback will aid us getting this update out to other '
+             'Ubuntu users.\n\nIf this package fixes the bug for you, '
+             'please add a comment to this bug, mentioning the version of the '
+             'package you tested, and change the tag from verification-needed '
+             'to verification-done. If it does not fix the bug for you, '
+             'please add a comment stating that, and change the tag to '
+             'verification-failed.  In either case, details of your testing '
+             'will help us make a better decision.\n\nFurther information '
+             'regarding the verification process can be found at '
              'https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  '
              'Thank you in advance!')
     bug.newMessage(content=text, subject='Please test proposed package')
@@ -339,7 +344,7 @@
 
 if __name__ == '__main__':
 
-    default_release = 'xenial'
+    default_release = 'yakkety'
     ppa_url = ('https://launchpad.net/~%s/+archive/ubuntu/%s/+packages?'
                'field.series_filter=%s')
 
@@ -416,11 +421,11 @@
         for b in changes['bugs']:
             # use a full url so the right task is highlighted
             webbrowser.open('https://bugs.launchpad.net/ubuntu/+source/'
-                            '%s/+bug/%s' % (upload.package_name, b))
+                            '%s/+bug/%s' % (sourcepkg, b))
     # also open the source package page to review version numbers
     if opts.browser:
         webbrowser.open('https://launchpad.net/ubuntu/+source/'
-                        '%s' % (upload.package_name))
+                        '%s' % (sourcepkg))
 
     if debdiff and opts.view:
         tfile = tempfile.mkstemp()