]> err.no Git - dak/commitdiff
show-new: reduce DB sessions even more
authorTorsten Werner <twerner@debian.org>
Thu, 8 Jul 2010 21:27:52 +0000 (21:27 +0000)
committerTorsten Werner <twerner@debian.org>
Thu, 8 Jul 2010 21:27:52 +0000 (21:27 +0000)
Signed-off-by: Torsten Werner <twerner@debian.org>
dak/show_new.py
daklib/queue.py

index cb40a4c98804be917ed9477039f9b65192bc1058..6d6469184b20d4af1e27a4e22a0c0331564d0139 100755 (executable)
@@ -172,7 +172,7 @@ def do_pkg(changes_file):
             u.check_source_against_db(deb_filename, session)
     u.pkg.changes["suite"] = u.pkg.changes["distribution"]
 
-    new = determine_new(u.pkg.changes, files, 0)
+    new = determine_new(u.pkg.changes, files, 0, session)
 
     htmlname = changes["source"] + "_" + changes["version"] + ".html"
     sources.add(htmlname)
index ca3c133e1fd23e3b62bc00fca013124e2c973064..d3727216355edc92c6433a6e34794ed82f29cbbe 100755 (executable)
@@ -91,7 +91,7 @@ def get_type(f, session):
 
 # Determine what parts in a .changes are NEW
 
-def determine_new(changes, files, warn=1):
+def determine_new(changes, files, warn=1, session = None):
     """
     Determine what parts in a C{changes} file are NEW.
 
@@ -110,8 +110,6 @@ def determine_new(changes, files, warn=1):
     """
     new = {}
 
-    session = DBConn().session()
-
     # Build up a list of potentially new things
     for name, f in files.items():
         # Skip byhand elements
@@ -181,8 +179,6 @@ def determine_new(changes, files, warn=1):
             if new[pkg].has_key("othercomponents"):
                 print "WARNING: %s already present in %s distribution." % (pkg, new[pkg]["othercomponents"])
 
-    session.close()
-
     return new
 
 ################################################################################