]> git.decadent.org.uk Git - dak.git/blobdiff - utils.py
sane config stuff
[dak.git] / utils.py
index 11f0c475efd66bc114d44927dd92fcf4ee607e61..60efbb9b14832c4d734c80c3737e4e14aa62e7e6 100644 (file)
--- a/utils.py
+++ b/utils.py
@@ -1,6 +1,6 @@
 # Utility functions
-# Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: utils.py,v 1.21 2001-04-13 20:11:20 troup Exp $
+# Copyright (C) 2000, 2001  James Troup <james@nocrew.org>
+# $Id: utils.py,v 1.33 2001-09-26 03:49:16 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
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
 import commands, os, pwd, re, socket, shutil, stat, string, sys, tempfile
+import apt_pkg
 
 re_comments = re.compile(r"\#.*")
 re_no_epoch = re.compile(r"^\d*\:")
 re_no_revision = re.compile(r"\-[^-]*$")
 re_arch_from_filename = re.compile(r"/binary-[^/]+/")
 re_extract_src_version = re.compile (r"(\S+)\s*\((.*)\)")
-re_isadeb = re.compile (r'.*\.u?deb$');
-re_issource = re.compile (r'(.+)_(.+?)\.(orig\.tar\.gz|diff\.gz|tar\.gz|dsc)');
+re_isadeb = re.compile (r".*\.u?deb$");
+re_issource = re.compile (r"(.+)_(.+?)\.(orig\.tar\.gz|diff\.gz|tar\.gz|dsc)");
+
+re_single_line_field = re.compile(r"^(\S*)\s*:\s*(.*)");
+re_multi_line_field = re.compile(r"^\s(.*)");
+
+re_parse_maintainer = re.compile(r"^\s*(\S.*\S)\s*\<([^\> \t]+)\>");
 
 changes_parse_error_exc = "Can't parse line in .changes file";
 invalid_dsc_format_exc = "Invalid .dsc file";
@@ -33,10 +39,28 @@ no_files_exc = "No Files: field in .dsc file.";
 cant_open_exc = "Can't read file.";
 unknown_hostname_exc = "Unknown hostname";
 cant_overwrite_exc = "Permission denied; can't overwrite existent file."
-       
+file_exists_exc = "Destination file exists";
+send_mail_invalid_args_exc = "Both arguments are non-null.";
+sendmail_failed_exc = "Sendmail invocation failed";
+tried_too_hard_exc = "Tried too hard to find a free filename.";
+
+# Valid components; used by extract_component_from_section() because
+# it doesn't know about Conf from it's caller.  FIXME
+
+valid_components = {
+    "main": "",
+    "contrib": "",
+    "non-free": ""
+    };
+
+default_config = "/etc/katie/katie.conf";
+default_apt_config = "/etc/katie/apt.conf";
+DefaultCnf = apt_pkg.newConfiguration();
+apt_pkg.ReadConfigFileISC(DefaultCnf,default_config);
+
 ######################################################################################
 
-def open_file(filename, mode):
+def open_file(filename, mode='r'):
     try:
        f = open(filename, mode);
     except IOError:
@@ -65,23 +89,33 @@ def str_isnum (s):
 
 ######################################################################################
 
-# What a mess.  FIXME
+# Prefix and components hardcoded into this like a good'un; need to unhardcod at some
+# stage. [FIXME]
+
 def extract_component_from_section(section):
     component = "";
-    
-    if string.find(section, '/') != -1: 
+
+    if string.find(section, '/') != -1:
         component = string.split(section, '/')[0];
     if string.lower(component) == "non-us" and string.count(section, '/') > 0:
         s = string.split(section, '/')[1];
-        if s == "main" or s == "non-free" or s == "contrib": # Avoid e.g. non-US/libs
+        if valid_components.has_key(s): # Avoid e.g. non-US/libs
             component = string.split(section, '/')[0]+ '/' + string.split(section, '/')[1];
 
     if string.lower(section) == "non-us":
         component = "non-US/main";
-            
+
+    # non-US prefix is case insensitive
+    if string.lower(component)[:6] == "non-us":
+        component = "non-US"+component[6:];
+
+    # Expand default component
     if component == "":
-        component = "main";
-    elif string.lower(component) == "non-us":
+        if valid_components.has_key(section):
+            component = section;
+        else:
+            component = "main";
+    elif component == "non-US":
         component = "non-US/main";
 
     return (section, component);
@@ -131,37 +165,40 @@ def parse_changes(filename, dsc_whitespace_rules):
                 if index > max(indices):
                     raise invalid_dsc_format_exc, index;
                 line = indexed_lines[index];
-                if not re.match('^-----BEGIN PGP SIGNATURE', line):
+                if line[:24] != "-----BEGIN PGP SIGNATURE":
                     raise invalid_dsc_format_exc, index;
                 inside_signature = 0;
                 break;
-        if re.match('^-----BEGIN PGP SIGNATURE', line):
+        if line[:24] == "-----BEGIN PGP SIGNATURE":
             break;
-        if re.match(r'^-----BEGIN PGP SIGNED MESSAGE', line):
+        if line[:29] == "-----BEGIN PGP SIGNED MESSAGE":
             if dsc_whitespace_rules:
                 inside_signature = 1;
                 while index < max(indices) and line != "":
                     index = index + 1;
                     line = indexed_lines[index];
             continue;
-        slf = re.match(r'^(\S*)\s*:\s*(.*)', line);
+        slf = re_single_line_field.match(line);
         if slf:
             field = string.lower(slf.groups()[0]);
             changes[field] = slf.groups()[1];
+           first = 1;
             continue;
-        mld = re.match(r'^ \.$', line);
-        if mld:
+        if line == " .":
             changes[field] = changes[field] + '\n';
             continue;
-        mlf = re.match(r'^\s(.*)', line);
+        mlf = re_multi_line_field.match(line);
         if mlf:
+            if first == 1 and changes[field] != "":
+                changes[field] = changes[field] + '\n';
+            first = 0;
            changes[field] = changes[field] + mlf.groups()[0] + '\n';
             continue;
        error = error + line;
 
     if dsc_whitespace_rules and inside_signature:
         raise invalid_dsc_format_exc, index;
-        
+
     changes_in.close();
     changes["filecontents"] = string.join (lines, "");
 
@@ -180,12 +217,12 @@ def build_file_list(changes, dsc):
     if format != "":
        format = float(format)
     if dsc == "" and (format < 1.5 or format > 2.0):
-       raise nk_format_exc, changes["format"];
+       raise nk_format_exc, format;
 
     # No really, this has happened.  Think 0 length .dsc file.
     if not changes.has_key("files"):
        raise no_files_exc
-    
+
     for i in string.split(changes["files"], "\n"):
         if i == "":
             break
@@ -203,7 +240,7 @@ def build_file_list(changes, dsc):
         if priority == "": priority = "-"
 
         (section, component) = extract_component_from_section(section);
-        
+
         files[name] = { "md5sum" : md5,
                         "size" : size,
                         "section": section,
@@ -220,17 +257,17 @@ def build_file_list(changes, dsc):
 # 06:28|<Culus> 'The standard sucks, but my tool is supposed to
 #                interoperate with it. I know - I'll fix the suckage
 #                and make things incompatible!'
-        
+
 def fix_maintainer (maintainer):
-    m = re.match(r"^\s*(\S.*\S)\s*\<([^\> \t]+)\>", maintainer)
+    m = re_parse_maintainer.match(maintainer);
     rfc822 = maintainer
     name = ""
     email = ""
     if m != None and len(m.groups()) == 2:
         name = m.group(1)
         email = m.group(2)
-        if re.search(r'[,.]', name) != None:
-            rfc822 = re.sub(r"^\s*(\S.*\S)\s*\<([^\> \t]+)\>", r"\2 (\1)", maintainer)
+        if string.find(name, ',') != -1 or string.find(name, '.') != -1:
+            rfc822 = re_parse_maintainer.sub(r"\2 (\1)", maintainer)
     return (rfc822, name, email)
 
 ######################################################################################
@@ -242,22 +279,23 @@ def send_mail (message, filename):
 
        # Sanity check arguments
        if message != "" and filename != "":
-               sys.stderr.write ("send_mail() can't be called with both arguments as non-null! (`%s' and `%s')\n%s" % (message, filename))
-               sys.exit(1)
+            raise send_mail_invalid_args_exc;
+
        # If we've been passed a string dump it into a temporary file
        if message != "":
-               filename = tempfile.mktemp()
-               fd = os.open(filename, os.O_RDWR|os.O_CREAT|os.O_EXCL, 0700)
-               os.write (fd, message)
-               os.close (fd)
+            filename = tempfile.mktemp();
+            fd = os.open(filename, os.O_RDWR|os.O_CREAT|os.O_EXCL, 0700);
+            os.write (fd, message);
+            os.close (fd);
+
        # Invoke sendmail
-       (result, output) = commands.getstatusoutput("%s < %s" % (sendmail_command, filename))
+       (result, output) = commands.getstatusoutput("%s < %s" % (sendmail_command, filename));
        if (result != 0):
-               sys.stderr.write ("Sendmail invocation (`%s') failed for `%s'!\n%s" % (sendmail_command, filename, output))
-               sys.exit(result)
+            raise sendmail_failed_exc, output;
+
        # Clean up any temporary files
        if message !="":
-               os.unlink (filename)
+            os.unlink (filename);
 
 ######################################################################################
 
@@ -274,7 +312,7 @@ def poolify (source, component):
 
 ######################################################################################
 
-def move (src, dest):
+def move (src, dest, overwrite = 0):
     if os.path.exists(dest) and os.path.isdir(dest):
        dest_dir = dest;
     else:
@@ -286,14 +324,18 @@ def move (src, dest):
     #print "Moving %s to %s..." % (src, dest);
     if os.path.exists(dest) and os.path.isdir(dest):
        dest = dest + '/' + os.path.basename(src);
-    # Check for overwrite permission on existent files
-    if os.path.exists(dest) and not os.access(dest, os.W_OK):
-        raise cant_overwrite_exc
+    # Don't overwrite unless forced to
+    if os.path.exists(dest):
+        if not overwrite:
+            raise file_exists_exc;
+        else:
+            if not os.access(dest, os.W_OK):
+                raise cant_overwrite_exc
     shutil.copy2(src, dest);
     os.chmod(dest, 0664);
     os.unlink(src);
 
-def copy (src, dest):
+def copy (src, dest, overwrite = 0):
     if os.path.exists(dest) and os.path.isdir(dest):
        dest_dir = dest;
     else:
@@ -305,49 +347,38 @@ def copy (src, dest):
     #print "Copying %s to %s..." % (src, dest);
     if os.path.exists(dest) and os.path.isdir(dest):
        dest = dest + '/' + os.path.basename(src);
-    if os.path.exists(dest) and not os.access(dest, os.W_OK):
-        raise cant_overwrite_exc
+    # Don't overwrite unless forced to
+    if os.path.exists(dest):
+        if not overwrite:
+            raise file_exists_exc
+        else:
+            if not os.access(dest, os.W_OK):
+                raise cant_overwrite_exc
     shutil.copy2(src, dest);
     os.chmod(dest, 0664);
 
 ######################################################################################
 
-# FIXME: this is inherently nasty.  Can't put this mapping in a conf
-# file because the conf file depends on the archive.. doh.  Maybe an
-# archive independent conf file is needed.
-
 def where_am_i ():
     res = socket.gethostbyaddr(socket.gethostname());
-    if res[0] == 'pandora.debian.org':
-        return 'non-US';
-    elif res[0] == 'auric.debian.org':
-        return 'ftp-master';
+    if DefaultCnf.get("Config::" + res[0] + "::DatbaseHostname"):
+       return DefaultCnf["Config::" + res[0] + "::DatabaseHostname"]
     else:
-        raise unknown_hostname_exc, res;
-
-######################################################################################
-
-# FIXME: this isn't great either.
+        return res[0];
 
 def which_conf_file ():
-    archive = where_am_i ();
-    if archive == 'non-US':
-        return '/org/non-us.debian.org/katie/katie.conf-non-US';
-    elif archive == 'ftp-master':
-        return '/org/ftp.debian.org/katie/katie.conf';
+    res = socket.gethostbyaddr(socket.gethostname());
+    if DefaultCnf.get("Config::" + res[0] + "::KatieConfig"):
+       return DefaultCnf["Config::" + res[0] + "::KatieConfig"]
     else:
-        raise unknown_hostname_exc, archive
-
-# FIXME: if the above isn't great, this can't be either :)
+       return default_config;
 
 def which_apt_conf_file ():
-    archive = where_am_i ();
-    if archive == 'non-US':
-        return '/org/non-us.debian.org/katie/apt.conf-non-US';
-    elif archive == 'ftp-master':
-        return '/org/ftp.debian.org/katie/apt.conf';
+    res = socket.gethostbyaddr(socket.gethostname());
+    if DefaultCnf.get("Config::" + res[0] + "::AptConfig"):
+       return DefaultCnf["Config::" + res[0] + "::AptConfig"]
     else:
-        raise unknown_hostname_exc, archive
+       return default_apt_config;
 
 ######################################################################################
 
@@ -361,7 +392,7 @@ def regex_safe (s):
 
 ######################################################################################
 
-# Perform a substition of template 
+# Perform a substition of template
 def TemplateSubst(Map,Template):
     for x in Map.keys():
         Template = string.replace(Template,x,Map[x]);
@@ -394,3 +425,83 @@ def size_type (c):
         c = c / 1000;
         t = " Mb";
     return ("%d%s" % (c, t))
+
+################################################################################
+
+def cc_fix_changes (changes):
+    o = changes.get("architecture", "")
+    if o != "":
+        del changes["architecture"]
+    changes["architecture"] = {}
+    for j in string.split(o):
+        changes["architecture"][j] = 1
+
+# Sort by 'have source', by source name, by source version number, by filename
+
+def changes_compare (a, b):
+    try:
+        a_changes = parse_changes(a, 0)
+    except:
+        return -1;
+
+    try:
+        b_changes = parse_changes(b, 0)
+    except:
+        return 1;
+
+    cc_fix_changes (a_changes);
+    cc_fix_changes (b_changes);
+
+    # Sort by 'have source'
+
+    a_has_source = a_changes["architecture"].get("source")
+    b_has_source = b_changes["architecture"].get("source")
+    if a_has_source and not b_has_source:
+        return -1;
+    elif b_has_source and not a_has_source:
+        return 1;
+
+    # Sort by source name
+
+    a_source = a_changes.get("source");
+    b_source = b_changes.get("source");
+    q = cmp (a_source, b_source);
+    if q:
+        return q;
+
+    # Sort by source version
+
+    a_version = a_changes.get("version");
+    b_version = b_changes.get("version");
+    q = apt_pkg.VersionCompare(a_version, b_version);
+    if q:
+        return q
+
+    # Fall back to sort by filename
+
+    return cmp(a, b);
+
+################################################################################
+
+def find_next_free (dest, too_many=100):
+    extra = 0;
+    orig_dest = dest;
+    while os.path.exists(dest) and extra < too_many:
+        dest = orig_dest + '.' + repr(extra);
+        extra = extra + 1;
+    if extra >= too_many:
+        raise tried_too_hard_exc;
+    return dest;
+
+################################################################################
+
+def result_join (original, sep = '\t'):
+    list = [];
+    for i in xrange(len(original)):
+        if original[i] == None:
+            list.append("");
+        else:
+            list.append(original[i]);
+    return string.join(list, sep);
+
+################################################################################