From: Chris Lamb <lamby@debian.org>
Date: Tue, 27 Oct 2009 19:41:13 +0000 (+0000)
Subject: Return all the lintian-related rejections, not just the first one.
X-Git-Url: https://git.decadent.org.uk/gitweb/?a=commitdiff_plain;ds=sidebyside;h=8964d542a2c0cc755a45d98db45d884fdc68de3c;p=dak.git

Return all the lintian-related rejections, not just the first one.

Signed-off-by: Chris Lamb <lamby@debian.org>
---

diff --git a/daklib/queue.py b/daklib/queue.py
index bf2cf418..a64df472 100755
--- a/daklib/queue.py
+++ b/daklib/queue.py
@@ -1268,19 +1268,16 @@ class Upload(object):
                     # We know it and it is overriden. Check that override is allowed.
                     if lintiantags['warning'][etag]:
                         # The tag is overriden, and it is allowed to be overriden.
-                        # Continue as if it isnt there.
-                        continue
+                        # Don't add a reject message.
                     elif lintiantags['error'][etag]:
                         # The tag is overriden - but is not allowed to be
                         self.rejects.append("%s: Overriden tag %s found, but this tag may not be overwritten." % (epackage, etag))
-                        return
                 else:
                     # Tag is known, it is not overriden, direct reject.
                     self.rejects.append("%s: Found lintian output: '%s %s', automatically rejected package." % (epackage, etag, etext))
                     # Now tell if they *might* override it.
                     if lintiantags['wayout'][etag]:
                         self.rejects.append("%s: If you have a good reason, you may override this lintian tag. Laziness to fix your crap is NOT A GOOD REASON, sod off" % (epackage))
-                    return
 
     ###########################################################################
     def check_urgency(self):