[Secure-testing-commits] r19975 - lib/python

[Raphael Geissert]

Author: geissert
Date: 2012-08-19 23:32:00 +0000 (Sun, 19 Aug 2012)
New Revision: 19975

Modified:
   lib/python/bugs.py
Log:
Raise a proper exception instead of failing an assertion

fw: why is there a reason field? it doesn't seem to be used at all. In
fact, it pretty much looks like 'comment' is redundant and 'reason'
should be the one getting a value assigned.



Modified: lib/python/bugs.py
===================================================================
--- lib/python/bugs.py	2012-08-19 23:11:07 UTC (rev 19974)
+++ lib/python/bugs.py	2012-08-19 23:32:00 UTC (rev 19975)
@@ -593,6 +593,10 @@
                                 self.raiseSyntaxError(
                                     "no-dsa note needs release specification",
                                     lineno)
+                            if not d:
+                                self.raiseSyntaxError(
+                                    "no-dsa note needs comment",
+                                    lineno)
                             pkg_notes.append(PackageNoteNoDSA(
                                 release=release,
                                 package=p,