X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;ds=sidebyside;f=dak%2Ftransitions.py;h=f329277a719db02944a9141d8fc662426a5a990a;hb=be753b4845678270ee3b3282c6218d5b511fba0c;hp=33b744b3de1c377ac7cd3eb3e6abe970674e92e6;hpb=395ac2f98083b515037e24be7d363c4838a68314;p=dak.git diff --git a/dak/transitions.py b/dak/transitions.py index 33b744b3..f329277a 100755 --- 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.package, 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, expected, source.version) print "-------------------------------------------------------------------------" ################################################################################