From: Chris Lamb 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 --- 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):