]> git.decadent.org.uk Git - dak.git/commitdiff
mips triage
authorJames Troup <james@nocrew.org>
Tue, 23 Jan 2001 20:27:35 +0000 (20:27 +0000)
committerJames Troup <james@nocrew.org>
Tue, 23 Jan 2001 20:27:35 +0000 (20:27 +0000)
THANKS
TODO
katie

diff --git a/THANKS b/THANKS
index 0baa86ed0393682a72c4bd150f3dfd65848a80fe..4f64b130820551a7745a1399a6b761029a7a773e 100644 (file)
--- a/THANKS
+++ b/THANKS
@@ -22,3 +22,4 @@ Michael Mattice                      <mattice@debian.org>
 Peter Gervai                  <grin@debian.org>
 Robert Bihlmeyer              <robbe@orcus.priv.at>
 Roland Bauerschmidt           <rb@debian.org>
+Ryan Murray                   <rmurray@debian.org>
diff --git a/TODO b/TODO
index 62c994dd9fc81a791759c5028c5f8a4cf76f2eec..abff3f332327c181d280286b2142ab2a7b6095e1 100644 (file)
--- a/TODO
+++ b/TODO
@@ -4,12 +4,13 @@ TOFIX
 Urgent
 ------
 
-  o Claire needs to know about sections on auric.
-  o maintainers file needs overrides.
-  o jenna needs to use order by to keep aj from going nutzo
+  o Claire needs to know about sections on auric
+  o maintainers file needs overrides
+  o jenna needs to use order by [aj]
+  o seperate apt.conf for stable... [jgg]
   o need to have process_it run a big try: loop so that individual
     package screws up don't kill the daily cron run.
-  o Add an 'add' ability to melanie
+  o Add an 'add' ability to melanie (? separate prog maybe)
   o Make experimental work
 
 Less Urgent
diff --git a/katie b/katie
index 7a51cd87fb0c64d9ec3eb116dade0220e956e938..5596586d14d3a0042ff50160c64a88c66232096a 100755 (executable)
--- a/katie
+++ b/katie
@@ -2,7 +2,7 @@
 
 # Installs Debian packaes
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: katie,v 1.21 2001-01-22 22:32:47 troup Exp $
+# $Id: katie,v 1.22 2001-01-23 20:27:35 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
@@ -221,6 +221,10 @@ def check_changes(filename):
         if not Cnf.has_key("Suite::%s" % (i)):
             reject_message = reject_message + "Rejected: Unknown distribution `%s'.\n" % (i)
 
+    # Ensure there _is_ a target distribution
+    if changes["distribution"].keys() == []:
+        reject_message = reject_message + "Rejected: huh? Distribution field is empty in changes file.\n";
+            
     # Map unreleased arches from stable to unstable
     if changes["distribution"].has_key("stable"):
         for i in changes["architecture"].keys():
@@ -284,11 +288,15 @@ def check_files():
             files[file]["type"] = "byhand";
         # Checks for a binary package...
         elif re_isadeb.match(file) != None:
+            files[file]["type"] = "deb";
+
             # Extract package information using dpkg-deb
             try:
                 control = apt_pkg.ParseSection(apt_inst.debExtractControl(utils.open_file(file,"r")))
             except:
                 reject_message = reject_message + "Rejected: %s: debExtractControl() raised %s.\n" % (file, sys.exc_type);
+                # Can't continue, none of the checks on control would work.
+                continue;
 
             # Check for mandatory fields
             if control.Find("Package") == None:
@@ -327,7 +335,6 @@ def check_files():
                 files[file]["dbtype"] = "deb";
             else:
                 reject_message = reject_message + "Rejected: %s is neither a .deb or a .udeb.\n " % (file);
-            files[file]["type"] = "deb";
             files[file]["fullname"] = "%s_%s_%s.deb" % (control.Find("Package", ""), epochless_version, control.Find("Architecture", ""))
             files[file]["source"] = control.Find("Source", "");
             if files[file]["source"] == "":