]> err.no Git - dak/commitdiff
lots and lots of python 2.1 changes. rene: remove bogus argument handling. katie...
authorJames Troup <james@nocrew.org>
Wed, 16 Oct 2002 02:47:32 +0000 (02:47 +0000)
committerJames Troup <james@nocrew.org>
Wed, 16 Oct 2002 02:47:32 +0000 (02:47 +0000)
33 files changed:
alyson
amber
andrea
ashley
catherine
charisma
claire.py
db_access.py
denise
fernanda.py
halle
heidi
helena
jenna
jennifer
jeri
julia
katie.py
kelly [moved from katie with 95% similarity]
lisa
logging.py
madison
melanie
natalie
neve
rene
rhona
shania
tea
test/003/test.py
test/004/test.py
utils.py
ziyi

diff --git a/alyson b/alyson
index c26273f64d09082796ae8ba3ed57079efb6d6909..831f54c4a1d2b7d385ab615c2713037869e87234 100755 (executable)
--- a/alyson
+++ b/alyson
@@ -2,7 +2,7 @@
 
 # Sync the ISC configuartion file and the SQL database
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: alyson,v 1.9 2002-05-14 15:28:53 troup Exp $
+# $Id: alyson,v 1.10 2002-10-16 02:47:32 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
@@ -20,8 +20,8 @@
 
 ################################################################################
 
-import pg, sys, string
-import utils, db_access
+import pg, sys;
+import utils, db_access;
 import apt_pkg;
 
 ################################################################################
@@ -74,7 +74,7 @@ def main ():
     for name in Cnf.SubTree("Component").List():
         Component = Cnf.SubTree("Component::%s" % (name));
         description = get(Component, "Description");
-        if string.lower(Component.get("MeetsDFSG")) == "true":
+        if Component.get("MeetsDFSG").lower() == "true":
             meets_dfsg = "true";
         else:
             meets_dfsg = "false";
@@ -110,7 +110,7 @@ def main ():
         origin = get(Suite, "Origin");
         description = get(Suite, "Description");
         projectB.query("INSERT INTO suite (suite_name, version, origin, description) VALUES ('%s', %s, %s, %s)"
-                       % (string.lower(suite), version, origin, description));
+                       % (suite.lower(), version, origin, description));
         for architecture in Cnf.ValueList("Suite::%s::Architectures" % (suite)):
             architecture_id = db_access.get_architecture_id (architecture);
             if architecture_id < 0:
@@ -147,7 +147,7 @@ def main ():
                 prefix = "";
         else:
             prefix = "";
-            component = string.replace(component, "non-US/", "");
+            component = component.replace("non-US/", "");
             if component != 'main':
                 suffix = '/' + component;
             else:
diff --git a/amber b/amber
index 0c10ea47e8724d7656b2f0e37f01460e300efed3..404b52853b98d92bafa9fe741f011f2490c0ac6e 100755 (executable)
--- a/amber
+++ b/amber
@@ -2,7 +2,7 @@
 
 # Wrapper for Debian Security team
 # Copyright (C) 2002  James Troup <james@nocrew.org>
-# $Id: amber,v 1.5 2002-07-14 15:03:26 troup Exp $
+# $Id: amber,v 1.6 2002-10-16 02:47:32 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
@@ -31,7 +31,7 @@
 
 ################################################################################
 
-import commands, os, pwd, re, string, sys, time;
+import commands, os, pwd, re, sys, time;
 import apt_pkg;
 import katie, utils;
 
@@ -95,11 +95,11 @@ def do_upload(changes_files):
         num_upload_uris = len(upload_uris.keys());
         if num_upload_uris == 0:
             utils.fubar("%s: No valid upload URI found from components (%s)."
-                        % (changes_file, string.join(components.keys(), ", ")));
+                        % (changes_file, ", ".join(components.keys())));
         elif num_upload_uris > 1:
             utils.fubar("%s: more than one upload URI (%s) from components (%s)."
-                        % (changes_file, string.join(upload_uris.keys(), ", "),
-                           string.join(components.keys(), ", ")));
+                        % (changes_file, ", ".join(upload_uris.keys()),
+                           ", ".join(components.keys())));
         upload_uri = upload_uris.keys()[0];
         # Update the file list for the upload uri
         if not uploads.has_key(upload_uri):
@@ -125,8 +125,8 @@ def do_upload(changes_files):
             return;
 
     for uri in uploads.keys():
-        (host, path) = string.split(uri, ":");
-        file_list = string.join(uploads[uri]);
+        (host, path) = uri.split(":");
+        file_list = " ".join(uploads[uri]);
         print "Uploading files to %s..." % (host);
         spawn("lftp -c 'open %s; cd %s; put %s'" % (host, path, file_list));
 
@@ -135,7 +135,7 @@ def do_upload(changes_files):
         file = utils.open_file(filename, 'a');
         for source in package_list.keys():
             for version in package_list[source].keys():
-                file.write(string.join([source, version])+'\n');
+                file.write(" ".join([source, version])+'\n');
         file.close();
 
 ######################################################################
@@ -154,7 +154,7 @@ def make_advisory(advisory_nr, changes_files):
 
        src = Katie.pkg.changes["source"];
        if src not in adv_packages:
-           adv_packages = adv_packages + [src];
+           adv_packages += [src];
 
        suites = Katie.pkg.changes["distribution"].keys();
        for suite in suites:
@@ -168,7 +168,7 @@ def make_advisory(advisory_nr, changes_files):
            size = files[file]["size"];
            poolname = Cnf["Dir::PoolRoot"] + \
                utils.poolify(src, files[file]["component"]);
-           if arch == "source" and file[-4:] == ".dsc":
+           if arch == "source" and file.endswith(".dsc"):
                dscpoolname = poolname;
            for suite in suites:
                if not updated_pkgs[suite].has_key(arch):
@@ -195,17 +195,17 @@ def make_advisory(advisory_nr, changes_files):
                     "poolname": dscpoolname };
 
     if os.environ.has_key("SUDO_UID"):
-        whoami = string.atol(os.environ["SUDO_UID"]);
+        whoami = os.environ["SUDO_UID"].atol();
     else:
         whoami = os.getuid();
     whoamifull = pwd.getpwuid(whoami);
-    username = string.split(whoamifull[4], ",")[0];
+    username = whoamifull[4].split(",")[0];
 
     Subst = {
        "__ADVISORY__": advisory_nr,
        "__WHOAMI__": username,
        "__DATE__": time.strftime("%B %d, %Y", time.gmtime(time.time())),
-       "__PACKAGE__": string.join(adv_packages,", "),
+       "__PACKAGE__": ", ".join(adv_packages),
         "__KATIE_ADDRESS__": Cnf["Dinstall::MyEmailAddress"]
         };
 
@@ -217,7 +217,7 @@ def make_advisory(advisory_nr, changes_files):
     for suite in updated_pkgs.keys():
         suite_header = "%s %s (%s)" % (Cnf["Dinstall::MyDistribution"],
                                        Cnf["Suite::%s::Version" % suite], suite);
-        adv = adv + "%s\n%s\n\n" % (suite_header, "-"*len(suite_header));
+        adv += "%s\n%s\n\n" % (suite_header, "-"*len(suite_header));
 
        arches = Cnf.ValueList("Suite::%s::Architectures" % suite);
        if "source" in arches:
@@ -226,29 +226,29 @@ def make_advisory(advisory_nr, changes_files):
             arches.remove("all");
        arches.sort();
 
-       adv = adv + "  %s was released for %s.\n\n" % (
-               string.capitalize(suite), utils.join_with_commas_and(arches));
+       adv += "  %s was released for %s.\n\n" % (
+               suite.capitalize(), utils.join_with_commas_and(arches));
 
        for a in ["source", "all"] + arches:
            if not updated_pkgs[suite].has_key(a):
                 continue;
 
            if a == "source":
-               adv = adv + "  Source archives:\n\n";
+               adv += "  Source archives:\n\n";
            elif a == "all":
-               adv = adv + "  Architecture independent packages:\n\n";
+               adv += "  Architecture independent packages:\n\n";
            else:
-               adv = adv + "  %s architecture (%s)\n\n" % (a,
+               adv += "  %s architecture (%s)\n\n" % (a,
                        Cnf["Architectures::%s" % a]);
 
            for file in updated_pkgs[suite][a].keys():
-               adv = adv + "    http://%s/%s%s\n" % (
+               adv += "    http://%s/%s%s\n" % (
                                archive, updated_pkgs[suite][a][file]["poolname"], file);
-               adv = adv + "      Size/MD5 checksum: %8s %s\n" % (
+               adv += "      Size/MD5 checksum: %8s %s\n" % (
                        updated_pkgs[suite][a][file]["size"],
                        updated_pkgs[suite][a][file]["md5"]);
-           adv = adv + "\n";
-    adv = string.rstrip(adv);
+           adv += "\n";
+    adv = adv.rstrip();
 
     Subst["__ADVISORY_TEXT__"] = adv;
 
@@ -284,7 +284,7 @@ def init():
 
     advisory_number = arguments[0];
     changes_files = arguments[1:];
-    if advisory_number[-8:] == ".changes":
+    if advisory_number.endswith(".changes"):
         utils.warn("first argument must be the advisory number.");
         usage(1);
     for file in changes_files:
@@ -295,7 +295,7 @@ def init():
 
 def yes_no(prompt):
     while 1:
-        answer = string.lower(utils.our_raw_input(prompt+" "));
+        answer = utils.our_raw_input(prompt+" ").lower();
         if answer == "y" or answer == "n":
             break;
         else:
@@ -331,7 +331,7 @@ def main():
 
     os.chdir(Cnf["Dir::Queue::Accepted"]);
     print "Installing packages into the archive...";
-    spawn("%s/katie -pa %s" % (Cnf["Dir::Katie"], string.join(changes_files)));
+    spawn("%s/kelly -pa %s" % (Cnf["Dir::Katie"], " ".join(changes_files)));
     os.chdir(Cnf["Dir::Katie"]);
     print "Updating file lists for apt-ftparchive...";
     spawn("./jenna");
diff --git a/andrea b/andrea
index 4d2894e95e59b41e14bf849fb56a78796644093b..8e3bb5c72276505aa16083005c69cbca4d8d8936 100755 (executable)
--- a/andrea
+++ b/andrea
@@ -2,7 +2,7 @@
 
 # Check for fixable discrepancies between stable and unstable
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: andrea,v 1.7 2002-05-14 15:28:53 troup Exp $
+# $Id: andrea,v 1.8 2002-10-16 02:47:32 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
@@ -21,7 +21,7 @@
 
 ################################################################################
 
-import pg, string, sys;
+import pg, sys;
 import utils, db_access;
 import apt_pkg;
 
@@ -76,7 +76,7 @@ SELECT b_src.package, b_src.version, a.arch_string
 ORDER BY b_src.package;"""
                            % (src_suite_id, arch_id, dst_suite_id, dsc_type_id, arch_id, arch_all_id, dst_suite_id, dst_suite_id));
         for i in q.getresult():
-            print string.join(i, ' ');
+            print " ".join(i);
 
 #######################################################################################
 
diff --git a/ashley b/ashley
index 1c4356424a8a3b6b963fb0ae87ea1a527c0e1378..cce91ce5f4e89720884a76d4c25e08a392fc5295 100755 (executable)
--- a/ashley
+++ b/ashley
@@ -2,7 +2,7 @@
 
 # Dump variables from a .katie file to stdout
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: ashley,v 1.6 2002-08-07 17:19:29 troup Exp $
+# $Id: ashley,v 1.7 2002-10-16 02:47:32 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
@@ -27,7 +27,7 @@
 
 ################################################################################
 
-import string, sys;
+import sys;
 import katie, utils;
 
 def main():
@@ -44,16 +44,16 @@ def main():
         print " Changes:";
         # Mandatory changes fields
         for i in [ "source", "version", "maintainer", "urgency", "changedby822", "changedbyname", "maintainername", "maintaineremail", "fingerprint" ]:
-            print "  %s: %s" % (string.capitalize(i), changes[i]);
+            print "  %s: %s" % (i.capitalize(), changes[i]);
             del changes[i];
         # Mandatory changes lists
         for i in [ "distribution", "architecture", "closes" ]:
-            print "  %s: %s" % (string.capitalize(i), string.join(changes[i].keys()));
+            print "  %s: %s" % (i.capitalize(), " ".join(changes[i].keys()));
             del changes[i];
         # Optional changes fields
         for i in [ "changed-by", "maintainer822", "filecontents", "format" ]:
             if changes.has_key(i):
-                print "  %s: %s" % (string.capitalize(i), changes[i]);
+                print "  %s: %s" % (i.capitalize(), changes[i]);
                 del changes[i];
         print;
         if changes:
@@ -63,7 +63,7 @@ def main():
         print " Dsc:";
         for i in [ "source", "version", "maintainer", "fingerprint", "uploaders" ]:
             if dsc.has_key(i):
-                print "  %s: %s" % (string.capitalize(i), dsc[i]);
+                print "  %s: %s" % (i.capitalize(), dsc[i]);
                 del dsc[i];
         print;
         if dsc:
@@ -78,7 +78,7 @@ def main():
                        "source version", "maintainer", "dbtype", "files id",
                        "new", "section", "priority" ]:
                 if files[file].has_key(i):
-                    print "   %s: %s" % (string.capitalize(i), files[file][i]);
+                    print "   %s: %s" % (i.capitalize(), files[file][i]);
                     del files[file][i];
             if files[file]:
                 utils.warn("files[%s] still has following unrecognised keys: %s" % (file, files[file].keys()));
@@ -90,12 +90,12 @@ def main():
             print "  %s:" % (file);
             # Mandatory fields
             for i in [ "size", "md5sum" ]:
-                print "   %s: %s" % (string.capitalize(i), dsc_files[file][i]);
+                print "   %s: %s" % (i.capitalize(), dsc_files[file][i]);
                 del dsc_files[file][i];
             # Optional fields
             for i in [ "files id" ]:
                 if dsc_files[file].has_key(i):
-                    print "   %s: %s" % (string.capitalize(i), dsc_files[file][i]);
+                    print "   %s: %s" % (i.capitalize(), dsc_files[file][i]);
                     del dsc_files[file][i];
             if dsc_files[file]:
                 utils.warn("dsc_files[%s] still has following unrecognised keys: %s" % (file, dsc_files[file].keys()));
index 80ff5b7202907963ae0c8c6741e7c34e8a252fbb..722f4e820ad116b045055a37ed2affb7c233a3b2 100755 (executable)
--- a/catherine
+++ b/catherine
@@ -2,7 +2,7 @@
 
 # Poolify (move packages from "legacy" type locations to pool locations)
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: catherine,v 1.16 2002-05-08 11:13:02 troup Exp $
+# $Id: catherine,v 1.17 2002-10-16 02:47:32 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
@@ -24,8 +24,8 @@
 
 ################################################################################
 
-import os, pg, stat, string, sys
-import utils, db_access
+import os, pg, stat, sys;
+import utils, db_access;
 import apt_pkg, apt_inst;
 
 ################################################################################
@@ -70,8 +70,8 @@ def poolize (q, limit, verbose, no_action):
         if (poolized_size + size) > limit and limit >= 0:
             utils.warn("Hit %s limit." % (utils.size_type(limit)));
             break;
-        poolized_size = poolized_size + size;
-        poolized_count = poolized_count + 1;
+        poolized_size += size;
+        poolized_count += 1;
         base_filename = os.path.basename(legacy_filename);
         destination_filename = base_filename;
         # Work out the source package name
@@ -79,7 +79,7 @@ def poolize (q, limit, verbose, no_action):
             control = apt_pkg.ParseSection(apt_inst.debExtractControl(utils.open_file(legacy_filename)))
             package = control.Find("Package", "");
             source = control.Find("Source", package);
-            if string.find(source, "(") != -1:
+            if source.find("(") != -1:
                 m = utils.re_extract_src_version.match(source)
                 source = m.group(1)
             # If it's a binary, we need to also rename the file to include the architecture
index cc352dc1a838fb6a597258df014c13dc1e3b63c1..cc92de44917dcbbf6bce6fc5f9f599780ee31f97 100755 (executable)
--- a/charisma
+++ b/charisma
@@ -2,7 +2,7 @@
 
 # Generate Maintainers file used by e.g. the Debian Bug Tracking System
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: charisma,v 1.14 2002-05-03 16:06:45 troup Exp $
+# $Id: charisma,v 1.15 2002-10-16 02:47:32 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
@@ -26,7 +26,7 @@
 
 ################################################################################
 
-import pg, string, sys;
+import pg, sys;
 import db_access, utils;
 import apt_pkg;
 
@@ -92,7 +92,7 @@ def main():
     db_access.init(Cnf, projectB);
 
     for suite in Cnf.SubTree("Suite").List():
-        suite = string.lower(suite);
+        suite = suite.lower();
         suite_priority = int(Cnf["Suite::%s::Priority" % (suite)]);
 
         # Source packages
@@ -132,16 +132,14 @@ def main():
     for filename in extra_files:
         file = utils.open_file(filename);
         for line in file.readlines():
-            line = string.strip(utils.re_comments.sub('', line[:-1]))
+            line = utils.re_comments.sub('', line[:-1]).strip();
             if line == "":
                 continue;
-            split = string.split(line);
+            split = line.split();
             lhs = split[0];
-            maintainer = fix_maintainer(string.join(split[1:]));
-            if string.find(lhs,'~') != -1:
-                lhs_split = string.split(lhs, '~');
-                package = lhs_split[0];
-                version = lhs_split[1];
+            maintainer = fix_maintainer(" ".join(split[1:]));
+            if lhs.find('~') != -1:
+                (package, version) = lhs.split('~');
             else:
                 package = lhs;
                 version = '*';
@@ -154,7 +152,7 @@ def main():
     package_keys = packages.keys()
     package_keys.sort()
     for package in package_keys:
-        lhs = string.join([package, packages[package]["version"]], '~');
+        lhs = "~".join([package, packages[package]["version"]]);
         print "%-30s %s" % (lhs, packages[package]["maintainer"]);
 
 ################################################################################
index 70e42f37935f7568a1d9eb515b56d1e74f7f9da0..6c0219fa82014c603cf370ad6ff4fc6afcad3a35 100755 (executable)
--- a/claire.py
+++ b/claire.py
@@ -2,7 +2,7 @@
 
 # 'Fix' stable to make debian-cd and dpkg -BORGiE users happy
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: claire.py,v 1.17 2002-06-08 00:15:57 troup Exp $
+# $Id: claire.py,v 1.18 2002-10-16 02:47:32 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
@@ -26,8 +26,8 @@
 
 ################################################################################
 
-import os, pg, re, string, sys
-import utils, db_access
+import os, pg, re, sys;
+import utils, db_access;
 import apt_pkg;
 
 ################################################################################
@@ -53,10 +53,10 @@ Create compatibility symlinks from legacy locations to the pool.
 # Relativize an absolute symlink from 'src' -> 'dest' relative to 'root'.
 # Returns fixed 'src'
 def clean_symlink (src, dest, root):
-    src = string.replace(src, root, '', 1);
-    dest = string.replace(dest, root, '', 1);
+    src = src.replace(root, '', 1);
+    dest = dest.replace(root, '', 1);
     dest = os.path.dirname(dest);
-    new_src = '../' * len(string.split(dest, '/'));
+    new_src = '../' * len(dest.split('/'));
     return new_src + src;
 
 ################################################################################
@@ -67,11 +67,11 @@ def fix_component_section (component, section):
 
     # FIXME: ugly hacks to work around override brain damage
     section = re_strip_section_prefix.sub('', section);
-    section = string.replace(string.lower(section), 'non-us', '');
+    section = section.lower().replace('non-us', '');
     if section == "main" or section == "contrib" or section == "non-free":
         section = '';
     if section != '':
-        section = section + '/';
+        section += '/';
 
     return (component, section);
 
index ed0c482047ed5cf8c3cf536f3172243b32cbe001..c4f213e4740d22cf52915b7d15fa78efea561dee 100644 (file)
@@ -1,6 +1,8 @@
+#!/usr/bin/env python
+
 # DB access fucntions
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: db_access.py,v 1.13 2002-05-03 16:06:45 troup Exp $
+# $Id: db_access.py,v 1.14 2002-10-16 02:47:32 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
@@ -18,7 +20,7 @@
 
 ############################################################################################
 
-import string, sys, time;
+import sys, time;
 
 ############################################################################################
 
@@ -140,7 +142,7 @@ def get_architecture_id (architecture):
 def get_archive_id (archive):
     global archive_id_cache
 
-    archive = string.lower(archive);
+    archive = archive.lower();
 
     if archive_id_cache.has_key(archive):
         return archive_id_cache[archive]
@@ -158,7 +160,7 @@ def get_archive_id (archive):
 def get_component_id (component):
     global component_id_cache
 
-    component = string.lower(component);
+    component = component.lower();
 
     if component_id_cache.has_key(component):
         return component_id_cache[component]
diff --git a/denise b/denise
index b3bc7a90c2c26d2431a9f89f41c829839afcd930..1566cb95466f1730cf453f9c0323915eb00507ef 100755 (executable)
--- a/denise
+++ b/denise
@@ -2,7 +2,7 @@
 
 # Output override files for apt-ftparchive and indices/
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: denise,v 1.13 2002-07-30 19:02:06 troup Exp $
+# $Id: denise,v 1.14 2002-10-16 02:47:32 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
@@ -26,8 +26,8 @@
 
 ################################################################################
 
-import pg, sys, string
-import utils, db_access
+import pg, sys;
+import utils, db_access;
 import apt_pkg;
 
 ################################################################################
@@ -101,7 +101,7 @@ def main ():
                     override_type = ".debian-installer";
                 elif type == "dsc":
                     override_type = ".src";
-                filename = "%s/override.%s.%s%s" % (Cnf["Dir::Override"], override_suite, string.replace(component, "non-US/", ""), override_type);
+                filename = "%s/override.%s.%s%s" % (Cnf["Dir::Override"], override_suite, component.replace("non-US/", ""), override_type);
                 file = utils.open_file(filename, 'w');
                 list(file, suite, component, type);
                 file.close();
@@ -126,7 +126,7 @@ def main ():
                 override_type = ".src";
             elif type == "udeb":
                 continue;
-            filename = "%s/override.%s.%s%s" % (Cnf["Dir::Override"], override_suite, string.replace(component, "non-US/", ""), override_type);
+            filename = "%s/override.%s.%s%s" % (Cnf["Dir::Override"], override_suite, component.replace("non-US/", ""), override_type);
             file = utils.open_file(filename, 'w');
             for i in q.getresult():
                 package = i[0];
index 9197a179ffb72ab1b5d54ab400f52ba1e497bc8d..0de0234be1522e147c79b5e216cb2c14aebfd108 100755 (executable)
@@ -2,7 +2,7 @@
 
 # Script to automate some parts of checking NEW packages
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: fernanda.py,v 1.3 2002-05-18 23:54:51 troup Exp $
+# $Id: fernanda.py,v 1.4 2002-10-16 02:47:32 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
@@ -90,7 +90,7 @@ def check_dsc (dsc_filename):
 def check_deb (deb_filename):
     filename = os.path.basename(deb_filename);
 
-    if filename[-5:] == ".udeb":
+    if filename.endswith(".udeb"):
        is_a_udeb = 1;
     else:
        is_a_udeb = 0;
@@ -131,9 +131,9 @@ def check_changes (changes_filename):
     changes = utils.parse_changes (changes_filename);
     files = utils.build_file_list(changes);
     for file in files.keys():
-       if file[-4:] == ".deb" or file[-5:] == ".udeb":
+       if file.endswith(".deb") or file.endswith(".udeb"):
            check_deb(file);
-        if file[-4:] == ".dsc":
+        if file.endswith(".dsc"):
             check_dsc(file);
         # else: => byhand
 
@@ -162,11 +162,11 @@ def main ():
             sys.stdout = less_fd;
 
             try:
-                if file[-8:] == ".changes":
+                if file.endswith(".changes"):
                     check_changes(file);
-                elif file[-4:] == ".deb" or file[-5:] == ".udeb":
+                elif file.endswith(".deb") or file.endswith(".udeb"):
                     check_deb(file);
-                elif file[-4:] == ".dsc":
+                elif file.endswith(".dsc"):
                     check_dsc(file);
                 else:
                     utils.fubar("Unrecognised file type: '%s'." % (file));
diff --git a/halle b/halle
index 47698ec51373a3ffe781031f4095328d0015c2b9..d218fd20b6418a282b10bfe718e79de37365bab6 100755 (executable)
--- a/halle
+++ b/halle
@@ -2,7 +2,7 @@
 
 # Remove obsolete .changes files from proposed-updates
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: halle,v 1.8 2002-06-08 00:17:53 troup Exp $
+# $Id: halle,v 1.9 2002-10-16 02:47:32 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
@@ -20,7 +20,7 @@
 
 ################################################################################
 
-import os, pg, re, sys, string;
+import os, pg, re, sys;
 import utils, db_access;
 import apt_pkg;
 
@@ -71,7 +71,7 @@ def check_changes (filename):
                 type = m.group(3);
                 if type != "dsc":
                     del files[file];
-                    num_files = num_files - 1;
+                    num_files -= 1;
                     continue;
                 arch = "source";
                 if Options["debug"]:
@@ -111,9 +111,9 @@ def check_joey (filename):
     os.chdir("%s/dists/proposed-updates" % (Cnf["Dir::Root"]));
 
     for line in file.readlines():
-        line = string.rstrip(line)
-        if string.find(line, 'install') != -1:
-            split_line = string.split(line);
+        line = line.rstrip();
+        if line.find('install') != -1:
+            split_line = line.split();
             if len(split_line) != 2:
                 utils.fubar("Parse error (not exactly 2 elements): %s" % (line));
             install_type = split_line[0];
@@ -177,9 +177,9 @@ def main ():
     init_pu();
 
     for file in arguments:
-        if file[-8:] == ".changes":
+        if file.endswith(".changes"):
             check_changes(file);
-        elif file[-5:] == ".joey":
+        elif file.endswith(".joey"):
             check_joey(file);
         else:
             utils.fubar("Unrecognised file type: '%s'." % (file));
diff --git a/heidi b/heidi
index e708cbdf81f0629c01e4bf24554437ade0eb6776..8b3579af7d1743b7857f137a83843223fd2dbacc 100755 (executable)
--- a/heidi
+++ b/heidi
@@ -2,7 +2,7 @@
 
 # Manipulate suite tags
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: heidi,v 1.14 2002-05-14 15:29:18 troup Exp $
+# $Id: heidi,v 1.15 2002-10-16 02:47:32 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
@@ -42,7 +42,7 @@
 
 #######################################################################################
 
-import pg, string, sys;
+import pg, sys;
 import apt_pkg;
 import utils, db_access, logging;
 
@@ -96,28 +96,28 @@ def set_suite (file, suite_id):
     q = projectB.query("SELECT b.package, b.version, a.arch_string, ba.id FROM binaries b, bin_associations ba, architecture a WHERE ba.suite = %s AND ba.bin = b.id AND b.architecture = a.id" % (suite_id));
     ql = q.getresult();
     for i in ql:
-        key = string.join(i[:3]);
+        key = " ".join(i[:3]);
         current[key] = i[3];
     q = projectB.query("SELECT s.source, s.version, sa.id FROM source s, src_associations sa WHERE sa.suite = %s AND sa.source = s.id" % (suite_id));
     ql = q.getresult();
     for i in ql:
-        key = string.join(i[:2]) + " source";
+        key = " ".join(i[:2]) + " source";
         current[key] = i[2];
 
     # Build up a dictionary of what should be in the suite
     desired = {};
     for line in lines:
-        split_line = string.split(string.strip(line[:-1]));
+        split_line = line[:-1].strip().split();
         if len(split_line) != 3:
             utils.warn("'%s' does not break into 'package version architecture'." % (line[:-1]));
             continue;
-        key = string.join(split_line);
+        key = " ".join(split_line);
         desired[key] = "";
 
     # Check to see which packages need removed and remove them
     for key in current.keys():
         if not desired.has_key(key):
-            (package, version, architecture) = string.split(key);
+            (package, version, architecture) = key.split();
             id = current[key];
             if architecture == "source":
                 q = projectB.query("DELETE FROM src_associations WHERE id = %s" % (id));
@@ -128,7 +128,7 @@ def set_suite (file, suite_id):
     # Check to see which packages need added and add them
     for key in desired.keys():
         if not current.has_key(key):
-            (package, version, architecture) = string.split(key);
+            (package, version, architecture) = key.split();
             id = get_id (package, version, architecture);
             if not id:
                 continue;
@@ -155,7 +155,7 @@ def process_file (file, suite, action):
     projectB.query("BEGIN WORK");
 
     for line in lines:
-        split_line = string.split(string.strip(line[:-1]));
+        split_line = line[:-1].strip().split();
         if len(split_line) != 3:
             utils.warn("'%s' does not break into 'package version architecture'." % (line[:-1]));
             continue;
@@ -219,13 +219,13 @@ def get_list (suite):
     q = projectB.query("SELECT b.package, b.version, a.arch_string FROM binaries b, bin_associations ba, architecture a WHERE ba.suite = %s AND ba.bin = b.id AND b.architecture = a.id" % (suite_id));
     ql = q.getresult();
     for i in ql:
-        print string.join(i);
+        print " ".join(i);
 
     # List source
     q = projectB.query("SELECT s.source, s.version FROM source s, src_associations sa WHERE sa.suite = %s AND sa.source = s.id" % (suite_id));
     ql = q.getresult();
     for i in ql:
-        print string.join(i) + " source";
+        print " ".join(i) + " source";
 
 #######################################################################################
 
diff --git a/helena b/helena
index 60b655c85803f23756517105098f793502ae77bc..1d2ebc58a5a7a2a8cea32b138400a101aefcf0ab 100755 (executable)
--- a/helena
+++ b/helena
@@ -2,7 +2,7 @@
 
 # Produces a report on NEW and BYHAND packages
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: helena,v 1.1 2002-06-05 00:17:22 troup Exp $
+# $Id: helena,v 1.2 2002-10-16 02:47:32 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
@@ -35,7 +35,7 @@
 
 ################################################################################
 
-import copy, glob, os, stat, string, sys, time;
+import copy, glob, os, stat, sys, time;
 import apt_pkg;
 import katie, utils;
 
@@ -56,22 +56,22 @@ def time_pp(x):
     if x < 60:
         unit="second";
     elif x < 3600:
-        x = x / 60;
+        x /= 60;
         unit="minute";
     elif x < 86400:
-        x = x / 3600;
+        x /= 3600;
         unit="hour";
     elif x < 604800:
-        x = x / 86400;
+        x /= 86400;
         unit="day";
     elif x < 2419200:
-        x = x / 604800;
+        x /= 604800;
         unit="week";
     elif x < 29030400:
-        x = x / 2419200;
+        x /= 2419200;
         unit="month";
     else:
-        x = x / 29030400;
+        x /= 29030400;
         unit="years";
     x = int(x);
     return "%s %s%s" % (x, unit, plural(x));
@@ -126,7 +126,7 @@ def process_changes_files(changes_files, type):
             ctime = os.stat(d["filename"])[stat.ST_CTIME];
             if ctime < oldest:
                 oldest = ctime;
-            have_note = have_note + (d.has_key("lisa note"));
+            have_note += (d.has_key("lisa note"));
         per_source[source]["oldest"] = oldest;
         if not have_note:
             per_source[source]["note_state"] = 0; # none
@@ -146,21 +146,21 @@ def process_changes_files(changes_files, type):
             for arch in j["architecture"].keys():
                 arches[arch] = "";
             versions[j["version"]] = "";
-        arch_list = string.join(arches.keys(), ", ");
-        version_list = string.join(versions.keys(), ", ");
+        arch_list = ", ".join(arches.keys());
+        version_list = ", ".join(versions.keys());
         if i[1]["note_state"]:
             note = " | [note]";
         else:
             note = "";
-        msg = msg + "%10s | %10s | %10s%s | %s old\n" % (source, version_list, arch_list, note, time_pp(last_modified));
+        msg += "%10s | %10s | %10s%s | %s old\n" % (source, version_list, arch_list, note, time_pp(last_modified));
 
     if msg:
         total_count = len(changes_files);
         source_count = len(per_source_items);
-        print string.upper(type)
-        print "-"*len(type)
+        print type.upper();
+        print "-"*len(type);
         print
-        print msg
+        print msg;
         print "%s %s source package%s / %s %s package%s in total." % (source_count, type, plural(source_count), total_count, type, plural(total_count));
         print
 
diff --git a/jenna b/jenna
index 8fb54eb0f4dff767c47a94636b68cb6e9d1d4c73..69ca6e07d0c2ea8f934358a0870e1886ca115999 100755 (executable)
--- a/jenna
+++ b/jenna
@@ -2,7 +2,7 @@
 
 # Generate file lists used by apt-ftparchive to generate Packages and Sources files
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: jenna,v 1.20 2002-07-14 15:02:07 troup Exp $
+# $Id: jenna,v 1.21 2002-10-16 02:47:32 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
@@ -185,7 +185,7 @@ def cleanup(packages):
             # Check to see if we have arch: all and arch: !all (ignoring source)
             num_arches = len(arches.keys());
             if arches.has_key("source"):
-                num_arches = num_arches - 1;
+                num_arches -= 1;
             # If we do, remove the duplicates
             if num_arches > 1:
                 versions = [];
@@ -281,14 +281,14 @@ def write_filelists(packages, dislocated_files):
     if not Options["Suite"]:
         suites = Cnf.SubTree("Suite").List();
     else:
-        suites = string.split(Options["Suite"]);
+        suites = Options["Suite"].split();
     for suite in map(string.lower, suites):
         if not d.has_key(suite):
             d[suite] = {};
         if not Options["Component"]:
             components = Cnf.ValueList("Suite::%s::Components" % (suite));
         else:
-            components = string.split(Options["Components"]);
+            components = Options["Components"].split();
         udeb_components = Cnf.ValueList("Suite::%s::UdebComponents" % (suite));
         udeb_components = udeb_components;
         for component in components:
@@ -301,7 +301,7 @@ def write_filelists(packages, dislocated_files):
             if not Options["Architecture"]:
                 architectures = Cnf.ValueList("Suite::%s::Architectures" % (suite));
             else:
-                architectures = string.split(Options["Architectures"]);
+                architectures = Options["Architectures"].split();
             for arch in map(string.lower, architectures):
                 if not d[suite][component].has_key(arch):
                     d[suite][component][arch] = {};
@@ -349,7 +349,7 @@ def stable_dislocation_p():
     if not Options["Suite"]:
         return 1;
     # Otherwise, look in what suites the user specified
-    suites = string.split(Options["Suite"]);
+    suites = Options["Suite"].split();
     return suites.count("stable");
 
 ################################################################################
@@ -372,7 +372,7 @@ SELECT b.id, b.package, a.arch_string, b.version, l.path, f.filename, c.name,
     AND f.location = l.id AND l.component = c.id AND ba.suite = su.id
     %s %s %s""" % (con_suites, con_architectures, con_components);
     if check_source:
-        query = query + """
+        query += """
 UNION
 SELECT s.id, s.source, 'source', s.version, l.path, f.filename, c.name, f.id,
        su.suite_name, 'dsc'
@@ -387,7 +387,7 @@ SELECT s.id, s.source, 'source', s.version, l.path, f.filename, c.name, f.id,
     for i in ql:
         (id, pkg, arch, version, path, filename, component, file_id, suite, type) = i;
         # 'id' comes from either 'binaries' or 'source', so it's not unique
-        unique_id = unique_id + 1;
+        unique_id += 1;
         packages[unique_id] = Dict(id=id, pkg=pkg, arch=arch, version=version,
                                    path=path, filename=filename,
                                    component=component, file_id=file_id,
index 1df7e7a815e0a11411e9fb80701cd5401f7fd571..f23d62a5997cd1ebee598ddad28ca1fc1fcbb359 100755 (executable)
--- a/jennifer
+++ b/jennifer
@@ -2,7 +2,7 @@
 
 # Checks Debian packages from Incoming
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: jennifer,v 1.26 2002-07-14 18:19:25 troup Exp $
+# $Id: jennifer,v 1.27 2002-10-16 02:47:32 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
@@ -29,7 +29,7 @@
 
 ################################################################################
 
-import FCNTL, errno, fcntl, gzip, os, re, select, shutil, stat, string, sys, time, traceback;
+import FCNTL, errno, fcntl, gzip, os, re, select, shutil, stat, sys, time, traceback;
 import apt_inst, apt_pkg;
 import db_access, katie, logging, utils;
 
@@ -45,7 +45,7 @@ re_valid_pkg_name = re.compile(r"^[\dA-Za-z][\dA-Za-z\+\-\.]+$");
 ################################################################################
 
 # Globals
-jennifer_version = "$Revision: 1.26 $";
+jennifer_version = "$Revision: 1.27 $";
 
 Cnf = None;
 Options = None;
@@ -152,9 +152,9 @@ def get_status_output(cmd, status_read, status_write):
             if len(r) > 0:
                 more_data.append(fd);
                 if fd == c2pwrite or fd == errin:
-                    output = output + r;
+                    output += r;
                 elif fd == status_read:
-                    status = status + r;
+                    status += r;
                 else:
                     utils.fubar("Unexpected file descriptor [%s] returned from select\n" % (fd));
         if not more_data:
@@ -180,7 +180,7 @@ def Dict(**dict): return dict
 def reject (str, prefix="Rejected: "):
     global reject_message;
     if str:
-        reject_message = reject_message + prefix + str + "\n";
+        reject_message += prefix + str + "\n";
 
 #########################################################################################
 
@@ -197,21 +197,21 @@ def check_signature (filename):
     # Process the status-fd output
     keywords = {};
     bad = internal_error = "";
-    for line in string.split(status, '\n'):
-        line = string.strip(line);
+    for line in status.split('\n'):
+        line = line.strip();
         if line == "":
             continue;
-        split = string.split(line);
+        split = line.split();
         if len(split) < 2:
-            internal_error = internal_error + "gpgv status line is malformed (< 2 atoms) ['%s'].\n" % (line);
+            internal_error += "gpgv status line is malformed (< 2 atoms) ['%s'].\n" % (line);
             continue;
         (gnupg, keyword) = split[:2];
         if gnupg != "[GNUPG:]":
-            internal_error = internal_error + "gpgv status line is malformed (incorrect prefix '%s').\n" % (gnupg);
+            internal_error += "gpgv status line is malformed (incorrect prefix '%s').\n" % (gnupg);
             continue;
         args = split[2:];
         if keywords.has_key(keyword) and keyword != "NODATA":
-            internal_error = internal_error + "found duplicate status token ('%s').\n" % (keyword);
+            internal_error += "found duplicate status token ('%s').\n" % (keyword);
             continue;
         else:
             keywords[keyword] = args;
@@ -252,7 +252,7 @@ def check_signature (filename):
     # Next check gpgv exited with a zero return code
     if exit_status:
         reject("gpgv failed while checking %s." % (filename));
-        if string.strip(status):
+        if status.strip():
             reject(utils.prefix_multi_line_string(status, " [GPG status-fd output:] "), "");
         else:
             reject(utils.prefix_multi_line_string(output, " [GPG output:] "), "");
@@ -283,7 +283,7 @@ def check_signature (filename):
 
     for keyword in keywords.keys():
         if not known_keywords.has_key(keyword):
-            reject("found unknown status token '%s' from gpgv with args '%s' in %s." % (keyword, repr(keywords[keyword]), filename));
+            reject("found unknown status token '%s' from gpgv with args '%r' in %s." % (keyword, keywords[keyword], filename));
             bad = 1;
 
     if bad:
@@ -327,7 +327,6 @@ def copy_to_holding(filename):
         raise;
 
     in_holding[base_filename] = "";
-    return dest;
 
 ################################################################################
 
@@ -338,7 +337,7 @@ def clean_holding():
     os.chdir(Cnf["Dir::Queue::Holding"]);
     for file in in_holding.keys():
         if os.path.exists(file):
-            if string.find(file, '/') != -1:
+            if file.find(file, '/') != -1:
                 utils.fubar("WTF? clean_holding() got a file ('%s') with / in it!" % (file));
             else:
                 os.unlink(file);
@@ -386,7 +385,7 @@ def check_changes():
         if o != "":
             del changes[i]
         changes[i] = {}
-        for j in string.split(o):
+        for j in o.split():
             changes[i][j] = 1
 
     # Fix the Maintainer: field to be RFC822 compatible
@@ -422,7 +421,7 @@ def check_distributions():
 
     # Handle suite mappings
     for map in Cnf.ValueList("SuiteMappings"):
-        args = string.split(map);
+        args = map.split();
         type = args[0];
         if type == "map" or type == "silent-map":
             (source, dest) = args[1:3];
@@ -554,9 +553,9 @@ def check_files():
             files[file]["architecture"] = architecture;
             files[file]["version"] = version;
             files[file]["maintainer"] = control.Find("Maintainer", "");
-            if file[-5:] == ".udeb":
+            if file.endswith(".udeb"):
                 files[file]["dbtype"] = "udeb";
-            elif file[-4:] == ".deb":
+            elif file.endswith(".deb"):
                 files[file]["dbtype"] = "deb";
             else:
                 reject("%s is neither a .deb or a .udeb." % (file));
@@ -564,7 +563,7 @@ def check_files():
             # Get the source version
             source = files[file]["source"];
             source_version = ""
-            if string.find(source, "(") != -1:
+            if source.find("(") != -1:
                 m = utils.re_extract_src_version.match(source)
                 source = m.group(1)
                 source_version = m.group(2)
@@ -654,7 +653,7 @@ def check_files():
 
             # Handle component mappings
             for map in Cnf.ValueList("ComponentMappings"):
-                (source, dest) = string.split(map);
+                (source, dest) = map.split();
                 if files[file]["component"] == source:
                     files[file]["original component"] = source;
                     files[file]["component"] = dest;
@@ -676,7 +675,7 @@ def check_files():
                 continue;
 
             # Validate the priority
-            if string.find(files[file]["priority"],'/') != -1:
+            if files[file]["priority"].find('/') != -1:
                 reject("file '%s' has invalid priority '%s' [contains '/']." % (file, files[file]["priority"]));
 
             # Determine the location
@@ -833,7 +832,7 @@ def check_urgency ():
         if changes["urgency"] not in Cnf.ValueList("Urgency::Valid"):
             reject("%s is not a valid urgency; it will be treated as %s by testing." % (changes["urgency"], Cnf["Urgency::Default"]), "Warning: ");
             changes["urgency"] = Cnf["Urgency::Default"];
-        changes["urgency"] = string.lower(changes["urgency"]);
+        changes["urgency"] = changes["urgency"].lower();
 
 ################################################################################
 
@@ -975,7 +974,7 @@ def action ():
     if Options["No-Action"] or Options["Automatic"]:
         answer = 'S'
 
-    if string.find(reject_message, "Rejected") != -1:
+    if reject_message.find("Rejected") != -1:
         if upload_too_new():
             print "SKIP (too new)\n" + reject_message,;
             prompt = "[S]kip, Quit ?";
@@ -985,7 +984,7 @@ def action ():
             if Options["Automatic"]:
                 answer = 'R';
     elif new:
-        print "NEW to %s\n%s%s" % (string.join(changes["distribution"].keys(), ", "), reject_message, summary),;
+        print "NEW to %s\n%s%s" % (", ".join(changes["distribution"].keys()), reject_message, summary),;
         prompt = "[N]ew, Skip, Quit ?";
         if Options["Automatic"]:
             answer = 'N';
@@ -1000,12 +999,12 @@ def action ():
         if Options["Automatic"]:
             answer = 'A';
 
-    while string.find(prompt, answer) == -1:
+    while prompt.find(answer) == -1:
         answer = utils.our_raw_input(prompt);
         m = katie.re_default_answer.match(prompt);
         if answer == "":
             answer = m.group(1);
-        answer = string.upper(answer[:1]);
+        answer = answer[:1].upper();
 
     if answer == 'R':
         os.chdir (pkg.directory);
@@ -1164,7 +1163,7 @@ def main():
 
     # Ensure all the arguments we were given are .changes files
     for file in changes_files:
-        if file[-8:] != ".changes":
+        if not file.endswith(".changes"):
             utils.warn("Ignoring '%s' because it's not a .changes file." % (file));
             changes_files.remove(file);
 
diff --git a/jeri b/jeri
index 2542f43491079d71946d29150ef197e78fd83fdd..9cc2635942234177ebe25b83de189a52d806e5b2 100755 (executable)
--- a/jeri
+++ b/jeri
@@ -2,7 +2,7 @@
 
 # Dependency check proposed-updates
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: jeri,v 1.9 2002-06-08 00:17:59 troup Exp $
+# $Id: jeri,v 1.10 2002-10-16 02:47:32 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
@@ -30,7 +30,7 @@
 
 ################################################################################
 
-import pg, sys, os, string
+import pg, sys, os;
 import utils, db_access
 import apt_pkg, apt_inst;
 
@@ -67,7 +67,7 @@ def pp_dep (deps):
         else:
             pp_dep = pkg;
         pp_deps.append(pp_dep);
-    return string.join(pp_deps, " |");
+    return " |".join(pp_deps);
 
 ################################################################################
 
@@ -117,25 +117,25 @@ def check_dep (depends, dep_type, check_archs, filename, files):
                 error = "%s not found. [Real: " % (pp_dep(parsed_dep))
                 if stable.has_key(dep):
                     if stable[dep].has_key(arch):
-                        error = error + "%s:%s:%s" % (dep, arch, stable[dep][arch]);
+                        error += "%s:%s:%s" % (dep, arch, stable[dep][arch]);
                     else:
-                        error = error + "%s:-:-" % (dep);
+                        error += "%s:-:-" % (dep);
                 else:
-                    error = error + "-:-:-";
-                error = error + ", Virtual: ";
+                    error += "-:-:-";
+                error += ", Virtual: ";
                 if stable_virtual.has_key(dep):
                     if stable_virtual[dep].has_key(arch):
-                        error = error + "%s:%s" % (dep, arch);
+                        error += "%s:%s" % (dep, arch);
                     else:
-                        error = error + "%s:-";
+                        error += "%s:-";
                 else:
-                    error = error + "-:-";
-                error = error + ", Upload: ";
+                    error += "-:-";
+                error += ", Upload: ";
                 if files.has_key(dep_filename):
-                    error = error + "yes";
+                    error += "yes";
                 else:
-                    error = error + "no";
-                error = error + "]";
+                    error += "no";
+                error += "]";
                 unsat.append(error);
 
             if unsat:
@@ -164,12 +164,12 @@ def check_package(filename, files):
 
     pkg_unsat = 0;
     if Pre_Depends:
-        pkg_unsat = pkg_unsat + check_dep(Pre_Depends, "pre-dependency", check_archs, base_file, files);
+        pkg_unsat += check_dep(Pre_Depends, "pre-dependency", check_archs, base_file, files);
 
     if Depends:
-        pkg_unsat = pkg_unsat + check_dep(Depends, "dependency", check_archs, base_file, files);
+        pkg_unsat += check_dep(Depends, "dependency", check_archs, base_file, files);
     #if Recommends:
-    #pkg_unsat = pkg_unsat + check_dep(Recommends, "recommendation", check_archs, base_file, files);
+    #pkg_unsat += check_dep(Recommends, "recommendation", check_archs, base_file, files);
 
     return pkg_unsat;
 
@@ -203,11 +203,11 @@ def check_changes (filename):
 
     changes_result = 0;
     for file in files.keys():
-        if file[-4:] == ".deb":
+        if file.endswith(".deb"):
             result = check_package(file, files);
             if Options["verbose"]:
                 pass_fail(file, result);
-            changes_result = changes_result + result;
+            changes_result += result;
 
     pass_fail (filename, changes_result);
 
@@ -230,9 +230,9 @@ def check_joey (filename):
     os.chdir("%s/dists/proposed-updates" % (Cnf["Dir::Root"]));
 
     for line in file.readlines():
-        line = string.rstrip(line);
-        if string.find(line, 'install') != -1:
-            split_line = string.split(line);
+        line = line.rstrip();
+        if line.find('install') != -1:
+            split_line = line.split();
             if len(split_line) != 2:
                 utils.fubar("Parse error (not exactly 2 elements): %s" % (line));
             install_type = split_line[0];
@@ -269,8 +269,8 @@ def parse_packages():
                     stable[package] = {};
                 stable[package][architecture] = version;
                 if provides:
-                    for virtual_pkg in string.split(provides,","):
-                        virtual_pkg = string.strip(virtual_pkg);
+                    for virtual_pkg in provides.split(","):
+                        virtual_pkg = virtual_pkg.strip();
                         if not stable_virtual.has_key(virtual_pkg):
                             stable_virtual[virtual_pkg] = {};
                         stable_virtual[virtual_pkg][architecture] = "NA";
@@ -307,11 +307,11 @@ def main ():
     print "done.";
 
     for file in arguments:
-        if file[-8:] == ".changes":
+        if file.endswith(".changes"):
             check_changes(file);
-        elif file[-4:] == ".deb":
+        elif file.endswith(".deb"):
             check_deb(file);
-        elif file[-5:] == ".joey":
+        elif file.endswith(".joey"):
             check_joey(file);
         else:
             utils.fubar("Unrecognised file type: '%s'." % (file));
diff --git a/julia b/julia
index e7dff3bc40dcdc24aadf857f4e3e0754ee21122e..e3548784d608f7f5f4602200b156099d751a024d 100755 (executable)
--- a/julia
+++ b/julia
@@ -2,7 +2,7 @@
 
 # Sync PostgreSQL users with system users
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: julia,v 1.7 2002-08-14 00:40:19 troup Exp $
+# $Id: julia,v 1.8 2002-10-16 02:47:32 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
@@ -30,7 +30,7 @@
 
 ################################################################################
 
-import pg, pwd, string, sys;
+import pg, pwd, sys;
 import utils;
 import apt_pkg;
 
@@ -95,8 +95,8 @@ def main ():
         postgres_unames[uname] = "";
 
     known_postgres_unames = {};
-    for i in string.split(Cnf.get("Julia::KnownPostgres",""),","):
-        uname = string.strip(i);
+    for i in Cnf.get("Julia::KnownPostgres","").split(","):
+        uname = i.strip();
         known_postgres_unames[uname] = "";
 
     keys = postgres_unames.keys()
index 2614462de74d6053dcd6d914a25e3bd317bfdbe7..996ae0283fef263e5bacc98c27031e7032adc7a0 100644 (file)
--- a/katie.py
+++ b/katie.py
@@ -2,7 +2,7 @@
 
 # Utility functions for katie
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: katie.py,v 1.26 2002-08-26 18:07:24 ajt Exp $
+# $Id: katie.py,v 1.27 2002-10-16 02:47:32 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
@@ -25,7 +25,6 @@ import utils, db_access;
 import apt_inst, apt_pkg;
 
 from types import *;
-from string import lower;
 
 ###############################################################################
 
@@ -57,7 +56,7 @@ class nmu_p:
             filename = Cnf["Dir::Override"] + Cnf["Dinstall::GroupOverrideFilename"];
             file = utils.open_file(filename);
             for line in file.readlines():
-                line = lower(string.strip(utils.re_comments.sub('', line)));
+                line = utils.re_comments.sub('', line).lower().strip();
                 if line != "":
                     self.group_maint[line] = 1;
             file.close();
@@ -67,23 +66,23 @@ class nmu_p:
         changes = pkg.changes;
         dsc = pkg.dsc;
 
-        (dsc_rfc822, dsc_name, dsc_email) = utils.fix_maintainer (lower(dsc.get("maintainer",Cnf["Dinstall::MyEmailAddress"])));
+        (dsc_rfc822, dsc_name, dsc_email) = utils.fix_maintainer (dsc.get("maintainer",Cnf["Dinstall::MyEmailAddress"]).lower());
         # changes["changedbyname"] == dsc_name is probably never true, but better safe than sorry
-        if dsc_name == lower(changes["maintainername"]) and \
-           (changes["changedby822"] == "" or lower(changes["changedbyname"]) == dsc_name):
+        if dsc_name == changes["maintainername"].lower() and \
+           (changes["changedby822"] == "" or changes["changedbyname"].lower() == dsc_name):
             return 0;
 
         if dsc.has_key("uploaders"):
-            uploaders = string.split(lower(dsc["uploaders"]), ",");
+            uploaders = dsc["uploaders"].lower().split(",");
             uploadernames = {};
             for i in uploaders:
-                (rfc822, name, email) = utils.fix_maintainer (string.strip(i));
+                (rfc822, name, email) = utils.fix_maintainer (i.strip());
                 uploadernames[name] = "";
-            if uploadernames.has_key(lower(changes["changedbyname"])):
+            if uploadernames.has_key(changes["changedbyname"].lower()):
                 return 0;
 
         # Some group maintained packages (e.g. Debian QA) are never NMU's
-        if self.group_maint.has_key(lower(changes["maintaineremail"])):
+        if self.group_maint.has_key(changes["maintaineremail"].lower()):
             return 0;
 
         return 1;
@@ -212,7 +211,7 @@ class Katie:
         if not changes.has_key("maintainer822"):
             changes["maintainer822"] = self.Cnf["Dinstall::MyEmailAddress"];
 
-        Subst["__ARCHITECTURE__"] = string.join(changes["architecture"].keys(), ' ' );
+        Subst["__ARCHITECTURE__"] = " ".join(changes["architecture"].keys());
         Subst["__CHANGES_FILENAME__"] = os.path.basename(self.pkg.changes_file);
         Subst["__FILE_CONTENTS__"] = changes.get("filecontents", "");
 
@@ -226,7 +225,7 @@ class Katie:
             Subst["__MAINTAINER_TO__"] = changes["maintainer822"];
             Subst["__MAINTAINER__"] = changes.get("maintainer", "Unknown");
         if self.Cnf.has_key("Dinstall::TrackingServer") and changes.has_key("source"):
-            Subst["__MAINTAINER_TO__"] = Subst["__MAINTAINER_TO__"] + "\nBcc: %s@%s" % (changes["source"], self.Cnf["Dinstall::TrackingServer"])
+            Subst["__MAINTAINER_TO__"] += "\nBcc: %s@%s" % (changes["source"], self.Cnf["Dinstall::TrackingServer"])
 
         # Apply any global override of the Maintainer field
         if self.Cnf.get("Dinstall::OverrideMaintainer"):
@@ -255,18 +254,18 @@ class Katie:
         for file in file_keys:
             if files[file].has_key("byhand"):
                 byhand = 1
-                summary = summary + file + " byhand\n"
+                summary += file + " byhand\n"
             elif files[file].has_key("new"):
                 new = 1
-                summary = summary + "(new) %s %s %s\n" % (file, files[file]["priority"], files[file]["section"])
+                summary += "(new) %s %s %s\n" % (file, files[file]["priority"], files[file]["section"])
                 if files[file].has_key("othercomponents"):
-                    summary = summary + "WARNING: Already present in %s distribution.\n" % (files[file]["othercomponents"])
+                    summary += "WARNING: Already present in %s distribution.\n" % (files[file]["othercomponents"])
                 if files[file]["type"] == "deb":
-                    summary = summary + apt_pkg.ParseSection(apt_inst.debExtractControl(utils.open_file(file)))["Description"] + '\n';
+                    summary += apt_pkg.ParseSection(apt_inst.debExtractControl(utils.open_file(file)))["Description"] + '\n';
             else:
                 files[file]["pool name"] = utils.poolify (changes["source"], files[file]["component"])
                 destination = self.Cnf["Dir::PoolRoot"] + files[file]["pool name"] + file
-                summary = summary + file + "\n  to " + destination + "\n"
+                summary += file + "\n  to " + destination + "\n"
 
         short_summary = summary;
 
@@ -274,9 +273,9 @@ class Katie:
         f = re_fdnic.sub("\n .\n", changes.get("changes",""));
 
         if byhand or new:
-            summary = summary + "Changes: " + f;
+            summary += "Changes: " + f;
 
-        summary = summary + self.announce(short_summary, 0)
+        summary += self.announce(short_summary, 0)
 
         return (summary, short_summary);
 
@@ -294,9 +293,9 @@ class Katie:
 
         bugs.sort();
         if not self.nmu.is_an_nmu(self.pkg):
-            summary = summary + "Closing bugs: ";
+            summary += "Closing bugs: ";
             for bug in bugs:
-                summary = summary + "%s " % (bug);
+                summary += "%s " % (bug);
                 if action:
                     Subst["__BUG_NUMBER__"] = bug;
                     if changes["distribution"].has_key("stable"):
@@ -313,18 +312,18 @@ distribution.""";
             if action:
                 self.Logger.log(["closing bugs"]+bugs);
         else:                     # NMU
-            summary = summary + "Setting bugs to severity fixed: ";
+            summary += "Setting bugs to severity fixed: ";
             control_message = "";
             for bug in bugs:
-                summary = summary + "%s " % (bug);
-                control_message = control_message + "tag %s + fixed\n" % (bug);
+                summary += "%s " % (bug);
+                control_message += "tag %s + fixed\n" % (bug);
             if action and control_message != "":
                 Subst["__CONTROL_MESSAGE__"] = control_message;
                 mail_message = utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/jennifer.bug-nmu-fixed");
                 utils.send_mail (mail_message, "");
             if action:
                 self.Logger.log(["setting bugs to fixed"]+bugs);
-        summary = summary + "\n";
+        summary += "\n";
         return summary;
 
     ###########################################################################
@@ -347,7 +346,7 @@ distribution.""";
             if list == "" or lists_done.has_key(list):
                 continue;
             lists_done[list] = 1;
-            summary = summary + "Announcing to %s\n" % (list);
+            summary += "Announcing to %s\n" % (list);
 
             if action:
                 Subst["__ANNOUNCE_LIST_ADDRESS__"] = list;
@@ -378,8 +377,8 @@ distribution.""";
         file_keys = files.keys();
         for file in file_keys:
             utils.move(file, Cnf["Dir::Queue::Accepted"]);
-            self.accept_bytes = self.accept_bytes + float(files[file]["size"])
-        self.accept_count = self.accept_count + 1;
+            self.accept_bytes += float(files[file]["size"])
+        self.accept_count += 1;
 
         # Send accept mail, announce to lists, close bugs and check for
         # override disparities
@@ -415,7 +414,7 @@ distribution.""";
             if self.pkg.orig_tar_id:
                 # Determine the .orig.tar.gz file name
                 for dsc_file in self.pkg.dsc_files.keys():
-                    if dsc_file[-12:] == ".orig.tar.gz":
+                    if dsc_file.endswith(".orig.tar.gz"):
                         filename = dsc_file;
                 dest = os.path.join(dest_dir, filename);
                 # If it doesn't exist, create a symlink
@@ -457,15 +456,15 @@ distribution.""";
             if not files[file].has_key("new") and files[file]["type"] == "deb":
                 section = files[file]["section"];
                 override_section = files[file]["override section"];
-                if lower(section) != lower(override_section) and section != "-":
+                if section.lower() != override_section.lower() and section != "-":
                     # Ignore this; it's a common mistake and not worth whining about
-                    if lower(section) == "non-us/main" and lower(override_section) == "non-us":
+                    if section.lower() == "non-us/main" and override_section.lower() == "non-us":
                         continue;
-                    summary = summary + "%s: section is overridden from %s to %s.\n" % (file, section, override_section);
+                    summary += "%s: section is overridden from %s to %s.\n" % (file, section, override_section);
                 priority = files[file]["priority"];
                 override_priority = files[file]["override priority"];
                 if priority != override_priority and priority != "-":
-                    summary = summary + "%s: priority is overridden from %s to %s.\n" % (file, priority, override_priority);
+                    summary += "%s: priority is overridden from %s to %s.\n" % (file, priority, override_priority);
 
         if summary == "":
             return;
@@ -528,18 +527,18 @@ distribution.""";
             while answer == 'E':
                 os.system("%s %s" % (editor, temp_filename))
                 file = utils.open_file(temp_filename);
-                reject_message = string.join(file.readlines());
+                reject_message = " ".join(file.readlines());
                 file.close();
                 print "Reject message:";
                 print utils.prefix_multi_line_string(reject_message,"  ");
                 prompt = "[R]eject, Edit, Abandon, Quit ?"
                 answer = "XXX";
-                while string.find(prompt, answer) == -1:
+                while prompt.find(answer) == -1:
                     answer = utils.our_raw_input(prompt);
                     m = re_default_answer.search(prompt);
                     if answer == "":
                         answer = m.group(1);
-                    answer = string.upper(answer[:1]);
+                    answer = answer[:1].upper();
             os.unlink(temp_filename);
             if answer == 'A':
                 return 1;
@@ -646,7 +645,7 @@ distribution.""";
         type_id = db_access.get_override_type_id(type);
 
         # FIXME: nasty non-US speficic hack
-        if lower(component[:7]) == "non-us/":
+        if component[:7].lower() == "non-us/":
             component = component[7:];
 
         q = self.projectB.query("SELECT s.section, p.priority FROM override o, section s, priority p WHERE package = '%s' AND suite = %s AND component = %s AND type = %s AND o.section = s.id AND o.priority = p.id"
@@ -674,8 +673,8 @@ distribution.""";
             # Unlike other rejects we add new lines first to avoid trailing
             # new lines when this message is passed back up to a caller.
             if self.reject_message:
-                self.reject_message = self.reject_message + "\n";
-            self.reject_message = self.reject_message + prefix + str;
+                self.reject_message += "\n";
+            self.reject_message += prefix + str;
 
     ################################################################################
 
@@ -686,8 +685,8 @@ distribution.""";
 
         # Check versions for each target suite
         for target_suite in self.pkg.changes["distribution"].keys():
-            must_be_newer_than = map(lower, self.Cnf.ValueList("Suite::%s::VersionChecks::MustBeNewerThan" % (target_suite)));
-            must_be_older_than = map(lower, self.Cnf.ValueList("Suite::%s::VersionChecks::MustBeOlderThan" % (target_suite)));
+            must_be_newer_than = map(string.lower, self.Cnf.ValueList("Suite::%s::VersionChecks::MustBeNewerThan" % (target_suite)));
+            must_be_older_than = map(string.lower, self.Cnf.ValueList("Suite::%s::VersionChecks::MustBeOlderThan" % (target_suite)));
             # Enforce "must be newer than target suite" even if conffile omits it
             if target_suite not in must_be_newer_than:
                 must_be_newer_than.append(target_suite);
@@ -778,7 +777,7 @@ SELECT s.version, su.suite_name FROM source s, src_associations sa, suite su
                 if ql:
                     # Ignore exact matches for .orig.tar.gz
                     match = 0;
-                    if dsc_file[-12:] == ".orig.tar.gz":
+                    if dsc_file.endswith(".orig.tar.gz"):
                         for i in ql:
                             if files.has_key(dsc_file) and \
                                int(files[dsc_file]["size"]) == int(i[0]) and \
@@ -789,7 +788,7 @@ SELECT s.version, su.suite_name FROM source s, src_associations sa, suite su
 
                     if not match:
                         self.reject("can not overwrite existing copy of '%s' already in the archive." % (dsc_file));
-            elif dsc_file[-12:] == ".orig.tar.gz":
+            elif dsc_file.endswith(".orig.tar.gz"):
                 # Check in the pool
                 q = self.projectB.query("SELECT l.path, f.filename, l.type, f.id, l.id FROM files f, location l WHERE (f.filename ~ '/%s$' OR f.filename = '%s') AND l.id = f.location" % (utils.regex_safe(dsc_file), dsc_file));
                 ql = q.getresult();
diff --git a/katie b/kelly
similarity index 95%
rename from katie
rename to kelly
index d98efce2b4dc0a7295997ed1f798fcdf6942a2c8..80f1b204c272f9a65cd86642c5ddf4bfe717e505 100755 (executable)
--- a/katie
+++ b/kelly
@@ -2,7 +2,7 @@
 
 # Installs Debian packages
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: katie,v 1.85 2002-08-07 17:19:25 troup Exp $
+# $Id: kelly,v 1.1 2002-10-16 02:47:32 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
 
 ###############################################################################
 
-import FCNTL, fcntl, os, string, sys, time;
+import FCNTL, fcntl, os, sys, time;
 import apt_pkg;
 import db_access, katie, logging, utils;
 
 ###############################################################################
 
 # Globals
-katie_version = "$Revision: 1.85 $";
+kelly_version = "$Revision: 1.1 $";
 
 Cnf = None;
 Options = None;
@@ -71,7 +71,7 @@ class Urgency_Log:
     def __init__ (self, Cnf):
         "Initialize a new Urgency Logger object"
         self.Cnf = Cnf;
-        self.timestamp = time.strftime("%Y%m%d%H%M%S", time.localtime(time.time()));
+        self.timestamp = time.strftime("%Y%m%d%H%M%S");
         # Create the log directory if it doesn't exist
         self.log_dir = Cnf["Dir::UrgencyLog"];
         if not os.path.exists(self.log_dir):
@@ -84,9 +84,9 @@ class Urgency_Log:
 
     def log (self, source, version, urgency):
         "Log an event"
-        self.log_file.write(string.join([source, version, urgency])+'\n');
+        self.log_file.write(" ".join([source, version, urgency])+'\n');
         self.log_file.flush();
-        self.writes = self.writes + 1;
+        self.writes += 1;
 
     def close (self):
         "Close a Logger object"
@@ -103,10 +103,10 @@ class Urgency_Log:
 def reject (str, prefix="Rejected: "):
     global reject_message;
     if str:
-        reject_message = reject_message + prefix + str + "\n";
+        reject_message += prefix + str + "\n";
 
 # Recheck anything that relies on the database; since that's not
-# frozen between accept and katie's run time.
+# frozen between accept and our run time.
 
 def check():
     for file in files.keys():
@@ -188,7 +188,7 @@ def action ():
     if Options["No-Action"] or Options["Automatic"]:
         answer = 'S'
 
-    if string.find(reject_message, "Rejected") != -1:
+    if reject_message.find("Rejected") != -1:
         print "REJECT\n" + reject_message,;
         prompt = "[R]eject, Skip, Quit ?";
         if Options["Automatic"]:
@@ -199,12 +199,12 @@ def action ():
         if Options["Automatic"]:
             answer = 'I';
 
-    while string.find(prompt, answer) == -1:
+    while prompt.find(answer) == -1:
         answer = utils.our_raw_input(prompt);
         m = katie.re_default_answer.match(prompt);
         if answer == "":
             answer = m.group(1);
-        answer = string.upper(answer[:1]);
+        answer = answer[:1].upper();
 
     if answer == 'R':
         do_reject ();
@@ -227,7 +227,7 @@ def do_reject ():
     Subst["__REJECTOR_ADDRESS__"] = Cnf["Dinstall::MyEmailAddress"];
     Subst["__REJECT_MESSAGE__"] = reject_message;
     Subst["__CC__"] = "Cc: " + Cnf["Dinstall::MyEmailAddress"];
-    reject_mail_message = utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/katie.unaccept");
+    reject_mail_message = utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/kelly.unaccept");
 
     # Write the rejection email out as the <foo>.reason file
     reason_filename = os.path.basename(pkg.changes_file[:-8]) + ".reason";
@@ -261,10 +261,10 @@ def install ():
             package = dsc["source"]
             version = dsc["version"]  # NB: not files[file]["version"], that has no epoch
             maintainer = dsc["maintainer"]
-            maintainer = string.replace(maintainer, "'", "\\'")
+            maintainer = maintainer.replace("'", "\\'")
             maintainer_id = db_access.get_or_set_maintainer_id(maintainer);
             fingerprint_id = db_access.get_or_set_fingerprint_id(dsc["fingerprint"]);
-            install_date = time.strftime("%Y-%m-%d", time.localtime(time.time()));
+            install_date = time.strftime("%Y-%m-%d");
             filename = files[file]["pool name"] + file;
             dsc_location_id = files[file]["location id"];
             if not files[file].has_key("files id") or not files[file]["files id"]:
@@ -296,7 +296,7 @@ def install ():
             package = files[file]["package"]
             version = files[file]["version"]
             maintainer = files[file]["maintainer"]
-            maintainer = string.replace(maintainer, "'", "\\'")
+            maintainer = maintainer.replace("'", "\\'")
             maintainer_id = db_access.get_or_set_maintainer_id(maintainer);
             fingerprint_id = db_access.get_or_set_fingerprint_id(changes["fingerprint"]);
             architecture = files[file]["architecture"]
@@ -335,7 +335,7 @@ def install ():
             if legacy_source_untouchable.has_key(qid["files_id"]):
                 continue;
             # First move the files to the new location
-            legacy_filename = qid["path"]+qid["filename"];
+            legacy_filename = qid["path"] + qid["filename"];
             pool_location = utils.poolify (changes["source"], files[file]["component"]);
             pool_filename = pool_location + os.path.basename(qid["filename"]);
             destination = Cnf["Dir::Pool"] + pool_location
@@ -366,7 +366,7 @@ def install ():
         destination = Cnf["Dir::Pool"] + files[file]["pool name"] + file;
         utils.move(file, destination);
         Logger.log(["installed", file, files[file]["type"], files[file]["size"], files[file]["architecture"]]);
-        install_bytes = install_bytes + float(files[file]["size"]);
+        install_bytes += float(files[file]["size"]);
 
     # Copy the .changes file across for suite which need it.
     copy_changes_p = copy_katie_p = 0;
@@ -400,7 +400,7 @@ def install ():
     for suite in changes["distribution"].keys():
         if suite not in Cnf.ValueList("Dinstall::AcceptedAutoBuildSuites"):
             continue;
-        now_date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()));
+        now_date = time.strftime("%Y-%m-%d %H:%M");
         suite_id = db_access.get_suite_id(suite);
         dest_dir = Cnf["Dir::AcceptedAutoBuild"];
         if Cnf.FindB("Dinstall::SecurityAcceptedAutoBuild"):
@@ -420,14 +420,14 @@ def install ():
         if orig_tar_id:
             # Determine the .orig.tar.gz file name
             for dsc_file in dsc_files.keys():
-                if dsc_file[-12:] == ".orig.tar.gz":
+                if dsc_file.endswith(".orig.tar.gz"):
                     orig_tar_gz = os.path.join(dest_dir, dsc_file);
             # Remove it from the list of packages for later processing by apt-ftparchive
             projectB.query("UPDATE accepted_autobuild SET in_accepted = 'f', last_used = '%s' WHERE filename = '%s' AND suite = %s" % (now_date, orig_tar_gz, suite_id));
     projectB.query("COMMIT WORK");
 
     # Finally...
-    install_count = install_count + 1;
+    install_count += 1;
 
 ################################################################################
 
@@ -498,12 +498,12 @@ def stable_install (summary, short_summary):
         os.unlink(changelog_filename);
     utils.move(new_changelog_filename, changelog_filename);
 
-    install_count = install_count + 1;
+    install_count += 1;
 
     if not Options["No-Mail"] and changes["architecture"].has_key("source"):
         Subst["__SUITE__"] = " into stable";
         Subst["__SUMMARY__"] = summary;
-        mail_message = utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/katie.installed");
+        mail_message = utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/kelly.installed");
         utils.send_mail(mail_message, "");
         Katie.announce(short_summary, 1)
 
@@ -555,7 +555,7 @@ def main():
         usage();
 
     if Options["Version"]:
-        print "katie %s" % (katie_version);
+        print "kelly %s" % (kelly_version);
         sys.exit(0);
 
     # -n/--dry-run invalidates some other options which would involve things happening
@@ -568,7 +568,7 @@ def main():
         utils.fubar("Archive maintenance in progress.  Try again later.");
 
     # If running from within proposed-updates; assume an install to stable
-    if string.find(os.getcwd(), 'proposed-updates') != -1:
+    if os.getcwd().find('proposed-updates') != -1:
         installing_to_stable = 1;
 
     # Obtain lock if not in no-action mode and initialize the log
@@ -580,7 +580,7 @@ def main():
             Urgency_Logger = Urgency_Log(Cnf);
 
     # Initialize the substitution template mapping global
-    bcc = "X-Katie: %s" % (katie_version);
+    bcc = "X-Katie: %s" % (kelly_version);
     if Cnf.has_key("Dinstall::Bcc"):
         Subst["__BCC__"] = bcc + "\nBcc: %s" % (Cnf["Dinstall::Bcc"]);
     else:
diff --git a/lisa b/lisa
index d961ede0e8edbdb4224f9f2efb056f658a0fd755..e81ea710a16878c980a728afafe3097442eea86b 100755 (executable)
--- a/lisa
+++ b/lisa
@@ -2,7 +2,7 @@
 
 # Handles NEW and BYHAND packages
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: lisa,v 1.17 2002-05-23 12:19:05 troup Exp $
+# $Id: lisa,v 1.18 2002-10-16 02:47:32 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
 
 ################################################################################
 
-import copy, errno, os, readline, string, stat, sys, tempfile;
+import copy, errno, os, readline, stat, sys, tempfile;
 import apt_pkg, apt_inst;
 import db_access, fernanda, katie, logging, utils;
 
 # Globals
-lisa_version = "$Revision: 1.17 $";
+lisa_version = "$Revision: 1.18 $";
 
 Cnf = None;
 Options = None;
@@ -62,7 +62,7 @@ reject_message = "";
 def reject (str, prefix="Rejected: "):
     global reject_message;
     if str:
-        reject_message = reject_message + prefix + str + "\n";
+        reject_message += prefix + str + "\n";
 
 def recheck():
     global reject_message;
@@ -97,12 +97,12 @@ def recheck():
         print "REJECT\n" + reject_message,;
         prompt = "[R]eject, Skip, Quit ?";
 
-        while string.find(prompt, answer) == -1:
+        while prompt.find(answer) == -1:
             answer = utils.our_raw_input(prompt);
             m = katie.re_default_answer.match(prompt);
             if answer == "":
                 answer = m.group(1);
-            answer = string.upper(answer[:1]);
+            answer = answer[:1].upper();
 
         if answer == 'R':
             Katie.do_reject(0, reject_message);
@@ -253,7 +253,7 @@ def sort_changes(changes_files):
             ctime = os.stat(d["filename"])[stat.ST_CTIME];
             if ctime < oldest:
                 oldest = ctime;
-            have_note = have_note + (d.has_key("lisa note"));
+            have_note += (d.has_key("lisa note"));
         per_source[source]["oldest"] = oldest;
         if not have_note:
             per_source[source]["note_state"] = 0; # none
@@ -335,20 +335,20 @@ def print_new (new, indexed, file=sys.stdout):
     broken = 0;
     index = 0;
     for pkg in new.keys():
-        index = index + 1;
+        index += 1;
         section = new[pkg]["section"];
         priority = new[pkg]["priority"];
         if new[pkg]["section id"] == -1:
-            section = section + "[!]";
+            section += "[!]";
             broken = 1;
         if new[pkg]["priority id"] == -1:
-            priority = priority + "[!]";
+            priority += "[!]";
             broken = 1;
         if indexed:
             line = "(%s): %-20s %-20s %-20s" % (index, pkg, priority, section);
         else:
             line = "%-20s %-20s %-20s" % (pkg, priority, section);
-        line = string.strip(line)+'\n';
+        line = line.strip()+'\n';
         file.write(line);
     note = Katie.pkg.changes.get("lisa note");
     if note:
@@ -406,10 +406,10 @@ def edit_new (new):
     os.unlink(temp_filename);
     # Parse the new data
     for line in lines:
-        line = string.strip(line[:-1]);
+        line = line.strip();
         if line == "":
             continue;
-        s = string.split(line);
+        s = line.split();
         # Pad the list if necessary
         s[len(s):3] = [None] * (3-len(s));
         (pkg, priority, section) = s[:3];
@@ -417,9 +417,9 @@ def edit_new (new):
             utils.warn("Ignoring unknown package '%s'" % (pkg));
         else:
             # Strip off any invalid markers, print_new will readd them.
-            if section[-3:] == "[!]":
+            if section.endswith("[!]"):
                 section = section[:-3];
-            if priority[-3:] == "[!]":
+            if priority.endswith("[!]"):
                 priority = priority[:-3];
             for file in new[pkg]["files"]:
                 Katie.pkg.files[file]["section"] = section;
@@ -435,7 +435,7 @@ def edit_index (new, index):
     type = new[index]["type"];
     done = 0
     while not done:
-        print string.join([index, priority, section], '\t');
+        print "\t".join([index, priority, section]);
 
         answer = "XXX";
         if type != "dsc":
@@ -444,12 +444,12 @@ def edit_index (new, index):
             prompt = "[S]ection, Done ? ";
         edit_priority = edit_section = 0;
 
-        while string.find(prompt, answer) == -1:
+        while prompt.find(answer) == -1:
             answer = utils.our_raw_input(prompt);
             m = katie.re_default_answer.match(prompt)
             if answer == "":
                 answer = m.group(1)
-            answer = string.upper(answer[:1])
+            answer = answer[:1].upper()
 
         if answer == 'P':
             edit_priority = 1;
@@ -465,7 +465,7 @@ def edit_index (new, index):
             readline.set_completer(Priorities.complete);
             got_priority = 0;
             while not got_priority:
-                new_priority = string.strip(utils.our_raw_input("New priority: "));
+                new_priority = utils.our_raw_input("New priority: ").strip();
                 if Priorities.priorities.count(new_priority) == 0:
                     print "E: '%s' is not a valid priority, try again." % (new_priority);
                 else:
@@ -477,7 +477,7 @@ def edit_index (new, index):
             readline.set_completer(Sections.complete);
             got_section = 0;
             while not got_section:
-                new_section = string.strip(utils.our_raw_input("New section: "));
+                new_section = utils.our_raw_input("New section: ").strip();
                 if Sections.sections.count(new_section) == 0:
                     print "E: '%s' is not a valid section, try again." % (new_section);
                 else:
@@ -504,7 +504,7 @@ def edit_overrides (new):
         new_index = {};
         index = 0;
         for i in new.keys():
-            index = index + 1;
+            index += 1;
             new_index[index] = i;
 
         prompt = "(%s) edit override <n>, Editor, Done ? " % (index_range(index));
@@ -512,7 +512,7 @@ def edit_overrides (new):
         got_answer = 0
         while not got_answer:
             answer = utils.our_raw_input(prompt);
-            answer = string.upper(answer[:1]);
+            answer = answer[:1].upper();
             if answer == "E" or answer == "D":
                 got_answer = 1;
             elif katie.re_isanum.match (answer):
@@ -546,18 +546,18 @@ def edit_note(note):
     while answer == 'E':
         os.system("%s %s" % (editor, temp_filename))
         temp_file = utils.open_file(temp_filename);
-        note = string.rstrip(temp_file.read());
+        note = temp_file.read().rstrip();
         temp_file.close();
         print "Note:";
         print utils.prefix_multi_line_string(note,"  ");
         prompt = "[D]one, Edit, Abandon, Quit ?"
         answer = "XXX";
-        while string.find(prompt, answer) == -1:
+        while prompt.find(answer) == -1:
             answer = utils.our_raw_input(prompt);
             m = katie.re_default_answer.search(prompt);
             if answer == "":
                 answer = m.group(1);
-            answer = string.upper(answer[:1]);
+            answer = answer[:1].upper();
     os.unlink(temp_filename);
     if answer == 'A':
         return;
@@ -605,9 +605,9 @@ def do_bxa_notification():
     for file in files.keys():
         if files[file]["type"] == "deb":
             control = apt_pkg.ParseSection(apt_inst.debExtractControl(utils.open_file(file)));
-            summary = summary + "\n";
-            summary = summary + "Package: %s\n" % (control.Find("Package"));
-            summary = summary + "Description: %s\n" % (control.Find("Description"));
+            summary += "\n";
+            summary += "Package: %s\n" % (control.Find("Package"));
+            summary += "Description: %s\n" % (control.Find("Description"));
     Katie.Subst["__BINARY_DESCRIPTIONS__"] = summary;
     bxa_mail = utils.TemplateSubst(Katie.Subst,Cnf["Dir::Templates"]+"/lisa.bxa_notification");
     utils.send_mail(bxa_mail,"");
@@ -681,16 +681,16 @@ def do_new():
             print "W: [!] marked entries must be fixed before package can be processed.";
         if note:
             print "W: note must be removed before package can be processed.";
-            prompt = prompt + "Remove note, ";
+            prompt += "Remove note, ";
 
-        prompt = prompt + "Edit overrides, Check, Manual reject, Note edit, [S]kip, Quit ?";
+        prompt += "Edit overrides, Check, Manual reject, Note edit, [S]kip, Quit ?";
 
-        while string.find(prompt, answer) == -1:
+        while prompt.find(answer) == -1:
             answer = utils.our_raw_input(prompt);
             m = katie.re_default_answer.search(prompt);
             if answer == "":
                 answer = m.group(1)
-            answer = string.upper(answer[:1])
+            answer = answer[:1].upper()
 
         if answer == 'A':
             done = add_overrides (new);
@@ -706,7 +706,7 @@ def do_new():
         elif answer == 'N':
             edit_note(changes.get("lisa note", ""));
         elif answer == 'R':
-            confirm = string.lower(utils.our_raw_input("Really clear note (y/N)? "));
+            confirm = utils.our_raw_input("Really clear note (y/N)? ").lower();
             if confirm == "y":
                 del changes["lisa note"];
         elif answer == 'S':
@@ -794,12 +794,12 @@ def do_byhand():
         else:
             prompt = "Manual reject, [S]kip, Quit ?";
 
-        while string.find(prompt, answer) == -1:
+        while prompt.find(answer) == -1:
             answer = utils.our_raw_input(prompt);
             m = katie.re_default_answer.search(prompt);
             if answer == "":
                 answer = m.group(1);
-            answer = string.upper(answer[:1]);
+            answer = answer[:1].upper();
 
         if answer == 'A':
             done = 1;
index 1952ca9fc803fc5e3cf3656205c3dbd9d472270a..ae8ee6f2c81b7a895cfb9d8020243f8affe34016 100644 (file)
@@ -1,6 +1,8 @@
+#!/usr/bin/env python
+
 # Logging functions
 # Copyright (C) 2001, 2002  James Troup <james@nocrew.org>
-# $Id: logging.py,v 1.2 2002-05-08 11:17:45 troup Exp $
+# $Id: logging.py,v 1.3 2002-10-16 02:47:32 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
@@ -18,8 +20,8 @@
 
 ################################################################################
 
-import os, pwd, string, time
-import utils
+import os, pwd, time;
+import utils;
 
 ################################################################################
 
@@ -39,7 +41,7 @@ class Logger:
             umask = os.umask(00000);
             os.makedirs(logdir, 02775);
         # Open the logfile
-        logfilename = "%s/%s" % (logdir, time.strftime("%Y-%m", time.localtime(time.time())));
+        logfilename = "%s/%s" % (logdir, time.strftime("%Y-%m"));
         logfile = utils.open_file(logfilename, 'a');
         # Seek to the end of the logfile
         logfile.seek(0,2);
@@ -52,12 +54,12 @@ class Logger:
         "Log an event"
         # Prepend the timestamp and program name
         details.insert(0, self.program);
-        timestamp = time.strftime("%Y%m%d%H%M%S", time.localtime(time.time()));
+        timestamp = time.strftime("%Y%m%d%H%M%S");
         details.insert(0, timestamp);
         # Force the contents of the list to be string.join-able
         details = map(str, details);
         # Write out the log in TSV
-        self.logfile.write(string.join(details, '~')+'\n');
+        self.logfile.write("~".join(details)+'\n');
         # Flush the output to enable tail-ing
         self.logfile.flush();
 
diff --git a/madison b/madison
index 39a389dc847bfd92e0e0cc23d583c9f7ae5200b7..34532c8ddeb1963d7f6c1f93be3e4d40de1b30a9 100755 (executable)
--- a/madison
+++ b/madison
@@ -2,7 +2,7 @@
 
 # Display information about package(s) (suite, version, etc.)
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: madison,v 1.20 2002-07-14 15:02:07 troup Exp $
+# $Id: madison,v 1.21 2002-10-16 02:47:32 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
@@ -27,7 +27,7 @@
 
 ################################################################################
 
-import pg, string, sys;
+import pg, sys;
 import utils, db_access;
 import apt_pkg;
 
@@ -119,7 +119,7 @@ def main ():
             ql.extend(q.getresult());
         d = {};
         for i in ql:
-            results = results + 1;
+            results += 1;
             pkg = i[0];
             version = i[1];
             architecture = i[2];
@@ -144,7 +144,7 @@ def main ():
                     sys.stdout.write("%10s | %10s | %13s | " % (pkg, version, suite));
                     arches = d[pkg][version][suite];
                     arches.sort(arch_compare);
-                    sys.stdout.write(string.join(arches, ", "));
+                    sys.stdout.write(", ".join(arches));
                     sys.stdout.write('\n');
 
     if not results:
diff --git a/melanie b/melanie
index 4042ef195485d9659b2e724e5940e5ffd858639f..c196766689743d11f951847c7420d10152c33154 100755 (executable)
--- a/melanie
+++ b/melanie
@@ -2,7 +2,7 @@
 
 # General purpose package removal tool for ftpmaster
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: melanie,v 1.30 2002-08-29 22:56:29 troup Exp $
+# $Id: melanie,v 1.31 2002-10-16 02:47:32 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
@@ -40,7 +40,7 @@
 
 ################################################################################
 
-import commands, os, pg, re, string, sys, tempfile
+import commands, os, pg, re, sys, tempfile
 import utils, db_access
 import apt_pkg, apt_inst;
 
@@ -82,7 +82,7 @@ ARCH, BUG#, COMPONENT and SUITE can be space seperated lists, e.g.
 #  going to do now?"
 
 def game_over():
-    answer = string.lower(utils.our_raw_input("Continue (y/N)? "));
+    answer = utils.our_raw_input("Continue (y/N)? ").lower();
     if answer != "y":
         print "Aborted."
         sys.exit(1);
@@ -143,7 +143,7 @@ def main ():
     #  3) contains a '@' - assumed to be an email address, used unmofidied
     #
     carbon_copy = [];
-    for copy_to in string.split(Options.get("Carbon-Copy")):
+    for copy_to in Options.get("Carbon-Copy").split():
         if utils.str_isnum(copy_to):
             carbon_copy.append(copy_to + "@" + Cnf["Dinstall::BugServer"]);
         elif copy_to == 'package':
@@ -160,14 +160,14 @@ def main ():
         field = "b.package";
     else:
         field = "s.source";
-    con_packages = "AND %s IN (%s)" % (field, string.join(map(repr, arguments), ", "));
+    con_packages = "AND %s IN (%s)" % (field, ", ".join(map(repr, arguments)));
 
     (con_suites, con_architectures, con_components, check_source) = \
                  utils.parse_args(Options);
 
     # Additional suite checks
     suite_ids_list = [];
-    suites = string.split(Options["Suite"]);
+    suites = Options["Suite"].split();
     suites_list = utils.join_with_commas_and(suites);
     if not Options["No-Action"]:
         for suite in suites:
@@ -190,7 +190,7 @@ def main ():
         utils.warn("'source' in -a/--argument makes no sense and is ignored.");
 
     # Additional component processing
-    over_con_components = string.replace(con_components, "c.id", "component");
+    over_con_components = con_components.replace("c.id", "component");
 
     print "Working...",
     sys.stdout.flush();
@@ -223,14 +223,14 @@ def main ():
                 binary_packages[i[0]] = "";
             # Then parse each .dsc that we found earlier to see what binary packages it thinks it produces
             for i in source_packages.keys():
-                filename = string.join(source_packages[i], '/');
+                filename = "/".join(source_packages[i]);
                 try:
                     dsc = utils.parse_changes(filename);
                 except utils.cant_open_exc:
                     utils.warn("couldn't open '%s'." % (filename));
                     continue;
-                for package in string.split(dsc.get("binary"), ','):
-                    package = string.strip(package);
+                for package in dsc.get("binary").split(','):
+                    package = package.strip();
                     binary_packages[package] = "";
             # Then for each binary package: find any version in
             # unstable, check the Source: field in the deb matches our
@@ -239,7 +239,7 @@ def main ():
             for package in binary_packages.keys():
                 q = projectB.query("SELECT l.path, f.filename, b.package, b.version, a.arch_string, b.id, b.maintainer FROM binaries b, bin_associations ba, architecture a, suite su, files f, location l, component c WHERE ba.bin = b.id AND ba.suite = su.id AND b.architecture = a.id AND b.file = f.id AND f.location = l.id AND l.component = c.id %s %s %s AND b.package = '%s'" % (con_suites, con_components, con_architectures, package));
                 for i in q.getresult():
-                    filename = string.join(i[:2], '/');
+                    filename = "/".join(i[:2]);
                     control = apt_pkg.ParseSection(apt_inst.debExtractControl(utils.open_file(filename)))
                     source = control.Find("Source", control.Find("Package"));
                     source = re_strip_source_version.sub('', source);
@@ -263,7 +263,7 @@ def main ():
             utils.fubar ("vi invocation failed for `%s'!" % (temp_filename), result)
         file = utils.open_file(temp_filename);
         for line in file.readlines():
-            Options["Reason"] = Options["Reason"] + line;
+            Options["Reason"] += line;
         os.unlink(temp_filename);
 
     # Generate the summary of what's to be removed
@@ -291,15 +291,15 @@ def main ():
         versions = d[package].keys();
         versions.sort();
         for version in versions:
-            summary = summary + "%10s | %10s | %s\n" % (package, version, string.join(d[package][version], ", "));
+            summary += "%10s | %10s | %s\n" % (package, version, ", ".join(d[package][version]));
     print "Will remove the following packages from %s:" % (suites_list);
     print
     print summary
-    print "Maintainer: %s" % string.join(maintainer_list, ", ")
+    print "Maintainer: %s" % ", ".join(maintainer_list)
     if Options["Done"]:
         print "Will also close bugs: "+Options["Done"];
     if carbon_copy:
-        print "Will also send CCs to:" + string.join(carbon_copy, ", ")
+        print "Will also send CCs to:" + ", ".join(carbon_copy)
     print
     print "------------------- Reason -------------------"
     print Options["Reason"];
@@ -365,12 +365,12 @@ def main ():
         if Cnf.Find("Melanie::Bcc") != "":
             bcc.append(Cnf["Melanie::Bcc"]);
         if bcc:
-            Subst["__BCC__"] = "Bcc: " + string.join(bcc, ", ");
+            Subst["__BCC__"] = "Bcc: " + ", ".join(bcc);
         else:
             Subst["__BCC__"] = "X-Filler: 42";
-        Subst["__CC__"] = "X-Melanie: $Revision: 1.30 $";
+        Subst["__CC__"] = "X-Melanie: $Revision: 1.31 $";
         if carbon_copy:
-            Subst["__CC__"] = Subst["__CC__"] + "\nCc: " + string.join(carbon_copy, ", ");
+            Subst["__CC__"] += "\nCc: " + ", ".join(carbon_copy);
         Subst["__SUITE_LIST__"] = suites_list;
         Subst["__SUMMARY__"] = summary;
         Subst["__ADMIN_ADDRESS__"] = Cnf["Dinstall::MyAdminAddress"];
@@ -380,7 +380,7 @@ def main ():
         Archive = Cnf.SubTree("Archive::%s" % (whereami));
         Subst["__MASTER_ARCHIVE__"] = Archive["OriginServer"];
         Subst["__PRIMARY_MIRROR__"] = Archive["PrimaryMirror"];
-        for bug in string.split(Options["Done"]):
+        for bug in Options["Done"].split():
             Subst["__BUG_NUMBER__"] = bug;
             mail_message = utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/melanie.bug-close");
             utils.send_mail (mail_message, "")
diff --git a/natalie b/natalie
index cdcc75ad4631e58cf142ebcf1011120d9ad29bd2..d626130cdf6518ac03940636113d2f51a575fad6 100755 (executable)
--- a/natalie
+++ b/natalie
@@ -2,7 +2,7 @@
 
 # Manipulate override files
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: natalie,v 1.3 2002-07-14 15:01:19 troup Exp $
+# $Id: natalie,v 1.4 2002-10-16 02:47:32 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
@@ -20,7 +20,7 @@
 
 ################################################################################
 
-import pg, string, sys, time;
+import pg, sys, time;
 import utils, db_access, logging;
 import apt_pkg;
 
@@ -87,47 +87,47 @@ def process_file (file, suite, component, type, action):
     start_time = time.time();
     projectB.query("BEGIN WORK");
     for line in file.readlines():
-        line = string.strip(utils.re_comments.sub('', line[:-1]))
+        line = utils.re_comments.sub('', line[:-1]).strip();
         if line == "":
             continue;
 
         maintainer_override = None;
         if type == "dsc":
-            split_line = string.split(line, None, 2);
+            split_line = line.split(None, 2);
             if len(split_line) == 2:
                 (package, section) = split_line;
             elif len(split_line) == 3:
                 (package, section, maintainer_override) = split_line;
             else:
                 utils.warn("'%s' does not break into 'package section [maintainer-override]'." % (line));
-                c_error = c_error + 1;
+                c_error += 1;
                 continue;
             priority = "source";
         else: # binary or udeb
-            split_line = string.split(line, None, 3);
+            split_line = line.split(None, 3);
             if len(split_line) == 3:
                 (package, priority, section) = split_line;
             elif len(split_line) == 4:
                 (package, priority, section, maintainer_override) = split_line;
             else:
                 utils.warn("'%s' does not break into 'package priority section [maintainer-override]'." % (line));
-                c_error = c_error + 1;
+                c_error += 1;
                 continue;
 
         section_id = db_access.get_section_id(section);
         if section_id == -1:
             utils.warn("'%s' is not a valid section. ['%s' in suite %s, component %s]." % (section, package, suite, component));
-            c_error = c_error + 1;
+            c_error += 1;
             continue;
         priority_id = db_access.get_priority_id(priority);
         if priority_id == -1:
             utils.warn("'%s' is not a valid priority. ['%s' in suite %s, component %s]." % (priority, package, suite, component));
-            c_error = c_error + 1;
+            c_error += 1;
             continue;
 
         if new.has_key(package):
             utils.warn("Can't insert duplicate entry for '%s'; ignoring all but the first. [suite %s, component %s]" % (package, suite, component));
-            c_error = c_error + 1;
+            c_error += 1;
             continue;
         new[package] = "";
         if original.has_key(package):
@@ -136,12 +136,12 @@ def process_file (file, suite, component, type, action):
                old_section_id == section_id and \
                old_maintainer_override == maintainer_override:
                 # If it's unchanged or we're in 'add only' mode, ignore it
-                c_skipped = c_skipped + 1;
+                c_skipped += 1;
                 continue;
             else:
                 # If it's changed, delete the old one so we can
                 # reinsert it with the new information
-                c_updated = c_updated + 1;
+                c_updated += 1;
                 projectB.query("DELETE FROM override WHERE suite = %s AND component = %s AND package = '%s' AND type = %s"
                                % (suite_id, component_id, package, type_id));
                 # Log changes
@@ -153,7 +153,7 @@ def process_file (file, suite, component, type, action):
                     Logger.log(["changed maintainer override",package,old_maintainer_override,maintainer_override]);
                 update_p = 1;
         else:
-            c_added = c_added + 1;
+            c_added += 1;
             update_p = 0;
 
         if maintainer_override:
@@ -172,7 +172,7 @@ def process_file (file, suite, component, type, action):
             if not new.has_key(package):
                 projectB.query("DELETE FROM override WHERE suite = %s AND component = %s AND package = '%s' AND type = %s"
                                % (suite_id, component_id, package, type_id));
-                c_removed = c_removed + 1;
+                c_removed += 1;
                 Logger.log(["removed override",suite,component,type,package]);
 
     projectB.query("COMMIT WORK");
diff --git a/neve b/neve
index eb94e2e3e72c2ff9ff3a20d4858060df8cce2e81..59758d1ca5e649501c2b83b1c3a21b8f6dd83b69 100755 (executable)
--- a/neve
+++ b/neve
@@ -2,7 +2,7 @@
 
 # Populate the DB
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: neve,v 1.13 2002-06-09 17:33:46 troup Exp $
+# $Id: neve,v 1.14 2002-10-16 02:47:32 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
@@ -37,7 +37,7 @@
 
 ###############################################################################
 
-import commands, os, pg, re, select, string, tempfile, time;
+import commands, os, pg, re, select, tempfile, time;
 import apt_pkg;
 import db_access, utils;
 
@@ -115,9 +115,9 @@ def get_status_output(cmd, status_read, status_write):
             if len(r) > 0:
                 more_data.append(fd);
                 if fd == c2pwrite or fd == errin:
-                    output = output + r;
+                    output += r;
                 elif fd == status_read:
-                    status = status + r;
+                    status += r;
                 else:
                     utils.fubar("Unexpected file descriptor [%s] returned from select\n" % (fd));
         if not more_data:
@@ -141,7 +141,7 @@ def Dict(**dict): return dict
 def reject (str, prefix="Rejected: "):
     global reject_message;
     if str:
-        reject_message = reject_message + prefix + str + "\n";
+        reject_message += prefix + str + "\n";
 
 ###############################################################################
 
@@ -158,21 +158,21 @@ def check_signature (filename):
     # Process the status-fd output
     keywords = {};
     bad = internal_error = "";
-    for line in string.split(status, '\n'):
-        line = string.strip(line);
+    for line in status.split('\n'):
+        line = line.strip();
         if line == "":
             continue;
-        split = string.split(line);
+        split = line.split();
         if len(split) < 2:
-            internal_error = internal_error + "gpgv status line is malformed (< 2 atoms) ['%s'].\n" % (line);
+            internal_error += "gpgv status line is malformed (< 2 atoms) ['%s'].\n" % (line);
             continue;
         (gnupg, keyword) = split[:2];
         if gnupg != "[GNUPG:]":
-            internal_error = internal_error + "gpgv status line is malformed (incorrect prefix '%s').\n" % (gnupg);
+            internal_error += "gpgv status line is malformed (incorrect prefix '%s').\n" % (gnupg);
             continue;
         args = split[2:];
         if keywords.has_key(keyword) and keyword != "NODATA" and keyword != "SIGEXPIRED":
-            internal_error = internal_error + "found duplicate status token ('%s')." % (keyword);
+            internal_error += "found duplicate status token ('%s')." % (keyword);
             continue;
         else:
             keywords[keyword] = args;
@@ -218,7 +218,7 @@ def check_signature (filename):
     # Next check gpgv exited with a zero return code
     if exit_status and not keywords.has_key("NO_PUBKEY"):
         reject("gpgv failed while checking %s." % (filename));
-        if string.strip(status):
+        if status.strip():
             reject(utils.prefix_multi_line_string(status, " [GPG status-fd output:] "), "");
         else:
             reject(utils.prefix_multi_line_string(output, " [GPG output:] "), "");
@@ -250,7 +250,7 @@ def check_signature (filename):
 
     for keyword in keywords.keys():
         if not known_keywords.has_key(keyword):
-            reject("found unknown status token '%s' from gpgv with args '%s' in %s." % (keyword, repr(keywords[keyword]), filename));
+            reject("found unknown status token '%s' from gpgv with args '%r' in %s." % (keyword, keywords[keyword], filename));
             bad = 1;
 
     if bad:
@@ -304,10 +304,10 @@ def update_suites ():
     projectB.query("DELETE FROM suite")
     for suite in Cnf.SubTree("Suite").List():
         SubSec = Cnf.SubTree("Suite::%s" %(suite))
-        projectB.query("INSERT INTO suite (suite_name) VALUES ('%s')" % string.lower(suite));
+        projectB.query("INSERT INTO suite (suite_name) VALUES ('%s')" % suite.lower());
         for i in ("Version", "Origin", "Description"):
             if SubSec.has_key(i):
-                projectB.query("UPDATE suite SET %s = '%s' WHERE suite_name = '%s'" % (string.lower(i), SubSec[i], string.lower(suite)))
+                projectB.query("UPDATE suite SET %s = '%s' WHERE suite_name = '%s'" % (i.lower(), SubSec[i], suite.lower()))
         for architecture in Cnf.ValueList("Suite::%s::Architectures" % (suite)):
             architecture_id = db_access.get_architecture_id (architecture);
             projectB.query("INSERT INTO suite_architectures (suite, architecture) VALUES (currval('suite_id_seq'), %d)" % (architecture_id));
@@ -327,9 +327,9 @@ def update_priority():
 def get_or_set_files_id (filename, size, md5sum, location_id):
     global files_id_cache, files_id_serial, files_query_cache;
 
-    cache_key = string.join((filename, size, md5sum, repr(location_id)), '~')
+    cache_key = "~".join((filename, size, md5sum, repr(location_id)));
     if not files_id_cache.has_key(cache_key):
-        files_id_serial = files_id_serial + 1
+        files_id_serial += 1
         files_query_cache.write("%d\t%s\t%s\t%s\t%d\n" % (files_id_serial, filename, size, md5sum, location_id));
         files_id_cache[cache_key] = files_id_serial
 
@@ -340,7 +340,7 @@ def get_or_set_files_id (filename, size, md5sum, location_id):
 def process_sources (location, filename, suite, component, archive, dsc_dir):
     global source_cache, source_query_cache, src_associations_query_cache, dsc_files_query_cache, source_id_serial, src_associations_id_serial, dsc_files_id_serial, source_cache_for_binaries, orig_tar_gz_cache, reject_message;
 
-    suite = string.lower(suite);
+    suite = suite.lower();
     suite_id = db_access.get_suite_id(suite);
     try:
         file = utils.open_file (filename);
@@ -358,23 +358,23 @@ def process_sources (location, filename, suite, component, archive, dsc_dir):
         if reject_message:
             utils.fubar("%s: %s" % (dsc_file, reject_message));
         maintainer = Scanner.Section["maintainer"]
-        maintainer = string.replace(maintainer, "'", "\\'");
+        maintainer = maintainer.replace("'", "\\'");
         maintainer_id = db_access.get_or_set_maintainer_id(maintainer);
         directory = Scanner.Section["directory"];
         location_id = db_access.get_location_id (location, component, archive);
-        if directory[-1:] != "/":
-            directory = directory + '/';
+        if not directory.endswith("/"):
+            directory += '/';
         directory = poolify (directory, location);
-        if directory != "" and directory[-1:] != "/":
-            directory = directory + '/';
+        if directory != "" and not directory.endswith("/"):
+            directory += '/';
         no_epoch_version = utils.re_no_epoch.sub('', version);
         # Add all files referenced by the .dsc to the files table
         ids = [];
-        for line in string.split(Scanner.Section["files"],'\n'):
+        for line in Scanner.Section["files"].split('\n'):
             id = None;
-            (md5sum, size, filename) = string.split(string.strip(line));
+            (md5sum, size, filename) = line.strip().split();
             # Don't duplicate .orig.tar.gz's
-            if filename[-12:] == ".orig.tar.gz":
+            if filename.endswith(".orig.tar.gz"):
                 cache_key = "%s~%s~%s" % (filename, size, md5sum);
                 if orig_tar_gz_cache.has_key(cache_key):
                     id = orig_tar_gz_cache[cache_key];
@@ -385,25 +385,25 @@ def process_sources (location, filename, suite, component, archive, dsc_dir):
                 id = get_or_set_files_id (directory + filename, size, md5sum, location_id);
             ids.append(id);
             # If this is the .dsc itself; save the ID for later.
-            if filename[-4:] == ".dsc":
+            if filename.endswith(".dsc"):
                 files_id = id;
         filename = directory + package + '_' + no_epoch_version + '.dsc'
         cache_key = "%s~%s" % (package, version);
         if not source_cache.has_key(cache_key):
             nasty_key = "%s~%s" % (package, version)
-            source_id_serial = source_id_serial + 1;
+            source_id_serial += 1;
             if not source_cache_for_binaries.has_key(nasty_key):
                 source_cache_for_binaries[nasty_key] = source_id_serial;
             tmp_source_id = source_id_serial;
             source_cache[cache_key] = source_id_serial;
             source_query_cache.write("%d\t%s\t%s\t%d\t%d\t%s\t%s\n" % (source_id_serial, package, version, maintainer_id, files_id, install_date, fingerprint_id))
             for id in ids:
-                dsc_files_id_serial = dsc_files_id_serial + 1;
+                dsc_files_id_serial += 1;
                 dsc_files_query_cache.write("%d\t%d\t%d\n" % (dsc_files_id_serial, tmp_source_id,id));
         else:
             tmp_source_id = source_cache[cache_key];
 
-        src_associations_id_serial = src_associations_id_serial + 1;
+        src_associations_id_serial += 1;
         src_associations_query_cache.write("%d\t%d\t%d\n" % (src_associations_id_serial, suite_id, tmp_source_id))
 
     file.close();
@@ -415,7 +415,7 @@ def process_packages (location, filename, suite, component, archive):
 
     count_total = 0;
     count_bad = 0;
-    suite = string.lower(suite);
+    suite = suite.lower();
     suite_id = db_access.get_suite_id(suite);
     try:
         file = utils.open_file (filename);
@@ -427,7 +427,7 @@ def process_packages (location, filename, suite, component, archive):
         package = Scanner.Section["package"]
         version = Scanner.Section["version"]
         maintainer = Scanner.Section["maintainer"]
-        maintainer = string.replace(maintainer, "'", "\\'")
+        maintainer = maintainer.replace("'", "\\'")
         maintainer_id = db_access.get_or_set_maintainer_id(maintainer);
         architecture = Scanner.Section["architecture"]
         architecture_id = db_access.get_architecture_id (architecture);
@@ -438,7 +438,7 @@ def process_packages (location, filename, suite, component, archive):
         else:
             source = Scanner.Section["source"]
         source_version = ""
-        if string.find(source, "(") != -1:
+        if source.find("(") != -1:
             m = utils.re_extract_src_version.match(source)
             source = m.group(1)
             source_version = m.group(2)
@@ -462,19 +462,19 @@ def process_packages (location, filename, suite, component, archive):
             if not binary_cache.has_key(cache_key):
                 if not source_id:
                     source_id = "\N";
-                    count_bad = count_bad + 1;
+                    count_bad += 1;
                 else:
                     source_id = repr(source_id);
-                binaries_id_serial = binaries_id_serial + 1;
+                binaries_id_serial += 1;
                 binaries_query_cache.write("%d\t%s\t%s\t%d\t%s\t%d\t%d\t%s\t%s\n" % (binaries_id_serial, package, version, maintainer_id, source_id, architecture_id, files_id, type, fingerprint_id));
                 binary_cache[cache_key] = binaries_id_serial;
                 tmp_binaries_id = binaries_id_serial;
             else:
                 tmp_binaries_id = binary_cache[cache_key];
 
-            bin_associations_id_serial = bin_associations_id_serial + 1;
+            bin_associations_id_serial += 1;
             bin_associations_query_cache.write("%d\t%d\t%d\n" % (bin_associations_id_serial, suite_id, tmp_binaries_id));
-            count_total = count_total +1;
+            count_total += 1;
 
     file.close();
     if count_bad != 0:
diff --git a/rene b/rene
index 3ecb76c4f5c294d916d1c272da7b80d52a54dae6..7cc6cf3123e637474ae0466bd972ee4eafd99f11 100755 (executable)
--- a/rene
+++ b/rene
@@ -2,7 +2,7 @@
 
 # Check for obsolete binary packages
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: rene,v 1.14 2002-08-07 17:19:29 troup Exp $
+# $Id: rene,v 1.15 2002-10-16 02:47:32 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
@@ -26,7 +26,7 @@
 
 ################################################################################
 
-import commands, pg, os, string, sys, tempfile;
+import commands, pg, os, sys, tempfile;
 import utils, db_access;
 import apt_pkg;
 
@@ -42,13 +42,7 @@ def main ():
 
     Cnf = utils.get_conf()
 
-    Arguments = [('h',"help","Catherine::Options::Help"),
-                 ('V',"version","Catherine::Options::Version"),
-                 ('l',"limit", "Catherine::Options::Limit", "HasArg"),
-                 ('n',"no-action","Catherine::Options::No-Action"),
-                 ('v',"verbose","Catherine::Options::Verbose")];
-
-    apt_pkg.ParseCommandLine(Cnf,Arguments,sys.argv);
+    apt_pkg.ParseCommandLine(Cnf,[],sys.argv);
     projectB = pg.connect(Cnf["DB::Name"], Cnf["DB::Host"], int(Cnf["DB::Port"]));
     db_access.init(Cnf, projectB);
 
@@ -81,10 +75,10 @@ def main ():
             # Check for packages built on architectures they shouldn't be.
             if architecture != "any" and architecture != "all":
                 architectures = {};
-                for arch in string.split(architecture):
-                    architectures[string.strip(arch)] = "";
-                for binary in string.split(binaries, ','):
-                    binary = string.strip(binary);
+                for arch in architecture.split():
+                    architectures[arch.strip()] = "";
+                for binary in binaries.split(','):
+                    binary = binary.strip();
                     q = projectB.query("SELECT a.arch_string, b.version FROM binaries b, bin_associations ba, architecture a WHERE ba.suite = %s AND ba.bin = b.id AND b.architecture = a.id AND b.package = '%s'" % (suite_id, binary));
                     ql = q.getresult();
                     if not ql:
@@ -121,10 +115,10 @@ def main ():
             if src_pkgs.has_key(source):
                 print " %s is a duplicated source package (%s and %s)" % (source, source_index, src_pkgs[source]);
             src_pkgs[source] = source_index;
-            for binary in string.split(binaries, ','):
-                binary = string.strip(binary);
+            for binary in binaries.split(','):
+                binary = binary.strip();
                 if bin_pkgs.has_key(binary):
-                    print " %s is duplicated in %s and %s" % (binary, source, bin_pkgs[binary]);
+                    print " binary %s is duplicated in source packages %s and %s" % (binary, source, bin_pkgs[binary]);
                 bin_pkgs[binary] = source;
             source_binaries[source] = binaries;
 
@@ -142,7 +136,7 @@ def main ():
                 source = Packages.Section.Find('Source', "");
                 if source == "":
                     source = package;
-                if string.find(source, "(") != -1:
+                if source.find("(") != -1:
                     m = utils.re_extract_src_version.match(source)
                     source = m.group(1)
                 if not bin_pkgs.has_key(package) and not miss_src.has_key(package):
diff --git a/rhona b/rhona
index fe119520e0c543875c5110326b889f976b64446c..e0cf0f0fa4e90ec686b77de8b760879ed1df826e 100755 (executable)
--- a/rhona
+++ b/rhona
@@ -2,7 +2,7 @@
 
 # rhona, cleans up unassociated binary and source packages
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: rhona,v 1.24 2002-05-23 12:18:32 troup Exp $
+# $Id: rhona,v 1.25 2002-10-16 02:47:32 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
@@ -185,7 +185,7 @@ def clean():
 
     print "Cleaning out packages..."
 
-    date = time.strftime("%Y-%m-%d", time.localtime(time.time()));
+    date = time.strftime("%Y-%m-%d");
     dest = Cnf["Dir::Morgue"] + '/' + Cnf["Rhona::MorgueSubDir"] + '/' + date;
     if not os.path.exists(dest):
         os.mkdir(dest);
@@ -207,14 +207,14 @@ def clean():
             continue;
         if os.path.isfile(filename):
             if os.path.islink(filename):
-                count = count + 1;
+                count += 1;
                 if Options["No-Action"]:
                     print "Removing symlink %s..." % (filename);
                 else:
                     os.unlink(filename);
             else:
-                size = size + os.stat(filename)[stat.ST_SIZE];
-                count = count + 1;
+                size += os.stat(filename)[stat.ST_SIZE];
+                count += 1;
 
                 dest_filename = dest + '/' + os.path.basename(filename);
                 # If the destination file exists; try to find another filename to use
@@ -254,7 +254,7 @@ SELECT m.id FROM maintainer m
         maintainer_id = i[0];
         if not Options["No-Action"]:
             projectB.query("DELETE FROM maintainer WHERE id = %s" % (maintainer_id));
-            count = count + 1;
+            count += 1;
     projectB.query("COMMIT WORK");
 
     if count > 0:
@@ -277,7 +277,7 @@ SELECT f.id FROM fingerprint f
         fingerprint_id = i[0];
         if not Options["No-Action"]:
             projectB.query("DELETE FROM fingerprint WHERE id = %s" % (fingerprint_id));
-            count = count + 1;
+            count += 1;
     projectB.query("COMMIT WORK");
 
     if count > 0:
@@ -305,7 +305,7 @@ def clean_accepted_autobuild():
         if not Cnf.FindB("Dinstall::SecurityAcceptedAutoBuild") and not os.path.islink(filename):
             utils.fubar("%s (from accepted_autobuild) should be a symlink but isn't." % (filename));
         os.unlink(filename);
-        count = count + 1;
+        count += 1;
     projectB.query("DELETE FROM accepted_autobuild WHERE last_used <= '%s'" % (our_delete_date));
 
     if count:
@@ -332,7 +332,7 @@ def main():
     if Options["Help"]:
         usage();
 
-    now_date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()));
+    now_date = time.strftime("%Y-%m-%d %H:%M");
     delete_date = time.strftime("%Y-%m-%d %H:%M", time.localtime(time.time()-int(Cnf["Rhona::StayOfExecution"])));
 
     check_binaries();
diff --git a/shania b/shania
index f01d20f1f84290dbb3c2fbbbb18e28c2a2dc13f3..799aebd49566f0b883b768684fb22fb1e85477e0 100755 (executable)
--- a/shania
+++ b/shania
@@ -2,7 +2,7 @@
 
 # Clean incoming of old unused files
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: shania,v 1.16 2002-05-23 09:54:23 troup Exp $
+# $Id: shania,v 1.17 2002-10-16 02:47:32 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
 
 ################################################################################
 
+# <aj> Bdale, a ham-er, and the leader,
+# <aj> Willy, a GCC maintainer,
+# <aj> Lamont-work, 'cause he's the top uploader....
+# <aj>         Penguin Puff' save the day!
+# <aj> Porting code, trying to build the world,
+# <aj> Here they come just in time...
+# <aj>         The Penguin Puff' Guys!
+# <aj> [repeat]
+# <aj> Penguin Puff'!
+# <aj> willy: btw, if you don't maintain gcc you need to start, since
+#      the lyrics fit really well that way
+
+################################################################################
+
 import os, stat, sys, time;
 import utils;
 import apt_pkg;
@@ -54,7 +68,7 @@ def init ():
 
     # Ensure a directory exists to remove files to
     if not Options["No-Action"]:
-        date = time.strftime("%Y-%m-%d", time.localtime(time.time()));
+        date = time.strftime("%Y-%m-%d");
         del_dir = Cnf["Dir::Morgue"] + '/' + Cnf["Shania::MorgueSubDir"] + '/' + date;
         if not os.path.exists(del_dir):
             os.makedirs(del_dir, 02775);
@@ -106,7 +120,7 @@ def flush_orphans ():
     for i in os.listdir('.'):
         if os.path.isfile(i):
             all_files[i] = 1;
-            if i[-8:] == ".changes":
+            if i.endswith(".changes"):
                 changes_files.append(i);
 
     # Proces all .changes and .dsc files.
@@ -120,7 +134,7 @@ def flush_orphans ():
 
         dsc_files = {};
         for file in files.keys():
-            if file[-4:] == ".dsc":
+            if file.endswith(".dsc"):
                 try:
                     dsc = utils.parse_changes(file);
                     dsc_files = utils.build_file_list(dsc, is_a_dsc=1);
diff --git a/tea b/tea
index 9c5b00b786dc5633c1fa21d29fbf5851e6ef1ea3..f77bb68d2ca9259a54e82d68be1aea80a17e8ce8 100755 (executable)
--- a/tea
+++ b/tea
@@ -2,7 +2,7 @@
 
 # Sanity check the database
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: tea,v 1.20 2002-06-08 00:23:51 troup Exp $
+# $Id: tea,v 1.21 2002-10-16 02:47:32 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
@@ -25,7 +25,7 @@
 
 ################################################################################
 
-import pg, sys, os, string, stat, time
+import pg, sys, os, stat, time
 import utils, db_access
 import apt_pkg, apt_inst;
 
@@ -45,16 +45,16 @@ current_time = time.time();
 def process_dir (unused, dirname, filenames):
     global waste, db_files, excluded;
 
-    if string.find(dirname, '/disks-') != -1 or string.find(dirname, 'upgrade-') != -1:
+    if dirname.find('/disks-') != -1 or dirname.find('upgrade-') != -1:
         return;
     # hack; can't handle .changes files
-    if string.find(dirname, 'proposed-updates') != -1:
+    if dirname.find('proposed-updates') != -1:
         return;
     for name in filenames:
         filename = os.path.abspath(dirname+'/'+name);
-        filename = string.replace(filename, 'potato-proposed-updates', 'proposed-updates');
+        filename = filename.replace('potato-proposed-updates', 'proposed-updates');
         if os.path.isfile(filename) and not os.path.islink(filename) and not db_files.has_key(filename) and not excluded.has_key(filename):
-            waste = waste + os.stat(filename)[stat.ST_SIZE];
+            waste += os.stat(filename)[stat.ST_SIZE];
             print filename
 
 ################################################################################
@@ -93,7 +93,7 @@ def check_dscs():
     for component in Cnf.SubTree("Component").List():
         if component == "mixed":
             continue;
-        component = string.lower(component);
+        component = component.lower();
         list_filename = '%s%s_%s_source.list' % (Cnf["Dir::Lists"], suite, component);
         list_file = utils.open_file(list_filename);
         for line in list_file.readlines():
@@ -102,7 +102,7 @@ def check_dscs():
                 utils.parse_changes(file, dsc_whitespace_rules=1);
             except utils.invalid_dsc_format_exc, line:
                 utils.warn("syntax error in .dsc file '%s', line %s." % (file, line));
-                count = count + 1;
+                count += 1;
 
     if count:
         utils.warn("Found %s invalid .dsc files." % (count));
@@ -146,7 +146,7 @@ SELECT l.path, f.filename FROM files f, dsc_files df, location l WHERE df.source
         first_filename = "";
         broken = 0;
         for j in q2.getresult():
-            filename = j[0]+j[1];
+            filename = j[0] + j[1];
             path = os.path.dirname(filename);
             if first_path == "":
                 first_path = path;
@@ -157,7 +157,7 @@ SELECT l.path, f.filename FROM files f, dsc_files df, location l WHERE df.source
                     broken = 1;
                     print "WOAH, we got a live one here... %s [%s] {%s}" % (filename, source_id, symlink);
         if broken:
-            broken_count = broken_count + 1;
+            broken_count += 1;
     print "Found %d source packages where the source is not all in one directory." % (broken_count);
 
 ################################################################################
@@ -216,7 +216,7 @@ def check_timestamps():
             apt_inst.debExtract(file,Ent,"control.tar.gz");
             file.seek(0);
             apt_inst.debExtract(file,Ent,"data.tar.gz");
-            count = count + 1;
+            count += 1;
     print "Checked %d files (out of %d)." % (count, len(db_files.keys()));
 
 ################################################################################
@@ -249,7 +249,7 @@ def check_missing_tar_gz_in_dsc():
                 has_tar = 1;
         if not has_tar:
             utils.warn("%s has no .tar.gz in the .dsc file." % (file));
-            count = count + 1;
+            count += 1;
 
     if count:
         utils.warn("Found %s invalid .dsc files." % (count));
index c1d1c334f4e1f9d6007f390ae8eb643df5d9ff91..f8277f873e7a215f99f5894fc134786e7338a9cb 100755 (executable)
@@ -2,7 +2,7 @@
 
 # Check utils.parse_changes()'s for handling of multi-line fields
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: test.py,v 1.1 2001-05-17 01:17:54 troup Exp $
+# $Id: test.py,v 1.2 2002-10-16 02:47:32 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
@@ -32,7 +32,7 @@
 
 ################################################################################
 
-import os, string, sys
+import os, sys
 
 sys.path.append(os.path.abspath('../../'));
 
@@ -43,7 +43,7 @@ import utils
 def fail(message):
     sys.stderr.write("%s\n" % (message));
     sys.exit(1);
-    
+
 ################################################################################
 
 def main ():
@@ -57,7 +57,7 @@ def main ():
     if o != "":
         del changes["binary"]
     changes["binary"] = {}
-    for j in string.split(o):
+    for j in o.split():
         changes["binary"][j] = 1
 
     if not changes["binary"].has_key("krb5-ftpd"):
index 86aad8a697f8c5ffdcd6adcef82df79af7e1f1d0..be272ba3b93f60da9df71772cfdc407c5941c9d0 100755 (executable)
@@ -2,7 +2,7 @@
 
 # Check utils.extract_component_from_section()
 # Copyright (C) 2000  James Troup <james@nocrew.org>
-# $Id: test.py,v 1.2 2002-01-19 18:58:07 troup Exp $
+# $Id: test.py,v 1.3 2002-10-16 02:47:32 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
@@ -20,7 +20,7 @@
 
 ################################################################################
 
-import os, string, sys
+import os, sys;
 
 sys.path.append(os.path.abspath('../../'));
 
@@ -31,7 +31,7 @@ import utils
 def fail(message):
     sys.stderr.write("%s\n" % (message));
     sys.exit(1);
-    
+
 ################################################################################
 
 # prefix: non-US
@@ -49,7 +49,7 @@ def fail(message):
 def test(input, output):
     result = utils.extract_component_from_section(input);
     if result != output:
-        fail ("%s -> %s [should have been %s]" % (input, repr(result), repr(output)));
+        fail ("%s -> %r [should have been %r]" % (input, result, output));
 
 def main ():
     # Err, whoops?  should probably be "utils", "main"...
index fb32bc237926cccb571936cb35b5a739bb8e4e7f..5970b6cc81aab9e4afcd145f04b3f0cf8a5c2331 100644 (file)
--- a/utils.py
+++ b/utils.py
@@ -1,6 +1,8 @@
+#!/usr/bin/env python
+
 # Utility functions
 # Copyright (C) 2000, 2001, 2002  James Troup <james@nocrew.org>
-# $Id: utils.py,v 1.50 2002-07-14 15:01:04 troup Exp $
+# $Id: utils.py,v 1.51 2002-10-16 02:47:32 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
@@ -88,18 +90,18 @@ def str_isnum (s):
 def extract_component_from_section(section):
     component = "";
 
-    if string.find(section, '/') != -1:
-        component = string.split(section, '/')[0];
-    if string.lower(component) == "non-us" and string.count(section, '/') > 0:
-        s = component + '/' + string.split(section, '/')[1];
+    if section.find('/') != -1:
+        component = section.split('/')[0];
+    if component.lower() == "non-us" and section.count('/') > 0:
+        s = component + '/' + section.split('/')[1];
         if Cnf.has_key("Component::%s" % s): # Avoid e.g. non-US/libs
             component = s;
 
-    if string.lower(section) == "non-us":
+    if section.lower() == "non-us":
         component = "non-US/main";
 
     # non-US prefix is case insensitive
-    if string.lower(component)[:6] == "non-us":
+    if component.lower()[:6] == "non-us":
         component = "non-US"+component[6:];
 
     # Expand default component
@@ -142,7 +144,7 @@ def parse_changes(filename, dsc_whitespace_rules=0):
     index = 0;
     indexed_lines = {};
     for line in lines:
-        index = index + 1;
+        index += 1;
         indexed_lines[index] = line[:-1];
 
     inside_signature = 0;
@@ -151,52 +153,52 @@ def parse_changes(filename, dsc_whitespace_rules=0):
     index = 0;
     first = -1;
     while index < max(indices):
-        index = index + 1;
+        index += 1;
         line = indexed_lines[index];
         if line == "":
             if dsc_whitespace_rules:
-                index = index + 1;
+                index += 1;
                 if index > max(indices):
                     raise invalid_dsc_format_exc, index;
                 line = indexed_lines[index];
-                if string.find(line, "-----BEGIN PGP SIGNATURE") != 0:
+                if not line.startswith("-----BEGIN PGP SIGNATURE"):
                     raise invalid_dsc_format_exc, index;
                 inside_signature = 0;
                 break;
-        if string.find(line, "-----BEGIN PGP SIGNATURE") == 0:
+        if line.startswith("-----BEGIN PGP SIGNATURE"):
             break;
-        if string.find(line, "-----BEGIN PGP SIGNED MESSAGE") == 0:
+        if line.startswith("-----BEGIN PGP SIGNED MESSAGE"):
             if dsc_whitespace_rules:
                 inside_signature = 1;
                 while index < max(indices) and line != "":
-                    index = index + 1;
+                    index += 1;
                     line = indexed_lines[index];
             continue;
         slf = re_single_line_field.match(line);
         if slf:
-            field = string.lower(slf.groups()[0]);
+            field = slf.groups()[0].lower();
             changes[field] = slf.groups()[1];
            first = 1;
             continue;
         if line == " .":
-            changes[field] = changes[field] + '\n';
+            changes[field] += '\n';
             continue;
         mlf = re_multi_line_field.match(line);
         if mlf:
             if first == -1:
                 raise changes_parse_error_exc, "'%s'\n [Multi-line field continuing on from nothing?]" % (line);
             if first == 1 and changes[field] != "":
-                changes[field] = changes[field] + '\n';
+                changes[field] += '\n';
             first = 0;
-           changes[field] = changes[field] + mlf.groups()[0] + '\n';
+           changes[field] += mlf.groups()[0] + '\n';
             continue;
-       error = error + line;
+       error += line;
 
     if dsc_whitespace_rules and inside_signature:
         raise invalid_dsc_format_exc, index;
 
     changes_in.close();
-    changes["filecontents"] = string.join (lines, "");
+    changes["filecontents"] = "".join(lines);
 
     if error != "":
        raise changes_parse_error_exc, error;
@@ -219,10 +221,10 @@ def build_file_list(changes, is_a_dsc=0):
     if not changes.has_key("files"):
        raise no_files_exc
 
-    for i in string.split(changes["files"], "\n"):
+    for i in changes["files"].split("\n"):
         if i == "":
             break
-        s = string.split(i)
+        s = i.split();
         section = priority = "";
         try:
             if is_a_dsc:
@@ -262,7 +264,7 @@ def fix_maintainer (maintainer):
     if m != None and len(m.groups()) == 2:
         name = m.group(1)
         email = m.group(2)
-        if string.find(name, ',') != -1 or string.find(name, '.') != -1:
+        if name.find(',') != -1 or name.find('.') != -1:
             rfc822 = re_parse_maintainer.sub(r"\2 (\1)", maintainer)
     return (rfc822, name, email)
 
@@ -294,10 +296,9 @@ def send_mail (message, filename):
 
 def poolify (source, component):
     if component != "":
-       component = component + '/';
+       component += '/';
     # FIXME: this is nasty
-    component = string.lower(component);
-    component = string.replace(component, 'non-us/', 'non-US/');
+    component = component.lower().replace('non-us/', 'non-US/');
     if source[:3] == "lib":
        return component + source[:4] + '/' + source + '/'
     else:
@@ -316,7 +317,7 @@ def move (src, dest, overwrite = 0, perms = 0664):
        os.umask(umask);
     #print "Moving %s to %s..." % (src, dest);
     if os.path.exists(dest) and os.path.isdir(dest):
-       dest = dest + '/' + os.path.basename(src);
+       dest += '/' + os.path.basename(src);
     # Don't overwrite unless forced to
     if os.path.exists(dest):
         if not overwrite:
@@ -339,7 +340,7 @@ def copy (src, dest, overwrite = 0, perms = 0664):
        os.umask(umask);
     #print "Copying %s to %s..." % (src, dest);
     if os.path.exists(dest) and os.path.isdir(dest):
-       dest = dest + '/' + os.path.basename(src);
+       dest += '/' + os.path.basename(src);
     # Don't overwrite unless forced to
     if os.path.exists(dest):
         if not overwrite:
@@ -380,8 +381,8 @@ def which_apt_conf_file ():
 # (woefully incomplete)
 
 def regex_safe (s):
-    s = string.replace(s, '+', '\\\\+');
-    s = string.replace(s, '.', '\\\\.');
+    s = s.replace('+', '\\\\+');
+    s = s.replace('.', '\\\\.');
     return s
 
 ######################################################################################
@@ -391,7 +392,7 @@ def TemplateSubst(map, filename):
     file = open_file(filename);
     template = file.read();
     for x in map.keys():
-        template = string.replace(template,x,map[x]);
+        template = template.replace(x,map[x]);
     file.close();
     return template;
 
@@ -409,7 +410,7 @@ def warn(msg):
 # Returns the user name with a laughable attempt at rfc822 conformancy
 # (read: removing stray periods).
 def whoami ():
-    return string.replace(string.split(pwd.getpwuid(os.getuid())[4],',')[0], '.', '');
+    return pwd.getpwuid(os.getuid())[4].split(',')[0].replace('.', '');
 
 ######################################################################################
 
@@ -430,7 +431,7 @@ def cc_fix_changes (changes):
     if o != "":
         del changes["architecture"]
     changes["architecture"] = {}
-    for j in string.split(o):
+    for j in o.split():
         changes["architecture"][j] = 1
 
 # Sort by source name, source version, 'have source', and then by filename
@@ -480,7 +481,7 @@ def find_next_free (dest, too_many=100):
     orig_dest = dest;
     while os.path.exists(dest) and extra < too_many:
         dest = orig_dest + '.' + repr(extra);
-        extra = extra + 1;
+        extra += 1;
     if extra >= too_many:
         raise tried_too_hard_exc;
     return dest;
@@ -494,16 +495,16 @@ def result_join (original, sep = '\t'):
             list.append("");
         else:
             list.append(original[i]);
-    return string.join(list, sep);
+    return sep.join(list);
 
 ################################################################################
 
 def prefix_multi_line_string(str, prefix):
     out = "";
-    for line in string.split(str, '\n'):
-        line = string.strip(line);
+    for line in str.split('\n'):
+        line = line.strip();
         if line:
-            out = out + "%s%s\n" % (prefix, line);
+            out += "%s%s\n" % (prefix, line);
     # Strip trailing new line
     if out:
         out = out[:-1];
@@ -515,10 +516,10 @@ def validate_changes_file_arg(file, fatal=1):
     error = None;
 
     orig_filename = file
-    if file[-6:] == ".katie":
+    if file.endswith(".katie"):
         file = file[:-6]+".changes";
 
-    if file[-8:] != ".changes":
+    if not file.endswith(".changes"):
         error = "invalid file type; not a changes file";
     else:
         if not os.access(file,os.R_OK):
@@ -546,7 +547,7 @@ def real_arch(arch):
 def join_with_commas_and(list):
        if len(list) == 0: return "nothing";
        if len(list) == 1: return list[0];
-       return string.join(list[:-1], ", ") + " and " + list[-1];
+       return ", ".join(list[:-1]) + " and " + list[-1];
 
 ################################################################################
 
@@ -560,14 +561,14 @@ def parse_args(Options):
     # Process suite
     if Options["Suite"]:
         suite_ids_list = [];
-        for suite in string.split(Options["Suite"]):
+        for suite in Options["Suite"].split():
             suite_id = db_access.get_suite_id(suite);
             if suite_id == -1:
-                utils.warn("suite '%s' not recognised." % (suite));
+                warn("suite '%s' not recognised." % (suite));
             else:
                 suite_ids_list.append(suite_id);
         if suite_ids_list:
-            con_suites = "AND su.id IN (%s)" % string.join(map(str, suite_ids_list), ", ");
+            con_suites = "AND su.id IN (%s)" % ", ".join(map(str, suite_ids_list));
         else:
             fubar("No valid suite given.");
     else:
@@ -576,14 +577,14 @@ def parse_args(Options):
     # Process component
     if Options["Component"]:
         component_ids_list = [];
-        for component in string.split(Options["Component"]):
+        for component in Options["Component"].split():
             component_id = db_access.get_component_id(component);
             if component_id == -1:
                 warn("component '%s' not recognised." % (component));
             else:
                 component_ids_list.append(component_id);
         if component_ids_list:
-            con_components = "AND c.id IN (%s)" % string.join(map(str, component_ids_list), ", ");
+            con_components = "AND c.id IN (%s)" % ", ".join(map(str, component_ids_list));
         else:
             fubar("No valid component given.");
     else:
@@ -594,7 +595,7 @@ def parse_args(Options):
     if Options["Architecture"]:
         arch_ids_list = [];
         check_source = 0;
-        for architecture in string.split(Options["Architecture"]):
+        for architecture in Options["Architecture"].split():
             if architecture == "source":
                 check_source = 1;
             else:
@@ -604,7 +605,7 @@ def parse_args(Options):
                 else:
                     arch_ids_list.append(architecture_id);
         if arch_ids_list:
-            con_architectures = "AND a.id IN (%s)" % string.join(map(str, arch_ids_list), ", ");
+            con_architectures = "AND a.id IN (%s)" % ", ".join(map(str, arch_ids_list));
         else:
             if not check_source:
                 fubar("No valid architecture given.");
diff --git a/ziyi b/ziyi
index a3af2789e531cd14456682791fa5aa4356804e22..1edcf7dcf9d53bcfadd8f04e950a1133e8064328 100755 (executable)
--- a/ziyi
+++ b/ziyi
@@ -3,7 +3,7 @@
 # Create all the Release files
 
 # Copyright (C) 2001, 2002  Anthony Towns <ajt@debian.org>
-# $Id: ziyi,v 1.22 2002-06-08 00:18:47 troup Exp $
+# $Id: ziyi,v 1.23 2002-10-16 02:47:32 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
@@ -23,7 +23,7 @@
 
 ################################################################################
 
-import sys, os, popen2, tempfile, stat, string, time
+import sys, os, popen2, tempfile, stat, time
 import utils
 import apt_pkg
 
@@ -49,9 +49,9 @@ Generate Release files.
 def compressnames (tree,type,file):
     compress = AptCnf.get("%s::%s::Compress" % (tree,type), AptCnf.get("Default::%s::Compress" % (type), ". gzip"))
     result = []
-    cl = string.split(compress)
+    cl = compress.split()
     uncompress = ("." not in cl)
-    for mode in string.split(compress):
+    for mode in compress.split():
        if mode == ".":
            result.append(file)
        elif mode == "gzip":
@@ -79,7 +79,7 @@ def create_temp_file (cmd):
            del x,r
            break
        f.write(x)
-       size = size + len(x)
+       size += len(x)
     f.flush()
     f.seek(0)
     return (size, f)
@@ -89,8 +89,8 @@ def print_md5sha_files (tree, files, hashop):
     for name in files:
         try:
            if name[0] == "<":
-               j = string.index(name, "/")
-               k = string.index(name, ">")
+               j = name.index("/")
+               k = name.index(">")
                (cat, ext, name) = (name[1:j], name[j+1:k], name[k+1:])
                (size, file_handle) = create_temp_file("%s %s%s%s" %
                    (cat, path, name, ext))
@@ -143,7 +143,7 @@ def main ():
             print "Skipping: " + suite + " (untouchable)"
             continue
 
-       suite = string.lower(suite)
+       suite = suite.lower()
 
        origin = SuiteBlock["Origin"]
        label = SuiteBlock.get("Label", origin)
@@ -189,17 +189,17 @@ def main ():
        out.write("Date: %s\n" % (time.strftime("%a, %d %b %Y %H:%M:%S UTC", time.gmtime(time.time()))))
        if notautomatic != "":
            out.write("NotAutomatic: %s\n" % (notautomatic))
-       out.write("Architectures: %s\n" % (string.join(filter(utils.real_arch, SuiteBlock.ValueList("Architectures")))))
+       out.write("Architectures: %s\n" % (" ".join(filter(utils.real_arch, SuiteBlock.ValueList("Architectures")))))
        if components:
-            out.write("Components: %s\n" % (string.join(components)))
+            out.write("Components: %s\n" % (" ".join(components)))
 
        out.write("Description: %s\n" % (SuiteBlock["Description"]))
 
        files = []
 
        if AptCnf.has_key("tree::%s" % (tree)):
-           for sec in string.split(AptCnf["tree::%s::Sections" % (tree)]):
-               for arch in string.split(AptCnf["tree::%s::Architectures" % (tree)]):
+           for sec in AptCnf["tree::%s::Sections" % (tree)].split():
+               for arch in AptCnf["tree::%s::Architectures" % (tree)].split():
                    if arch == "source":
                        for file in compressnames("tree::%s" % (tree), "Sources", "%s/%s/Sources" % (sec, arch)):
                            files.append(file)
@@ -222,7 +222,7 @@ def main ():
 
                     try:
                         release = open(relpath, "w")
-                        #release = open(string.replace(longsuite,"/","_") + "_" + arch + "_" + sec + "_Release", "w")
+                        #release = open(longsuite.replace("/","_") + "_" + arch + "_" + sec + "_Release", "w")
                     except IOError:
                         utils.fubar("Couldn't write to " + relpath);
 
@@ -242,20 +242,20 @@ def main ():
                     files.append(rel)
 
            if AptCnf.has_key("tree::%s/main" % (tree)):
-               sec = string.split(AptCnf["tree::%s/main::Sections" % (tree)])[0]
+               sec = AptCnf["tree::%s/main::Sections" % (tree)].split()[0]
                if sec != "debian-installer":
                    print "ALERT: weird non debian-installer section in %s" % (tree)
 
-               for arch in string.split(AptCnf["tree::%s/main::Architectures" % (tree)]):
+               for arch in AptCnf["tree::%s/main::Architectures" % (tree)].split():
                    if arch != "source":  # always true
                        for file in compressnames("tree::%s/main" % (tree), "Packages", "main/%s/binary-%s/Packages" % (sec, arch)):
                            files.append(file)
 
        elif AptCnf.has_key("bindirectory::%s" % (tree)):
            for file in compressnames("bindirectory::%s" % (tree), "Packages", AptCnf["bindirectory::%s::Packages" % (tree)]):
-               files.append(string.replace(file,tree+"/","",1))
+               files.append(file.replace(tree+"/","",1))
            for file in compressnames("bindirectory::%s" % (tree), "Sources", AptCnf["bindirectory::%s::Sources" % (tree)]):
-               files.append(string.replace(file,tree+"/","",1))
+               files.append(file.replace(tree+"/","",1))
        else:
            print "ALERT: no tree/bindirectory for %s" % (tree)
 
@@ -268,11 +268,11 @@ def main ():
        if Cnf.has_key("Dinstall::SigningKeyring"):
            keyring = "--secret-keyring \"%s\"" % Cnf["Dinstall::SigningKeyring"]
            if Cnf.has_key("Dinstall::SigningPubKeyring"):
-               keyring = keyring + " --keyring \"%s\"" % Cnf["Dinstall::SigningPubKeyring"]
+               keyring += " --keyring \"%s\"" % Cnf["Dinstall::SigningPubKeyring"]
 
            arguments = "--no-options --batch --no-tty --armour"
            if Cnf.has_key("Dinstall::SigningKeyIds"):
-               signkeyids = string.split(Cnf["Dinstall::SigningKeyIds"])
+               signkeyids = Cnf["Dinstall::SigningKeyIds"].split()
            else:
                signkeyids = [""]