From: Joerg Jaspert <joerg@debian.org>
Date: Sun, 2 Mar 2008 13:39:13 +0000 (+0100)
Subject: Merge two checks if we should run or not.
X-Git-Url: https://git.decadent.org.uk/gitweb/?a=commitdiff_plain;h=ce2f7fa344cc4491662498987e17d5a9b7692b0a;p=dak.git

Merge two checks if we should run or not.
Also modify the database result check to not break.
---

diff --git a/dak/process_unchecked.py b/dak/process_unchecked.py
index ab8993a1..20a896b4 100755
--- a/dak/process_unchecked.py
+++ b/dak/process_unchecked.py
@@ -1148,19 +1148,16 @@ def upload_too_new():
 def check_transition():
     to_dump = 0
 
-    # Only check if there is a file defined with checks. It's a little bit
+    # Only check if there is a file defined (and existant) with checks. It's a little bit
     # specific to Debian, not much use for others, so return early there.
-    if not Cnf.has_key("Dinstall::Reject::ReleaseTransitions"):
+    if not Cnf.has_key("Dinstall::Reject::ReleaseTransitions") and
+    not os.path.exists("%s" % (Cnf["Dinstall::Reject::ReleaseTransitions"])):
         return
     
     # No need to do anything if this upload has no source included
     if not changes["architecture"].has_key("source"):
         return
 
-    # We first load the current set of transitions, if any
-    if not os.path.exists("%s" % (Cnf["Dinstall::Reject::ReleaseTransitions"])):
-        # Nothing to do, no file exists
-        return
     # Parse the yaml file
     sourcefile = file(Cnf["Dinstall::Reject::ReleaseTransitions"], 'r')
     transitions = load(sourcefile)
@@ -1181,9 +1178,7 @@ def check_transition():
           AND s.source='%s'"""
                                 % (source))
         ql = q.getresult()
-        if ql:
-            current_vers = ql[0][0]
-        if apt_pkg.VersionCompare(new_vers, current_vers) == 1:
+        if ql and apt_pkg.VersionCompare(new_vers, ql[0][0]) == 1:
             # This is still valid, the current version in database is older than
             # the new version we wait for
 
@@ -1194,7 +1189,7 @@ def check_transition():
 
                 Your package is part of a testing transition to get %s migrated.
 
-                Transition reason: %s
+                Transition description: %s
 
                 This transition will finish when %s, version %s, reaches testing.
                 This transition is managed by the Release Team and %s