]> git.decadent.org.uk Git - dak.git/commitdiff
Sanitize reject messages, clean up check_dsc to be less indented.
authorJames Troup <james@nocrew.org>
Tue, 14 Oct 2003 21:52:59 +0000 (21:52 +0000)
committerJames Troup <james@nocrew.org>
Tue, 14 Oct 2003 21:52:59 +0000 (21:52 +0000)
jennifer

index a88d3e5d0dcc63fb2bd6f014c63abf218cd3702b..0bec17828e6004c32a72a24ec3098968b9138a5a 100755 (executable)
--- a/jennifer
+++ b/jennifer
@@ -2,7 +2,7 @@
 
 # Checks Debian packages from Incoming
 # Copyright (C) 2000, 2001, 2002, 2003  James Troup <james@nocrew.org>
-# $Id: jennifer,v 1.40 2003-10-14 19:24:13 troup Exp $
+# $Id: jennifer,v 1.41 2003-10-14 21:52:59 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
@@ -45,7 +45,7 @@ re_valid_pkg_name = re.compile(r"^[\dA-Za-z][\dA-Za-z\+\-\.]+$");
 ################################################################################
 
 # Globals
-jennifer_version = "$Revision: 1.40 $";
+jennifer_version = "$Revision: 1.41 $";
 
 Cnf = None;
 Options = None;
@@ -137,7 +137,7 @@ def copy_to_holding(filename):
         # Shouldn't happen, but will if, for example, someone lists a
         # file twice in the .changes.
         if errno.errorcode[e.errno] == 'EEXIST':
-            reject("%s already exists in holding area; can not overwrite." % (base_filename));
+            reject("%s: already exists in holding area; can not overwrite." % (base_filename));
             return;
         raise;
 
@@ -148,11 +148,11 @@ def copy_to_holding(filename):
         # O_CREAT | O_EXCLed ghost file, so add the file to the list
         # of 'in holding' even if it's not the real file.
         if errno.errorcode[e.errno] == 'ENOENT':
-            reject("can not copy %s to holding area: file not found." % (base_filename));
+            reject("%s: can not copy to holding area: file not found." % (base_filename));
             os.unlink(dest);
             return;
         elif errno.errorcode[e.errno] == 'EACCES':
-            reject("can not copy %s to holding area: read permission denied." % (base_filename));
+            reject("%s: can not copy to holding area: read permission denied." % (base_filename));
             os.unlink(dest);
             return;
         raise;
@@ -189,25 +189,25 @@ def check_changes():
     try:
         changes.update(utils.parse_changes(filename));
     except utils.cant_open_exc:
-        reject("can't read changes file '%s'." % (filename));
+        reject("%s: can't read file." % (filename));
         return 0;
     except utils.changes_parse_error_exc, line:
-        reject("error parsing changes file '%s', can't grok: %s." % (filename, line));
+        reject("%s: parse error, can't grok: %s." % (filename, line));
         return 0;
 
     # Parse the Files field from the .changes into another dictionary
     try:
         files.update(utils.build_file_list(changes));
     except utils.changes_parse_error_exc, line:
-        reject("error parsing changes file '%s', can't grok: %s." % (filename, line));
+        reject("%s: parse error, can't grok: %s." % (filename, line));
     except utils.nk_format_exc, format:
-        reject("unknown format '%s' of changes file '%s'." % (format, filename));
+        reject("%s: unknown format '%s'." % (filename, format));
         return 0;
 
     # Check for mandatory fields
     for i in ("source", "binary", "architecture", "version", "distribution", "maintainer", "files"):
         if not changes.has_key(i):
-            reject("Missing field `%s' in changes file." % (i));
+            reject("%s: Missing mandatory field `%s'." % (filename, i));
             return 0    # Avoid <undef> errors during later tests
 
     # Split multi-value fields into a lower-level dictionary
@@ -229,7 +229,7 @@ def check_changes():
     if changes.has_key("closes"):
         for i in changes["closes"].keys():
             if katie.re_isanum.match (i) == None:
-                reject("`%s' from Closes field isn't a number." % (i));
+                reject("%s: `%s' from Closes field isn't a number." % (filename, i));
 
 
     # chopversion = no epoch; chopversion2 = no epoch and no revision (e.g. for .orig.tar.gz comparison)
@@ -566,99 +566,107 @@ SELECT c.name FROM binaries b, bin_associations ba, suite s, location l,
 def check_dsc ():
     global reprocess;
 
+    # Find the .dsc
+    dsc_filename = None;
     for file in files.keys():
-        # The .orig.tar.gz can disappear out from under us is it's a
-        # duplicate of one in the archive.
-        if not files.has_key(file):
-            continue;
         if files[file]["type"] == "dsc":
-            # Parse the .dsc file
-            try:
-                dsc.update(utils.parse_changes(file, dsc_whitespace_rules=1));
-            except utils.cant_open_exc:
-                # if not -n copy_to_holding() will have done this for us...
-                if Options["No-Action"]:
-                    reject("can't read .dsc file '%s'." % (file));
-            except utils.changes_parse_error_exc, line:
-                reject("error parsing .dsc file '%s', can't grok: %s." % (file, line));
-            except utils.invalid_dsc_format_exc, line:
-                reject("syntax error in .dsc file '%s', line %s." % (file, line));
-            # Build up the file list of files mentioned by the .dsc
-            try:
-                dsc_files.update(utils.build_file_list(dsc, is_a_dsc=1));
-            except utils.no_files_exc:
-                reject("no Files: field in .dsc file.");
-                continue;
-            except utils.changes_parse_error_exc, line:
-                reject("error parsing .dsc file '%s', can't grok: %s." % (file, line));
-                continue;
+            if dsc_filename:
+                reject("can not process a .changes file with multiple .dsc's.");
+                return;
+            else:
+                dsc_filename = file;
 
-            # Enforce mandatory fields
-            for i in ("format", "source", "version", "binary", "maintainer", "architecture", "files"):
-                if not dsc.has_key(i):
-                    reject("Missing field `%s' in dsc file." % (i));
-
-            # Validate the source and version fields
-            if dsc.has_key("source") and not re_valid_pkg_name.match(dsc["source"]):
-                reject("%s: invalid source name '%s'." % (file, dsc["source"]));
-            if dsc.has_key("version") and not re_valid_version.match(dsc["version"]):
-                reject("%s: invalid version number '%s'." % (file, dsc["version"]));
-
-            # Bumping the version number of the .dsc breaks extraction by stable's
-            # dpkg-source.  So let's not do that...
-            if dsc["format"] != "1.0":
-                reject("%s: incompatible 'Format' version produced by a broken version of dpkg-dev 1.9.1{3,4}." % (file));
-
-            # Validate the build-depends field(s)
-            for field_name in [ "build-depends", "build-depends-indep" ]:
-                field = dsc.get(field_name);
-                if field:
-                    # Check for broken dpkg-dev lossage...
-                    if field.find("ARRAY") == 0:
-                        reject("%s: invalid %s field produced by a broken version of dpkg-dev (1.10.11)" % (file, field_name.title()));
-
-                    # Have apt try to parse them...
-                    try:
-                        apt_pkg.ParseSrcDepends(field);
-                    except:
-                        reject("%s: invalid %s field (can not be parsed by apt)." % (file, field_name.title()));
-                        pass;
-
-            # Ensure the version number in the .dsc matches the version number in the .changes
-            epochless_dsc_version = utils.re_no_epoch.sub('', dsc.get("version"));
-            changes_version = files[file]["version"];
-            if epochless_dsc_version != files[file]["version"]:
-                reject("version ('%s') in .dsc does not match version ('%s') in .changes." % (epochless_dsc_version, changes_version));
-
-            # Ensure there is a .tar.gz in the .dsc file
-            has_tar = 0;
-            for f in dsc_files.keys():
-                m = utils.re_issource.match(f);
-                if not m:
-                    reject("%s mentioned in the Files field of %s not recognised as source." % (f, file));
-                type = m.group(3);
-                if type == "orig.tar.gz" or type == "tar.gz":
-                    has_tar = 1;
-            if not has_tar:
-                reject("no .tar.gz or .orig.tar.gz listed in the Files field of %s." % (file));
-
-            # Ensure source is newer than existing source in target suites
-            reject(Katie.check_source_against_db(file),"");
-
-            (reject_msg, is_in_incoming) = Katie.check_dsc_against_db(file);
-            reject(reject_msg, "");
-            if is_in_incoming:
-                if not Options["No-Action"]:
-                    copy_to_holding(is_in_incoming);
-                orig_tar_gz = os.path.basename(is_in_incoming);
-                files[orig_tar_gz] = {};
-                files[orig_tar_gz]["size"] = os.stat(orig_tar_gz)[stat.ST_SIZE];
-                files[orig_tar_gz]["md5sum"] = dsc_files[orig_tar_gz]["md5sum"];
-                files[orig_tar_gz]["section"] = files[file]["section"];
-                files[orig_tar_gz]["priority"] = files[file]["priority"];
-                files[orig_tar_gz]["component"] = files[file]["component"];
-                files[orig_tar_gz]["type"] = "orig.tar.gz";
-                reprocess = 2;
+    # If there isn't one, we have nothing to do...
+    if not dsc_filename:
+        return;
+
+    # Parse the .dsc file
+    try:
+        dsc.update(utils.parse_changes(dsc_filename, dsc_whitespace_rules=1));
+    except utils.cant_open_exc:
+        # if not -n copy_to_holding() will have done this for us...
+        if Options["No-Action"]:
+            reject("%s: can't read file." % (dsc_filename));
+    except utils.changes_parse_error_exc, line:
+        reject("%s: parse error, can't grok: %s." % (dsc_filename, line));
+    except utils.invalid_dsc_format_exc, line:
+        reject("%s: syntax error on line %s." % (dsc_filename, line));
+    # Build up the file list of files mentioned by the .dsc
+    try:
+        dsc_files.update(utils.build_file_list(dsc, is_a_dsc=1));
+    except utils.no_files_exc:
+        reject("%s: no Files: field." % (dsc_filename));
+        return;
+    except utils.changes_parse_error_exc, line:
+        reject("%s: parse error, can't grok: %s." % (dsc_filename, line));
+        return;
+
+    # Enforce mandatory fields
+    for i in ("format", "source", "version", "binary", "maintainer", "architecture", "files"):
+        if not dsc.has_key(i):
+            reject("%s: missing mandatory field `%s'." % (dsc_filename, i));
+
+    # Validate the source and version fields
+    if dsc.has_key("source") and not re_valid_pkg_name.match(dsc["source"]):
+        reject("%s: invalid source name '%s'." % (dsc_filename, dsc["source"]));
+    if dsc.has_key("version") and not re_valid_version.match(dsc["version"]):
+        reject("%s: invalid version number '%s'." % (dsc_filename, dsc["version"]));
+
+    # Bumping the version number of the .dsc breaks extraction by stable's
+    # dpkg-source.  So let's not do that...
+    if dsc["format"] != "1.0":
+        reject("%s: incompatible 'Format' version produced by a broken version of dpkg-dev 1.9.1{3,4}." % (dsc_filename));
+
+    # Validate the build-depends field(s)
+    for field_name in [ "build-depends", "build-depends-indep" ]:
+        field = dsc.get(field_name);
+        if field:
+            # Check for broken dpkg-dev lossage...
+            if field.find("ARRAY") == 0:
+                reject("%s: invalid %s field produced by a broken version of dpkg-dev (1.10.11)" % (dsc_filename, field_name.title()));
+
+            # Have apt try to parse them...
+            try:
+                apt_pkg.ParseSrcDepends(field);
+            except:
+                reject("%s: invalid %s field (can not be parsed by apt)." % (dsc_filename, field_name.title()));
+                pass;
+
+    # Ensure the version number in the .dsc matches the version number in the .changes
+    epochless_dsc_version = utils.re_no_epoch.sub('', dsc.get("version"));
+    changes_version = files[dsc_filename]["version"];
+    if epochless_dsc_version != files[dsc_filename]["version"]:
+        reject("version ('%s') in .dsc does not match version ('%s') in .changes." % (epochless_dsc_version, changes_version));
+
+    # Ensure there is a .tar.gz in the .dsc file
+    has_tar = 0;
+    for f in dsc_files.keys():
+        m = utils.re_issource.match(f);
+        if not m:
+            reject("%s: %s in Files field not recognised as source." % (dsc_filename, f));
+        type = m.group(3);
+        if type == "orig.tar.gz" or type == "tar.gz":
+            has_tar = 1;
+    if not has_tar:
+        reject("%s: no .tar.gz or .orig.tar.gz in 'Files' field." % (dsc_filename));
+
+    # Ensure source is newer than existing source in target suites
+    reject(Katie.check_source_against_db(dsc_filename),"");
+
+    (reject_msg, is_in_incoming) = Katie.check_dsc_against_db(dsc_filename);
+    reject(reject_msg, "");
+    if is_in_incoming:
+        if not Options["No-Action"]:
+            copy_to_holding(is_in_incoming);
+        orig_tar_gz = os.path.basename(is_in_incoming);
+        files[orig_tar_gz] = {};
+        files[orig_tar_gz]["size"] = os.stat(orig_tar_gz)[stat.ST_SIZE];
+        files[orig_tar_gz]["md5sum"] = dsc_files[orig_tar_gz]["md5sum"];
+        files[orig_tar_gz]["section"] = files[dsc_filename]["section"];
+        files[orig_tar_gz]["priority"] = files[dsc_filename]["priority"];
+        files[orig_tar_gz]["component"] = files[dsc_filename]["component"];
+        files[orig_tar_gz]["type"] = "orig.tar.gz";
+        reprocess = 2;
 
 ################################################################################