Merge lp:~brian-murray/ubuntu-archive-tools/sru-review-bug-check into lp:ubuntu-archive-tools

[Brian Murray]

Let's try and avoiding having SRUs without a bug reference see LP: #1444436.

[Brian Murray (brian-murray)]

This wouldn't have produced any Launchpad-Bugs-Fixed:

xserver-xorg-video-intel (2:2.99.914-1~exp1ubuntu4.3) utopic; urgency=medium

  * bdw-annotate-more-64bit-pointer-locations.diff
    bdw-clamp-urb-allocations-for-gt3.diff:
    Fix BDW GT3. (#1444436)

[lp:~brian-murray/ubuntu-archive-tools/sru-review-bug-check]

961. By Brian Murray on 2015-05-07

sru-review: remove continue and exit on N

962. By Brian Murray on 2015-05-08

sru-review: Also remind after viewing the debdiff that there are no Launchpad bugs.

=== modified file 'sru-review'
--- sru-review	2015-05-05 01:52:51 +0000
+++ sru-review	2015-05-08 19:23:20 +0000
@@ -378,6 +378,17 @@
     # to view the diff
     changes = parse_changes(changes_url)
 
+    changelog_bugs = True
+    if not changes['bugs']:
+        changelog_bugs = False
+        print('There are no Launchpad bugs in the changelog!',
+              file=sys.stderr)
+        print('''View the debdiff anyway? [yN]''', end="")
+        response = sys.stdin.readline()
+        if response.strip().lower().startswith('n'):
+            print('''Exiting''')
+            sys.exit(1)
+
     if opts.browser and changes['bugs']:
         for b in changes['bugs']:
             # use a full url so the right task is highlighted
@@ -403,6 +414,8 @@
               file=sys.stderr)
         sys.exit(0)
 
+    if not changelog_bugs:
+        print("The SRU has no Launchpad bugs referenced!\n")
     print("Accept the package into -proposed? [yN] ", end="")
     response = sys.stdin.readline()
     if response.strip().lower().startswith('y'):