From eb995fdaa1ecfe19da5b5ebbe3c42d08c000e6ac Mon Sep 17 00:00:00 2001
From: James Troup <james@nocrew.org>
Date: Fri, 15 Feb 2002 02:54:22 +0000
Subject: [PATCH] logging support; unconfuse stdout/stderr use; get option
 changes fields right in dump_vars

---
 jennifer | 10 ++++++----
 katie.py |  6 ++++--
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/jennifer b/jennifer
index 574302eb..20469d13 100755
--- a/jennifer
+++ b/jennifer
@@ -2,7 +2,7 @@
 
 # Checks Debian packages from Incoming
 # Copyright (C) 2000, 2001  James Troup <james@nocrew.org>
-# $Id: jennifer,v 1.1 2002-02-12 23:08:07 troup Exp $
+# $Id: jennifer,v 1.2 2002-02-15 02:54:22 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -44,7 +44,7 @@ re_is_changes = re.compile (r"(.+?)_(.+?)_(.+?)\.changes$");
 ################################################################################
 
 # Globals
-jennifer_version = "$Revision: 1.1 $";
+jennifer_version = "$Revision: 1.2 $";
 
 Cnf = None;
 Options = None;
@@ -971,6 +971,7 @@ def accept (summary, short_summary):
 
 def do_byhand (summary):
     print "Moving to BYHAND holding area."
+    Logger.log(["Moving to byhand", pkg.changes_file]);
 
     Katie.dump_vars(Cnf["Dir::QueueByhandDir"]);
 
@@ -998,6 +999,7 @@ def acknowledge_new (summary):
     Subst = Katie.Subst;
 
     print "Moving to NEW holding area."
+    Logger.log(["Moving to new", pkg.changes_file]);
 
     Katie.dump_vars(Cnf["Dir::QueueNewDir"]);
 
@@ -1071,7 +1073,7 @@ def process_it (changes_file):
         raise;
     except:
         print "ERROR";
-	traceback.print_exc(file=sys.stdout);
+	traceback.print_exc(file=sys.stderr);
         pass;
 
     # Restore previous WD
@@ -1142,7 +1144,7 @@ def main():
         sets = "set"
         if accept_count > 1:
             sets = "sets"
-        sys.stderr.write("Accepted %d package %s, %s.\n" % (accept_count, sets, utils.size_type(int(accept_bytes))));
+        print "Accepted %d package %s, %s." % (accept_count, sets, utils.size_type(int(accept_bytes)));
         Logger.log(["total",accept_count,accept_bytes]);
 
     if not Options["No-Action"]:
diff --git a/katie.py b/katie.py
index 77ff722d..24647778 100644
--- a/katie.py
+++ b/katie.py
@@ -2,7 +2,7 @@
 
 # Utility functions for katie
 # Copyright (C) 2001  James Troup <james@nocrew.org>
-# $Id: katie.py,v 1.2 2002-02-15 01:48:20 troup Exp $
+# $Id: katie.py,v 1.3 2002-02-15 02:54:22 troup Exp $
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -162,7 +162,8 @@ class Katie:
             d_changes[i] = changes[i];
         # Optional changes fields
         for i in [ "changed-by", "changedby822", "maintainer822", "filecontents" ]:
-            d_changes[i] = changes[i];
+            if d_changes.has_key(i):
+                d_changes[i] = changes[i];
         ## dsc
         for i in [ "source", "version", "maintainer", "fingerprint" ]:
             if dsc.has_key(i):
@@ -333,6 +334,7 @@ class Katie:
         files = self.pkg.files;
 
         print "Accepting."
+        self.Logger.log(["Accepting changes",self.pkg.changes_file]);
 
         self.dump_vars(Cnf["Dir::QueueAcceptedDir"]);
 
-- 
2.39.5