]> git.decadent.org.uk Git - dak.git/commitdiff
Dedent again by using continue.
authorChris Lamb <lamby@debian.org>
Tue, 27 Oct 2009 19:38:18 +0000 (19:38 +0000)
committerChris Lamb <lamby@debian.org>
Tue, 27 Oct 2009 19:45:02 +0000 (19:45 +0000)
Signed-off-by: Chris Lamb <lamby@debian.org>
daklib/queue.py

index bde44c17a34ecc4a15f72fa146c737eaf26f16ac..851ae4d4aae54d4f01f93d6c2315bc7580b662d4 100755 (executable)
@@ -1261,24 +1261,26 @@ class Upload(object):
                 etext = m.group(4)
 
                 # So lets check if we know the tag at all.
-                if etag in tags:
-                    if etype == 'O':
-                        # 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.
-                            next
-                        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))
+                if etag not in tags:
+                    continue
+
+                if etype == 'O':
+                    # 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.
+                        next
+                    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):