why do people want readable messages?

[dak.git] / dak / transitions.py

diff --git a/dak/transitions.py b/dak/transitions.py
index 33b744b3de1c377ac7cd3eb3e6abe970674e92e6..9aaed6247f26e9f9e418056a8115e5287a2445a1 100755 (executable)
--- a/dak/transitions.py
+++ b/dak/transitions.py
@@ -29,7 +29,6 @@ Display, edit and check the release manager's transition file.
 ################################################################################
 
 import os
-import pg
 import sys
 import time
 import errno
@@ -37,6 +36,7 @@ import fcntl
 import tempfile
 import pwd
 import apt_pkg
+
 from daklib.dbconn import *
 from daklib import utils
 from daklib.dak_exceptions import TransitionsError
@@ -84,9 +84,7 @@ def init():
     if Options["help"]:
         usage()
 
-    whoami = os.getuid()
-    whoamifull = pwd.getpwuid(whoami)
-    username = whoamifull[0]
+    username = utils.getusername()
     if username != "dak":
         print "Non-dak user: %s" % username
         Options["sudo"] = "y"
@@ -513,7 +511,7 @@ New Version: %s
 Responsible: %s
 Description: %s
 Blocked Packages (total: %d): %s
-""" % (trans, source, expected, rm, reason, len(packages), ", ".join(packages))
+""" % (trans, source.source, expected, rm, reason, len(packages), ", ".join(packages))
 
 ################################################################################
 
@@ -526,30 +524,32 @@ def transition_info(transitions):
     @type transitions: dict
     @param transitions: defined transitions
     """
+
+    session = DBConn().session()
+
     for trans in transitions:
         t = transitions[trans]
         source = t["source"]
         expected = t["new"]
 
-        # Will be empty list if nothing is in testing.
-        sources = get_suite_version(source, "testing")
+        # Will be None if nothing is in testing.
+        source = get_source_in_suite(source, "testing", session)
 
         print get_info(trans, source, expected, t["rm"], t["reason"], t["packages"])
 
-        if len(sources) < 1:
+        if source is None:
             # No package in testing
             print "Transition source %s not in testing, transition still ongoing." % (source)
         else:
-            current = sources[0].version
-            compare = apt_pkg.VersionCompare(current, expected)
+            compare = apt_pkg.VersionCompare(source.version, expected)
             print "Apt compare says: %s" % (compare)
             if compare < 0:
                 # This is still valid, the current version in database is older than
                 # the new version we wait for
-                print "This transition is still ongoing, we currently have version %s" % (current)
+                print "This transition is still ongoing, we currently have version %s" % (source.version)
             else:
                 print "This transition is over, the target package reached testing, should be removed"
-                print "%s wanted version: %s, has %s" % (source, expected, current)
+                print "%s wanted version: %s, has %s" % (source.source, expected, source.version)
         print "-------------------------------------------------------------------------"
 
 ################################################################################