X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;f=dak%2Frm.py;h=262039bf51a6174130b11a175ab66994ef710aa4;hb=d9822f04453a1b62ca0aa66e2efeea35f654778f;hp=12f15eb09ac7ee4b04e41a83cd2833dd8bbce1dc;hpb=06ce98c8111a8b09e5603dbbd34324a216412d69;p=dak.git diff --git a/dak/rm.py b/dak/rm.py old mode 100755 new mode 100644 index 12f15eb0..262039bf --- a/dak/rm.py +++ b/dak/rm.py @@ -40,8 +40,9 @@ ################################################################################ import commands, os, pg, re, sys -import dak.lib.utils, dak.lib.database import apt_pkg, apt_inst +import daklib.database +import daklib.utils ################################################################################ @@ -86,7 +87,7 @@ ARCH, BUG#, COMPONENT and SUITE can be comma (or space) separated lists, e.g. # the fuck are we gonna do now? What are we gonna do?" def game_over(): - answer = dak.lib.utils.our_raw_input("Continue (y/N)? ").lower() + answer = daklib.utils.our_raw_input("Continue (y/N)? ").lower() if answer != "y": print "Aborted." sys.exit(1) @@ -106,11 +107,11 @@ def reverse_depends_check(removals, suites): for component in components: filename = "%s/dists/%s/%s/binary-%s/Packages.gz" % (Cnf["Dir::Root"], suites[0], component, architecture) # apt_pkg.ParseTagFile needs a real file handle and can't handle a GzipFile instance... - temp_filename = dak.lib.utils.temp_filename() + temp_filename = daklib.utils.temp_filename() (result, output) = commands.getstatusoutput("gunzip -c %s > %s" % (filename, temp_filename)) if (result != 0): - dak.lib.utils.fubar("Gunzip invocation failed!\n%s\n" % (output), result) - packages = dak.lib.utils.open_file(temp_filename) + daklib.utils.fubar("Gunzip invocation failed!\n%s\n" % (output), result) + packages = daklib.utils.open_file(temp_filename) Packages = apt_pkg.ParseTagFile(packages) while Packages.Step(): package = Packages.Section.Find("Package") @@ -157,26 +158,26 @@ def reverse_depends_check(removals, suites): unsat = 0 for dep_package, _, _ in dep: if dep_package in removals: - unsat += 1 + unsat += 1 if unsat == len(dep): component = p2c[package] if component != "main": what = "%s/%s" % (package, component) else: what = "** %s" % (package) - print "%s has an unsatisfied dependency on %s: %s" % (what, architecture, dak.lib.utils.pp_deps(dep)) + print "%s has an unsatisfied dependency on %s: %s" % (what, architecture, daklib.utils.pp_deps(dep)) dep_problem = 1 # Check source dependencies (Build-Depends and Build-Depends-Indep) for component in components: filename = "%s/dists/%s/%s/source/Sources.gz" % (Cnf["Dir::Root"], suites[0], component) # apt_pkg.ParseTagFile needs a real file handle and can't handle a GzipFile instance... - temp_filename = dak.lib.utils.temp_filename() + temp_filename = daklib.utils.temp_filename() result, output = commands.getstatusoutput("gunzip -c %s > %s" % (filename, temp_filename)) if result != 0: sys.stderr.write("Gunzip invocation failed!\n%s\n" % (output)) sys.exit(result) - sources = dak.lib.utils.open_file(temp_filename, "r") + sources = daklib.utils.open_file(temp_filename, "r") Sources = apt_pkg.ParseTagFile(sources) while Sources.Step(): source = Sources.Section.Find("Package") @@ -195,13 +196,13 @@ def reverse_depends_check(removals, suites): unsat = 0 for dep_package, _, _ in dep: if dep_package in removals: - unsat += 1 + unsat += 1 if unsat == len(dep): if component != "main": source = "%s/%s" % (source, component) else: source = "** %s" % (source) - print "%s has an unsatisfied build-dependency: %s" % (source, dak.lib.utils.pp_deps(dep)) + print "%s has an unsatisfied build-dependency: %s" % (source, daklib.utils.pp_deps(dep)) dep_problem = 1 sources.close() os.unlink(temp_filename) @@ -213,13 +214,13 @@ def reverse_depends_check(removals, suites): else: print "No dependency problem found." print - + ################################################################################ def main (): global Cnf, Options, projectB - Cnf = dak.lib.utils.get_conf() + Cnf = daklib.utils.get_conf() Arguments = [('h',"help","Rm::Options::Help"), ('a',"architecture","Rm::Options::Architecture", "HasArg"), @@ -238,31 +239,31 @@ def main (): for i in [ "architecture", "binary-only", "carbon-copy", "component", "done", "help", "no-action", "partial", "rdep-check", "reason", "source-only" ]: - if not Cnf.has_key("Rm::Options::%s" % (i)): - Cnf["Rm::Options::%s" % (i)] = "" + if not Cnf.has_key("Rm::Options::%s" % (i)): + Cnf["Rm::Options::%s" % (i)] = "" if not Cnf.has_key("Rm::Options::Suite"): - Cnf["Rm::Options::Suite"] = "unstable" + Cnf["Rm::Options::Suite"] = "unstable" arguments = apt_pkg.ParseCommandLine(Cnf,Arguments,sys.argv) Options = Cnf.SubTree("Rm::Options") if Options["Help"]: - usage() + usage() projectB = pg.connect(Cnf["DB::Name"], Cnf["DB::Host"], int(Cnf["DB::Port"])) - dak.lib.database.init(Cnf, projectB) + daklib.database.init(Cnf, projectB) # Sanity check options if not arguments: - dak.lib.utils.fubar("need at least one package name as an argument.") + daklib.utils.fubar("need at least one package name as an argument.") if Options["Architecture"] and Options["Source-Only"]: - dak.lib.utils.fubar("can't use -a/--architecutre and -S/--source-only options simultaneously.") + daklib.utils.fubar("can't use -a/--architecutre and -S/--source-only options simultaneously.") if Options["Binary-Only"] and Options["Source-Only"]: - dak.lib.utils.fubar("can't use -b/--binary-only and -S/--source-only options simultaneously.") + daklib.utils.fubar("can't use -b/--binary-only and -S/--source-only options simultaneously.") if Options.has_key("Carbon-Copy") and not Options.has_key("Done"): - dak.lib.utils.fubar("can't use -C/--carbon-copy without also using -d/--done option.") + daklib.utils.fubar("can't use -C/--carbon-copy without also using -d/--done option.") if Options["Architecture"] and not Options["Partial"]: - dak.lib.utils.warn("-a/--architecture implies -p/--partial.") + daklib.utils.warn("-a/--architecture implies -p/--partial.") Options["Partial"] = "true" # Force the admin to tell someone if we're not doing a 'dak @@ -270,7 +271,7 @@ def main (): # as telling someone). if not Options["No-Action"] and not Options["Carbon-Copy"] \ and not Options["Done"] and Options["Reason"].find("[auto-cruft]") == -1: - dak.lib.utils.fubar("Need a -C/--carbon-copy if not closing a bug and not doing a cruft removal.") + daklib.utils.fubar("Need a -C/--carbon-copy if not closing a bug and not doing a cruft removal.") # Process -C/--carbon-copy # @@ -280,8 +281,8 @@ def main (): # 3) contains a '@' - assumed to be an email address, used unmofidied # carbon_copy = [] - for copy_to in dak.lib.utils.split_args(Options.get("Carbon-Copy")): - if dak.lib.utils.str_isnum(copy_to): + for copy_to in daklib.utils.split_args(Options.get("Carbon-Copy")): + if copy_to.isdigit(): carbon_copy.append(copy_to + "@" + Cnf["Dinstall::BugServer"]) elif copy_to == 'package': for package in arguments: @@ -291,24 +292,24 @@ def main (): elif '@' in copy_to: carbon_copy.append(copy_to) else: - dak.lib.utils.fubar("Invalid -C/--carbon-copy argument '%s'; not a bug number, 'package' or email address." % (copy_to)) + daklib.utils.fubar("Invalid -C/--carbon-copy argument '%s'; not a bug number, 'package' or email address." % (copy_to)) if Options["Binary-Only"]: field = "b.package" else: field = "s.source" - con_packages = "AND %s IN (%s)" % (field, ", ".join(map(repr, arguments))) + con_packages = "AND %s IN (%s)" % (field, ", ".join([ repr(i) for i in arguments ])) (con_suites, con_architectures, con_components, check_source) = \ - dak.lib.utils.parse_args(Options) + daklib.utils.parse_args(Options) # Additional suite checks suite_ids_list = [] - suites = dak.lib.utils.split_args(Options["Suite"]) - suites_list = dak.lib.utils.join_with_commas_and(suites) + suites = daklib.utils.split_args(Options["Suite"]) + suites_list = daklib.utils.join_with_commas_and(suites) if not Options["No-Action"]: for suite in suites: - suite_id = dak.lib.database.get_suite_id(suite) + suite_id = daklib.database.get_suite_id(suite) if suite_id != -1: suite_ids_list.append(suite_id) if suite == "stable": @@ -324,7 +325,7 @@ def main (): # Additional architecture checks if Options["Architecture"] and check_source: - dak.lib.utils.warn("'source' in -a/--argument makes no sense and is ignored.") + daklib.utils.warn("'source' in -a/--argument makes no sense and is ignored.") # Additional component processing over_con_components = con_components.replace("c.id", "component") @@ -362,9 +363,9 @@ def main (): for i in source_packages.keys(): filename = "/".join(source_packages[i]) try: - dsc = dak.lib.utils.parse_changes(filename) - except dak.lib.utils.cant_open_exc: - dak.lib.utils.warn("couldn't open '%s'." % (filename)) + dsc = daklib.utils.parse_changes(filename) + except daklib.utils.cant_open_exc: + daklib.utils.warn("couldn't open '%s'." % (filename)) continue for package in dsc.get("binary").split(','): package = package.strip() @@ -377,7 +378,7 @@ def main (): 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 = "/".join(i[:2]) - control = apt_pkg.ParseSection(apt_inst.debExtractControl(dak.lib.utils.open_file(filename))) + control = apt_pkg.ParseSection(apt_inst.debExtractControl(daklib.utils.open_file(filename))) source = control.Find("Source", control.Find("Package")) source = re_strip_source_version.sub('', source) if source_packages.has_key(source): @@ -391,12 +392,12 @@ def main (): # If we don't have a reason; spawn an editor so the user can add one # Write the rejection email out as the .reason file if not Options["Reason"] and not Options["No-Action"]: - temp_filename = dak.lib.utils.temp_filename() + temp_filename = daklib.utils.temp_filename() editor = os.environ.get("EDITOR","vi") result = os.system("%s %s" % (editor, temp_filename)) if result != 0: - dak.lib.utils.fubar ("vi invocation failed for `%s'!" % (temp_filename), result) - temp_file = dak.lib.utils.open_file(temp_filename) + daklib.utils.fubar ("vi invocation failed for `%s'!" % (temp_filename), result) + temp_file = daklib.utils.open_file(temp_filename) for line in temp_file.readlines(): Options["Reason"] += line temp_file.close() @@ -419,7 +420,7 @@ def main (): maintainer_list = [] for maintainer_id in maintainers.keys(): - maintainer_list.append(dak.lib.database.get_maintainer(maintainer_id)) + maintainer_list.append(daklib.database.get_maintainer(maintainer_id)) summary = "" removals = d.keys() removals.sort() @@ -427,7 +428,7 @@ def main (): versions = d[package].keys() versions.sort(apt_pkg.VersionCompare) for version in versions: - d[package][version].sort(dak.lib.utils.arch_compare_sw) + d[package][version].sort(daklib.utils.arch_compare_sw) summary += "%10s | %10s | %s\n" % (package, version, ", ".join(d[package][version])) print "Will remove the following packages from %s:" % (suites_list) print @@ -453,11 +454,11 @@ def main (): print "Going to remove the packages now." game_over() - whoami = dak.lib.utils.whoami() + whoami = daklib.utils.whoami() date = commands.getoutput('date -R') # Log first; if it all falls apart I want a record that we at least tried. - logfile = dak.lib.utils.open_file(Cnf["Rm::LogFile"], 'a') + logfile = daklib.utils.open_file(Cnf["Rm::LogFile"], 'a') logfile.write("=========================================================================\n") logfile.write("[Date: %s] [ftpmaster: %s]\n" % (date, whoami)) logfile.write("Removed the following packages from %s:\n\n%s" % (suites_list, summary)) @@ -467,8 +468,8 @@ def main (): logfile.write("----------------------------------------------\n") logfile.flush() - dsc_type_id = dak.lib.database.get_override_type_id('dsc') - deb_type_id = dak.lib.database.get_override_type_id('deb') + dsc_type_id = daklib.database.get_override_type_id('dsc') + deb_type_id = daklib.database.get_override_type_id('deb') # Do the actual deletion print "Deleting...", @@ -509,7 +510,7 @@ def main (): Subst["__BCC__"] = "Bcc: " + ", ".join(bcc) else: Subst["__BCC__"] = "X-Filler: 42" - Subst["__CC__"] = "X-DAK: dak rm\nX-Katie: this header is obsolete" + Subst["__CC__"] = "X-DAK: dak rm\nX-Katie: melanie $Revision: 1.44 $" if carbon_copy: Subst["__CC__"] += "\nCc: " + ", ".join(carbon_copy) Subst["__SUITE_LIST__"] = suites_list @@ -517,14 +518,14 @@ def main (): Subst["__ADMIN_ADDRESS__"] = Cnf["Dinstall::MyAdminAddress"] Subst["__DISTRO__"] = Cnf["Dinstall::MyDistribution"] Subst["__WHOAMI__"] = whoami - whereami = dak.lib.utils.where_am_i() + whereami = daklib.utils.where_am_i() Archive = Cnf.SubTree("Archive::%s" % (whereami)) Subst["__MASTER_ARCHIVE__"] = Archive["OriginServer"] Subst["__PRIMARY_MIRROR__"] = Archive["PrimaryMirror"] - for bug in dak.lib.utils.split_args(Options["Done"]): + for bug in daklib.utils.split_args(Options["Done"]): Subst["__BUG_NUMBER__"] = bug - mail_message = dak.lib.utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/rm.bug-close") - dak.lib.utils.send_mail(mail_message) + mail_message = daklib.utils.TemplateSubst(Subst,Cnf["Dir::Templates"]+"/rm.bug-close") + daklib.utils.send_mail(mail_message) logfile.write("=========================================================================\n") logfile.close() @@ -533,4 +534,3 @@ def main (): if __name__ == '__main__': main() -