From: Philipp Kern Date: Tue, 26 Aug 2008 10:25:39 +0000 (+0000) Subject: fix proposed-updates processing in dinstall: actually accept sourceful uploads X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=302210275ffbcab0b29c1d2af7433e8e28f40b93;p=dak fix proposed-updates processing in dinstall: actually accept sourceful uploads * dak/process_new.py (is_source_in_queue_dir): fix variable usage * dak/process_new.py (move_to_holding): just state what we intend to do in no-action mode * dak/process_new.py (do_accept_stableupdate): fetch summaries, fix invokation of is_source_in_queue_dir, actually accept sourceful uploads in p-u holding --- diff --git a/ChangeLog b/ChangeLog index 118a0570..2a18cf71 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2008-08-26 Philipp Kern + + * dak/process_new.py (is_source_in_queue_dir): fix variable usage + * dak/process_new.py (move_to_holding): just state what we intend + to do in no-action mode + * dak/process_new.py (do_accept_stableupdate): fetch summaries, + fix invokation of is_source_in_queue_dir, actually accept sourceful + uploads in p-u holding + 2008-08-26 Philipp Kern * dak/process_new.py (do_accept): do not try to free the unchecked diff --git a/dak/process_new.py b/dak/process_new.py index b40ca6ec..6a66e321 100755 --- a/dak/process_new.py +++ b/dak/process_new.py @@ -821,7 +821,7 @@ def is_source_in_queue_dir(qdir): u = queue.Upload(Cnf) u.pkg.changes_file = entry u.update_vars() - if not changes["architecture"].has_key("source"): + if not Upload.pkg.changes["architecture"].has_key("source"): # another binary upload, ignore continue if Upload.pkg.changes["version"] != u.pkg.changes["version"]: @@ -833,24 +833,27 @@ def is_source_in_queue_dir(qdir): def move_to_holding(suite, queue_dir): print "Moving to %s holding area." % (suite.upper(),) + if Options["No-Action"]: + return Logger.log(["Moving to %s" % (suite,), Upload.pkg.changes_file]) Upload.dump_vars(queue_dir) move_to_dir(queue_dir) os.unlink(Upload.pkg.changes_file[:-8]+".dak") def do_accept_stableupdate(suite, q): + (summary, short_summary) = Upload.build_summaries() queue_dir = Cnf["Dir::Queue::%s" % (q,)] if not Upload.pkg.changes["architecture"].has_key("source"): # It is not a sourceful upload. So its source may be either in p-u # holding, in new, in accepted or already installed. - if is_source_in_qdir(queue_dir): + if is_source_in_queue_dir(queue_dir): # It's in p-u holding, so move it there. move_to_holding(suite, queue_dir) - elif is_source_in_qdir(Cnf["Dir::Queue::New"]): + elif is_source_in_queue_dir(Cnf["Dir::Queue::New"]): # It's in NEW. We expect the source to land in p-u holding # pretty soon. move_to_holding(suite, queue_dir) - elif is_source_in_qdir(Cnf["Dir::Queue::Accepted"]): + elif is_source_in_queue_dir(Cnf["Dir::Queue::Accepted"]): # The source is in accepted, the binary cleared NEW: accept it. Upload.accept(summary, short_summary) os.unlink(Upload.pkg.changes_file[:-8]+".dak") @@ -861,8 +864,14 @@ def do_accept_stableupdate(suite, q): # accepted. Upload.accept(summary, short_summary) os.unlink(Upload.pkg.changes_file[:-8]+".dak") - return - move_to_holding(suite, queue_dir) + else: + # We are handling a sourceful upload. Move to accepted if currently + # in p-u holding and to p-u holding otherwise. + if is_source_in_queue_dir(queue_dir): + Upload.accept(summary, short_summary) + os.unlink(Upload.pkg.changes_file[:-8]+".dak") + else: + move_to_holding(suite, queue_dir) def do_accept(): print "ACCEPT"