X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;f=daklib%2Fbinary.py;h=5bfa8715efad026ec819a73bc7874e6d7f9aed87;hb=4258a4e33218d4b6c271bd19b0a0723dba1fbed5;hp=2e8028f25de593f569463538b0a4fb93e44fc9e1;hpb=b7e5c9a6deda1018da9023d1d01acec86e165ba4;p=dak.git diff --git a/daklib/binary.py b/daklib/binary.py index 2e8028f2..5bfa8715 100755 --- a/daklib/binary.py +++ b/daklib/binary.py @@ -48,7 +48,7 @@ import commands import traceback import atexit from debian_bundle import deb822 -from dbconn import DBConn +from dbconn import insert_content_paths, insert_pending_content_paths from config import Config import logging import utils @@ -56,10 +56,10 @@ import utils class Binary(object): def __init__(self, filename, reject=None): """ - @ptype filename: string + @type filename: string @param filename: path of a .deb - @ptype reject: function + @type reject: function @param reject: a function to log reject messages to """ self.filename = filename @@ -138,11 +138,11 @@ class Binary(object): self.__scan_ar() rejected = not self.chunks if relaxed: - if en(self.chunks) < 3: + if len(self.chunks) < 3: rejected = True self.reject("%s: found %d chunks, expected at least 3." % (self.filename, len(self.chunks))) - else - if en(self.chunks) != 3: + else: + if len(self.chunks) != 3: rejected = True self.reject("%s: found %d chunks, expected 3." % (self.filename, len(self.chunks))) if self.chunks[0] != "debian-binary": @@ -157,7 +157,7 @@ class Binary(object): return not rejected - def scan_package(self, bootstrap_id=0, relaxed=False): + def scan_package(self, bootstrap_id=0, relaxed=False, session=None): """ Unpack the .deb, do sanity checking, and gather info from it. @@ -165,12 +165,12 @@ class Binary(object): the hopefully near future, it should also include gathering info from the control file. - @ptype bootstrap_id: int + @type bootstrap_id: int @param bootstrap_id: the id of the binary these packages should be associated or zero meaning we are not bootstrapping so insert into a temporary table - @return True if the deb is valid and contents were imported + @return: True if the deb is valid and contents were imported """ result = False rejected = not self.valid_deb(relaxed) @@ -191,11 +191,11 @@ class Binary(object): data = tarfile.open(os.path.join(self.tmpdir, "data.tar.bz2" ), "r:bz2") if bootstrap_id: - result = DBConn().insert_content_paths(bootstrap_id, [tarinfo.name for tarinfo in data if not tarinfo.isdir()]) + result = insert_content_paths(bootstrap_id, [tarinfo.name for tarinfo in data if not tarinfo.isdir()], session) else: pkgs = deb822.Packages.iter_paragraphs(file(os.path.join(self.tmpdir,'control'))) pkg = pkgs.next() - result = DBConn().insert_pending_content_paths(pkg, [tarinfo.name for tarinfo in data if not tarinfo.isdir()]) + result = insert_pending_content_paths(pkg, [tarinfo.name for tarinfo in data if not tarinfo.isdir()], session) except: traceback.print_exc() @@ -212,12 +212,11 @@ class Binary(object): the hopefully near future, it should also include gathering info from the control file. - @ptype bootstrap_id: int - @param bootstrap_id: the id of the binary these packages - should be associated or zero meaning we are not bootstrapping - so insert into a temporary table + @type package: string + @param package: the name of the package to be checked - @return True if the deb is valid and contents were imported + @rtype: boolean + @return: True if the deb is valid and contents were imported """ rejected = not self.valid_deb(True) self.__unpack()