[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[dak/master] show-new: reduce DB sessions even more



Signed-off-by: Torsten Werner <twerner@debian.org>
---
 dak/show_new.py |    2 +-
 daklib/queue.py |    6 +-----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/dak/show_new.py b/dak/show_new.py
index cb40a4c..6d64691 100755
--- a/dak/show_new.py
+++ b/dak/show_new.py
@@ -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)
diff --git a/daklib/queue.py b/daklib/queue.py
index ca3c133..d372721 100755
--- a/daklib/queue.py
+++ b/daklib/queue.py
@@ -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
 
 ################################################################################
-- 
1.5.6.5


Reply to: