X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;ds=sidebyside;f=dak%2Fprocess_new.py;h=1c69fd9e3992507f90e1662d3117b4d540ac0d0d;hb=47b25ac6257ed67530e034c6f220b61a09e91a51;hp=36340d27bffcae03a801d522278024b44438b63f;hpb=4f5ce8467a1c46900779c9ac1de9204acc2a6d3c;p=dak.git diff --git a/dak/process_new.py b/dak/process_new.py index 36340d27..1c69fd9e 100755 --- a/dak/process_new.py +++ b/dak/process_new.py @@ -60,7 +60,7 @@ from daklib.dbconn import * from daklib.queue import * from daklib import daklog from daklib import utils -from daklib.regexes import re_no_epoch, re_default_answer, re_isanum +from daklib.regexes import re_no_epoch, re_default_answer, re_isanum, re_package from daklib.dak_exceptions import CantOpenError, AlreadyLockedError, CantGetLockError from daklib.summarystats import SummaryStats from daklib.config import Config @@ -816,36 +816,49 @@ def lock_package(package): finally: os.unlink(path) -def move_file_to_queue(to_q, f, session): - """mark a file as being in the unchecked queue""" - # update the queue_file entry for the existing queue - qf = session.query(QueueFile).filter_by(queueid=to_q.queueid, - filename=f.filename) - qf.queue = to_q +class clean_holding(object): + def __init__(self,pkg): + self.pkg = pkg + + def __enter__(self): + pass + + def __exit__(self, type, value, traceback): + h = Holding() + + for f in self.pkg.files.keys(): + if os.path.exists(os.path.join(h.holding_dir, f)): + os.unlink(os.path.join(h.holding_dir, f)) + - # update the changes_pending_files row - f.queue = to_q -def changes_to_unchecked(changes, session): - """move a changes file to unchecked""" - unchecked = get_policy_queue('unchecked', session ); - changes.in_queue = unchecked +def changes_to_newstage(upload, session): + """move a changes file to newstage""" + new = get_policy_queue('new', session ); + newstage = get_policy_queue('newstage', session ); - for f in changes.pkg.files: - move_file_to_queue(unchecked, f) + # changes.in_queue = newstage - # actually move files - changes.move_to_queue(unchecked) + chg = session.query(DBChange).filter_by(changesname=os.path.basename(upload.pkg.changes_file)).one() + chg.approved_for = newstage.policy_queue_id -def _accept(upload): + for f in chg.files: + # update the changes_pending_files row + f.queue = newstage + utils.move(os.path.join(new.path, f.filename), newstage.path, perms=int(newstage.perms, 8)) + + utils.move(os.path.join(new.path, upload.pkg.changes_file), newstage.path, perms=int(newstage.perms, 8)) + chg.in_queue = newstage + +def _accept(upload, session): if Options["No-Action"]: return (summary, short_summary) = upload.build_summaries() -# upload.accept(summary, short_summary, targetqueue) -# os.unlink(upload.pkg.changes_file[:-8]+".dak") - changes_to_unchecked(upload) + # upload.accept(summary, short_summary, targetqueue) + + changes_to_newstage(upload, session) -def do_accept(upload): +def do_accept(upload, session): print "ACCEPT" cnf = Config() if not Options["No-Action"]: @@ -859,7 +872,7 @@ def do_accept(upload): upload.Subst["__SUMMARY__"] = summary else: # Just a normal upload, accept it... - _accept(upload) + _accept(upload, session) def do_pkg(changes_file, session): new_queue = get_policy_queue('new', session ); @@ -880,19 +893,28 @@ def do_pkg(changes_file, session): u.Subst["__BCC__"] = bcc files = u.pkg.files + for deb_filename, f in files.items(): + if deb_filename.endswith(".udeb") or deb_filename.endswith(".deb"): + u.binary_file_checks(deb_filename, session) + u.check_binary_against_db(deb_filename, session) + else: + u.source_file_checks(deb_filename, session) + u.check_source_against_db(deb_filename, session) + + u.pkg.changes["suite"] = copy.copy(u.pkg.changes["distribution"]) try: with lock_package(u.pkg.changes["source"]): - if not recheck(u, session): - return + with clean_holding(u.pkg): + if not recheck(u, session): + return # FIXME: This does need byhand checks added! - print "files is %s" % (u.pkg.files) - new = determine_new(changes_file, files) + new = determine_new(u.pkg.changes, files) if new: do_new(u, session) else: - do_accept(u) + do_accept(u, session) # (new, byhand) = check_status(files) # if new or byhand: # if new: