From: Michael Casadevall Date: Wed, 24 Dec 2008 00:38:38 +0000 (-0500) Subject: Fixed handling of BYHAND packages in queue_report X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0bcdc1c91b4204ba798387d4ecfe61e0abe95b2d;p=dak Fixed handling of BYHAND packages in queue_report Signed-off-by: Michael Casadevall --- diff --git a/ChangeLog b/ChangeLog index c83e4c43..8db26638 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,6 +2,8 @@ * dak/queue_report.py - Added 822 output format + * dak/queue_report.py - Fixed display of BYHAND packages + * config/debian/dak.conf - Added 822 output location * dak/process_accepted.py - Added orig.tar.gz copying when going diff --git a/dak/queue_report.py b/dak/queue_report.py index adbc554a..097631d6 100755 --- a/dak/queue_report.py +++ b/dak/queue_report.py @@ -291,7 +291,7 @@ def table_row(source, version, arch, last_mod, maint, distribution, closes, fing ############################################################ -def process_changes_files(changes_files, type): +def process_changes_files(changes_files, type, log): msg = "" cache = {} # Read in all the .changes files @@ -439,40 +439,36 @@ def process_changes_files(changes_files, type): # Will be enhanced in the future. if Cnf.has_key("Queue-Report::Options::822"): - # Open the report file - f = open(Cnf["Queue-Report::ReportLocations::822Location"], "w") - # print stuff out in 822 format for entry in entries: (source, version_list, arch_list, note, last_modified, maint, distribution, closes, fingerprint, sponsor, changedby) = entry # We'll always have Source, Version, Arch, Mantainer, and Dist # For the rest, check to see if we have them, then print them out - f.write("Source: " + source + "\n") - f.write("Version: " + version_list + "\n") - f.write("Architectures: ") - f.write( (", ".join(arch_list.split(" "))) + "\n") - f.write("Age: " + time_pp(last_modified) + "\n") + log.write("Source: " + source + "\n") + log.write("Version: " + version_list + "\n") + log.write("Architectures: ") + log.write( (", ".join(arch_list.split(" "))) + "\n") + log.write("Age: " + time_pp(last_modified) + "\n") (name, mail) = maint.split(":") - f.write("Maintainer: " + name + " <"+mail+">" + "\n") + log.write("Maintainer: " + name + " <"+mail+">" + "\n") if changedby: (name, mail) = changedby.split(":") - f.write("Changed-By: " + name + " <"+mail+">" + "\n") + log.write("Changed-By: " + name + " <"+mail+">" + "\n") if sponsor: - f.write("Sponsored-By: " + sponsor + "\n") - f.write("Distribution:") + log.write("Sponsored-By: " + sponsor + "\n") + log.write("Distribution:") for dist in distribution: - f.write(" " + dist) - f.write("\n") - f.write("Fingerprint: " + fingerprint + "\n") + log.write(" " + dist) + log.write("\n") + log.write("Fingerprint: " + fingerprint + "\n") if closes: bug_string = "" for bugs in closes: bug_string += "#"+bugs+", " - f.write("Closes: " + bug_string[:-2] + "\n") - f.write("\n") - f.close() + log.write("Closes: " + bug_string[:-2] + "\n") + log.write("\n") if Cnf.has_key("Queue-Report::Options::New"): direction.append([4,1,"ao"]) @@ -537,9 +533,16 @@ def main(): if not directories: directories = [ "byhand", "new" ] + if Cnf.has_key("Queue-Report::Options::822"): + # Open the report file + f = open(Cnf["Queue-Report::ReportLocations::822Location"], "w") + for directory in directories: changes_files = glob.glob("%s/*.changes" % (Cnf["Dir::Queue::%s" % (directory)])) - process_changes_files(changes_files, directory) + process_changes_files(changes_files, directory, f) + + if Cnf.has_key("Queue-Report::Options::822"): + f.close() if Cnf.has_key("Queue-Report::Options::New"): footer()