]> git.decadent.org.uk Git - dak.git/commitdiff
* daklib/queue.py (check_transition): Added in here
authorJoerg Jaspert <joerg@debian.org>
Sun, 2 Mar 2008 15:12:20 +0000 (16:12 +0100)
committerJoerg Jaspert <joerg@debian.org>
Sun, 2 Mar 2008 15:12:20 +0000 (16:12 +0100)
  Modified so it only cleans up when called with cleanup
  parameter. Which will be called from a seperate script for the
  release managers.

* dak/process_unchecked.py
  (check_transition): Warn on broken transitions file and return,
  not doing anything.
  (check_transition): Moved out of here, into daklib/queue
  (process_it): Call it in daklib.queue, but only if
  changes[architecture] has source included.

ChangeLog
dak/process_unchecked.py
daklib/queue.py

index 71933374a54ce538b303d02def2c72ce69c11ab6..8a01bedcba64f52b4c5983cfd08799e7da17bd6a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,9 +1,19 @@
 2008-03-02  Joerg Jaspert  <joerg@debian.org>
 
+       * daklib/queue.py (check_transition): Added in here
+       Modified so it only cleans up when called with cleanup
+       parameter. Which will be called from a seperate script for the
+       release managers.
+
        * dak/process_unchecked.py (check_transition): Added. Checks if a
        release team member defined a transition, and rejects based on
        that data.
        (process_it): Use it.
+       (check_transition): Warn on broken transitions file and return,
+       not doing anything.
+       (check_transition): Moved out of here, into daklib/queue
+       (process_it): Call it in daklib.queue, but only if
+       changes[architecture] has source included.
 
 2008-02-06  Joerg Jaspert  <joerg@debian.org>
 
index 20a896b4b093a82429ba786b01178a20d81ec1d7..c3b5d2011c0306d4f551b6ed1bcc4336e7c7e25a 100755 (executable)
@@ -1141,74 +1141,6 @@ def upload_too_new():
     os.chdir(cwd)
     return too_new
 
-################################################################################
-################################################################################
-
-# We reject packages if the release team defined a transition for them
-def check_transition():
-    to_dump = 0
-
-    # 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") 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
-
-    # Parse the yaml file
-    sourcefile = file(Cnf["Dinstall::Reject::ReleaseTransitions"], 'r')
-    transitions = load(sourcefile)
-
-    # Now look through all defined transitions
-    for trans in transition:
-        t = transition[trans]
-        # We check if the transition is still valid
-        # If not we remove the whole setting from the dictionary and later dump it,
-        # so we don't process it again.
-        source = t["source"]
-        new_vers = t["new"]
-        q = Upload.projectB.query("""
-        SELECT s.version FROM source s, suite su, src_associations sa
-        WHERE sa.source=s.id
-          AND sa.suite=su.id
-          AND su.suite_name='testing'
-          AND s.source='%s'"""
-                                % (source))
-        ql = q.getresult()
-        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
-
-            # Check if the source we look at is affected by this.
-            if changes["source"] in t['packages']:
-                # The source is affected, lets reject it.
-                reject("""%s: part of the %s transition.
-
-                Your package is part of a testing transition to get %s migrated.
-
-                Transition description: %s
-
-                This transition will finish when %s, version %s, reaches testing.
-                This transition is managed by the Release Team and %s
-                is the Release-Team member responsible for it.
-                Please contact them or debian-release@lists.debian.org if you
-                need further assistance.
-                """
-                       % (changes["source"], trans, source, t["reason"], source, new_vers, t["rm"]))
-                return 0
-        else:
-            # We either have the wanted or a newer version in testing, or the package got
-            # removed completly. In that case we don't need to keep the transition blocker
-            del transition[trans]
-            to_dump = 1
-
-    if to_dump:
-        destfile = file(Cnf["Dinstall::Reject::ReleaseTransitions"], 'w')
-        dump(transition, destfile)
-
 ################################################################################
 
 def action ():
@@ -1592,7 +1524,8 @@ def process_it (changes_file):
                 check_urgency()
                 check_timestamps()
                 check_signed_by_key()
-                check_transition()
+                if changes["architecture"].has_key("source"):
+                    daklib.queue.check_transition(changes["source"])
         Upload.update_subst(reject_message)
         action()
     except SystemExit:
index 05cd0be0a2d142f4c74d7552512e21defdcb6a6d..3169e7a2122e1cdfbb846d77a200b5b0cbb58108 100755 (executable)
@@ -134,6 +134,72 @@ def check_valid(new):
            (priority != "source" and type == "dsc"):
             new[pkg]["priority id"] = -1
 
+################################################################################
+
+# We reject packages if the release team defined a transition for them
+def check_transition(sourcepkg, cleanup=0):
+    to_dump = 0
+
+    # 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") and
+    not os.path.exists("%s" % (Cnf["Dinstall::Reject::ReleaseTransitions"])):
+        return
+    
+    # Parse the yaml file
+    sourcefile = file(Cnf["Dinstall::Reject::ReleaseTransitions"], 'r')
+    try:
+        transitions = load(sourcefile)
+    except error, msg:
+        utils.warn("Not checking transitions, the transitions file is broken: %s." % (msg))
+        return
+
+    # Now look through all defined transitions
+    for trans in transition:
+        t = transition[trans]
+        # We check if the transition is still valid
+        # If not we remove the whole setting from the dictionary and later dump it,
+        # so we don't process it again.
+        source = t["source"]
+        new_vers = t["new"]
+        q = Upload.projectB.query("""
+        SELECT s.version FROM source s, suite su, src_associations sa
+        WHERE sa.source=s.id
+          AND sa.suite=su.id
+          AND su.suite_name='testing'
+          AND s.source='%s'"""
+                                % (source))
+        ql = q.getresult()
+        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
+
+            # Check if the source we look at is affected by this.
+            if sourcepkg in t['packages']:
+                # The source is affected, lets reject it.
+                reject("""%s: part of the %s transition.
+
+                Your package is part of a testing transition to get %s migrated.
+
+                Transition description: %s
+
+                This transition will finish when %s, version %s, reaches testing.
+                This transition is managed by the Release Team and %s
+                is the Release-Team member responsible for it.
+                Please contact them or debian-release@lists.debian.org if you
+                need further assistance.
+                """
+                       % (sourcepkg, trans, source, t["reason"], source, new_vers, t["rm"]))
+                return 0
+        else:
+            # We either have the wanted or a newer version in testing, or the package got
+            # removed completly. In that case we don't need to keep the transition blocker
+            del transition[trans]
+            to_dump = 1
+
+    if cleanup and to_dump:
+        destfile = file(Cnf["Dinstall::Reject::ReleaseTransitions"], 'w')
+        dump(transition, destfile)
 
 ###############################################################################