dak/rm.py

   1 #!/usr/bin/env python
   2
   3 """ General purpose package removal tool for ftpmaster """
   4 # Copyright (C) 2000, 2001, 2002, 2003, 2004, 2006  James Troup <james@nocrew.org>
   5 # Copyright (C) 2010 Alexander Reichle-Schmehl <tolimar@debian.org>
   6
   7 # This program is free software; you can redistribute it and/or modify
   8 # it under the terms of the GNU General Public License as published by
   9 # the Free Software Foundation; either version 2 of the License, or
  10 # (at your option) any later version.
  11
  12 # This program is distributed in the hope that it will be useful,
  13 # but WITHOUT ANY WARRANTY; without even the implied warranty of
  14 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  15 # GNU General Public License for more details.
  16
  17 # You should have received a copy of the GNU General Public License
  18 # along with this program; if not, write to the Free Software
  19 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  20
  21 ################################################################################
  22
  23 # o OpenBSD team wants to get changes incorporated into IPF. Darren no
  24 #    respond.
  25 # o Ask again -> No respond. Darren coder supreme.
  26 # o OpenBSD decide to make changes, but only in OpenBSD source
  27 #    tree. Darren hears, gets angry! Decides: "LICENSE NO ALLOW!"
  28 # o Insert Flame War.
  29 # o OpenBSD team decide to switch to different packet filter under BSD
  30 #    license. Because Project Goal: Every user should be able to make
  31 #    changes to source tree. IPF license bad!!
  32 # o Darren try get back: says, NetBSD, FreeBSD allowed! MUAHAHAHAH!!!
  33 # o Theo say: no care, pf much better than ipf!
  34 # o Darren changes mind: changes license. But OpenBSD will not change
  35 #    back to ipf. Darren even much more bitter.
  36 # o Darren so bitterbitter. Decides: I'LL GET BACK BY FORKING OPENBSD AND
  37 #    RELEASING MY OWN VERSION. HEHEHEHEHE.
  38
  39 #                        http://slashdot.org/comments.pl?sid=26697&cid=2883271
  40
  41 ################################################################################
  42
  43 import commands
  44 import os
  45 import sys
  46 import apt_pkg
  47 import apt_inst
  48 from re import sub
  49
  50 from daklib.config import Config
  51 from daklib.dbconn import *
  52 from daklib import utils
  53 from daklib.dak_exceptions import *
  54 from daklib.regexes import re_strip_source_version, re_bin_only_nmu
  55 import debianbts as bts
  56
  57 ################################################################################
  58
  59 Options = None
  60
  61 ################################################################################
  62
  63 def usage (exit_code=0):
  64     print """Usage: dak rm [OPTIONS] PACKAGE[...]
  65 Remove PACKAGE(s) from suite(s).
  66
  67   -a, --architecture=ARCH    only act on this architecture
  68   -b, --binary               PACKAGE are binary packages to remove
  69   -B, --binary-only          remove binaries only
  70   -c, --component=COMPONENT  act on this component
  71   -C, --carbon-copy=EMAIL    send a CC of removal message to EMAIL
  72   -d, --done=BUG#            send removal message as closure to bug#
  73   -D, --do-close             also close all bugs associated to that package
  74   -h, --help                 show this help and exit
  75   -m, --reason=MSG           reason for removal
  76   -n, --no-action            don't do anything
  77   -p, --partial              don't affect override files
  78   -R, --rdep-check           check reverse dependencies
  79   -s, --suite=SUITE          act on this suite
  80   -S, --source-only          remove source only
  81
  82 ARCH, BUG#, COMPONENT and SUITE can be comma (or space) separated lists, e.g.
  83     --architecture=amd64,i386"""
  84
  85     sys.exit(exit_code)
  86
  87 ################################################################################
  88
  89 # "Hudson: What that's great, that's just fucking great man, now what
  90 #  the fuck are we supposed to do? We're in some real pretty shit now
  91 #  man...That's it man, game over man, game over, man! Game over! What
  92 #  the fuck are we gonna do now? What are we gonna do?"
  93
  94 def game_over():
  95     answer = utils.our_raw_input("Continue (y/N)? ").lower()
  96     if answer != "y":
  97         print "Aborted."
  98         sys.exit(1)
  99
 100 ################################################################################
 101
 102 def reverse_depends_check(removals, suite, arches=None, session=None):
 103     print "Checking reverse dependencies..."
 104     if utils.check_reverse_depends(removals, suite, arches, session):
 105         print "Dependency problem found."
 106         if not Options["No-Action"]:
 107             game_over()
 108     else:
 109         print "No dependency problem found."
 110     print
 111
 112 ################################################################################
 113
 114 def main ():
 115     global Options
 116
 117     cnf = Config()
 118
 119     Arguments = [('h',"help","Rm::Options::Help"),
 120                  ('a',"architecture","Rm::Options::Architecture", "HasArg"),
 121                  ('b',"binary", "Rm::Options::Binary"),
 122                  ('B',"binary-only", "Rm::Options::Binary-Only"),
 123                  ('c',"component", "Rm::Options::Component", "HasArg"),
 124                  ('C',"carbon-copy", "Rm::Options::Carbon-Copy", "HasArg"), # Bugs to Cc
 125                  ('d',"done","Rm::Options::Done", "HasArg"), # Bugs fixed
 126                  ('D',"do-close","Rm::Options::Do-Close"),
 127                  ('R',"rdep-check", "Rm::Options::Rdep-Check"),
 128                  ('m',"reason", "Rm::Options::Reason", "HasArg"), # Hysterical raisins; -m is old-dinstall option for rejection reason
 129                  ('n',"no-action","Rm::Options::No-Action"),
 130                  ('p',"partial", "Rm::Options::Partial"),
 131                  ('s',"suite","Rm::Options::Suite", "HasArg"),
 132                  ('S',"source-only", "Rm::Options::Source-Only"),
 133                  ]
 134
 135     for i in [ "architecture", "binary", "binary-only", "carbon-copy", "component",
 136                "done", "help", "no-action", "partial", "rdep-check", "reason",
 137                "source-only", "Do-Close" ]:
 138         if not cnf.has_key("Rm::Options::%s" % (i)):
 139             cnf["Rm::Options::%s" % (i)] = ""
 140     if not cnf.has_key("Rm::Options::Suite"):
 141         cnf["Rm::Options::Suite"] = "unstable"
 142
 143     arguments = apt_pkg.parse_commandline(cnf.Cnf, Arguments, sys.argv)
 144     Options = cnf.subtree("Rm::Options")
 145
 146     if Options["Help"]:
 147         usage()
 148
 149     session = DBConn().session()
 150
 151     # Sanity check options
 152     if not arguments:
 153         utils.fubar("need at least one package name as an argument.")
 154     if Options["Architecture"] and Options["Source-Only"]:
 155         utils.fubar("can't use -a/--architecture and -S/--source-only options simultaneously.")
 156     if ((Options["Binary"] and Options["Source-Only"])
 157             or (Options["Binary"] and Options["Binary-Only"])
 158             or (Options["Binary-Only"] and Options["Source-Only"])):
 159         utils.fubar("Only one of -b/--binary, -B/--binary-only and -S/--source-only can be used.")
 160     if Options.has_key("Carbon-Copy") and not Options.has_key("Done"):
 161         utils.fubar("can't use -C/--carbon-copy without also using -d/--done option.")
 162     if Options["Architecture"] and not Options["Partial"]:
 163         utils.warn("-a/--architecture implies -p/--partial.")
 164         Options["Partial"] = "true"
 165     if Options["Do-Close"] and not Options["Done"]:
 166         utils.fubar("No.")
 167     if (Options["Do-Close"]
 168            and (Options["Binary"] or Options["Binary-Only"] or Options["Source-Only"])):
 169         utils.fubar("No.")
 170     if Options["Do-Close"] and Options["Suite"] != 'unstable':
 171         utils.fubar("No.")
 172
 173     # Force the admin to tell someone if we're not doing a 'dak
 174     # cruft-report' inspired removal (or closing a bug, which counts
 175     # as telling someone).
 176     if not Options["No-Action"] and not Options["Carbon-Copy"] \
 177            and not Options["Done"] and Options["Reason"].find("[auto-cruft]") == -1:
 178         utils.fubar("Need a -C/--carbon-copy if not closing a bug and not doing a cruft removal.")
 179
 180     # Process -C/--carbon-copy
 181     #
 182     # Accept 3 types of arguments (space separated):
 183     #  1) a number - assumed to be a bug number, i.e. nnnnn@bugs.debian.org
 184     #  2) the keyword 'package' - cc's $package@packages.debian.org for every argument
 185     #  3) contains a '@' - assumed to be an email address, used unmofidied
 186     #
 187     carbon_copy = []
 188     for copy_to in utils.split_args(Options.get("Carbon-Copy")):
 189         if copy_to.isdigit():
 190             if cnf.has_key("Dinstall::BugServer"):
 191                 carbon_copy.append(copy_to + "@" + cnf["Dinstall::BugServer"])
 192             else:
 193                 utils.fubar("Asked to send mail to #%s in BTS but Dinstall::BugServer is not configured" % copy_to)
 194         elif copy_to == 'package':
 195             for package in arguments:
 196                 if cnf.has_key("Dinstall::PackagesServer"):
 197                     carbon_copy.append(package + "@" + cnf["Dinstall::PackagesServer"])
 198                 if cnf.has_key("Dinstall::TrackingServer"):
 199                     carbon_copy.append(package + "@" + cnf["Dinstall::TrackingServer"])
 200         elif '@' in copy_to:
 201             carbon_copy.append(copy_to)
 202         else:
 203             utils.fubar("Invalid -C/--carbon-copy argument '%s'; not a bug number, 'package' or email address." % (copy_to))
 204
 205     if Options["Binary"]:
 206         field = "b.package"
 207     else:
 208         field = "s.source"
 209     con_packages = "AND %s IN (%s)" % (field, ", ".join([ repr(i) for i in arguments ]))
 210
 211     (con_suites, con_architectures, con_components, check_source) = \
 212                  utils.parse_args(Options)
 213
 214     # Additional suite checks
 215     suite_ids_list = []
 216     whitelists = []
 217     suites = utils.split_args(Options["Suite"])
 218     suites_list = utils.join_with_commas_and(suites)
 219     if not Options["No-Action"]:
 220         for suite in suites:
 221             s = get_suite(suite, session=session)
 222             if s is not None:
 223                 suite_ids_list.append(s.suite_id)
 224                 whitelists.append(s.mail_whitelist)
 225             if suite in ("oldstable", "stable"):
 226                 print "**WARNING** About to remove from the (old)stable suite!"
 227                 print "This should only be done just prior to a (point) release and not at"
 228                 print "any other time."
 229                 game_over()
 230             elif suite == "testing":
 231                 print "**WARNING About to remove from the testing suite!"
 232                 print "There's no need to do this normally as removals from unstable will"
 233                 print "propogate to testing automagically."
 234                 game_over()
 235
 236     # Additional architecture checks
 237     if Options["Architecture"] and check_source:
 238         utils.warn("'source' in -a/--argument makes no sense and is ignored.")
 239
 240     # Additional component processing
 241     over_con_components = con_components.replace("c.id", "component")
 242
 243     # Don't do dependency checks on multiple suites
 244     if Options["Rdep-Check"] and len(suites) > 1:
 245         utils.fubar("Reverse dependency check on multiple suites is not implemented.")
 246
 247     to_remove = []
 248     maintainers = {}
 249
 250     # We have 3 modes of package selection: binary, source-only, binary-only
 251     # and source+binary.
 252
 253     # XXX: TODO: This all needs converting to use placeholders or the object
 254     #            API. It's an SQL injection dream at the moment
 255
 256     if Options["Binary"]:
 257         # Removal by binary package name
 258         q = session.execute("SELECT b.package, b.version, a.arch_string, b.id, b.maintainer FROM binaries b, bin_associations ba, architecture a, suite su, files f, files_archive_map af, component c WHERE ba.bin = b.id AND ba.suite = su.id AND b.architecture = a.id AND b.file = f.id AND af.file_id = f.id AND af.archive_id = su.archive_id AND af.component_id = c.id %s %s %s %s" % (con_packages, con_suites, con_components, con_architectures))
 259         to_remove.extend(q)
 260     else:
 261         # Source-only
 262         if not Options["Binary-Only"]:
 263             q = session.execute("SELECT s.source, s.version, 'source', s.id, s.maintainer FROM source s, src_associations sa, suite su, archive, files f, files_archive_map af, component c WHERE sa.source = s.id AND sa.suite = su.id AND archive.id = su.archive_id AND s.file = f.id AND af.file_id = f.id AND af.archive_id = su.archive_id AND af.component_id = c.id %s %s %s" % (con_packages, con_suites, con_components))
 264             to_remove.extend(q)
 265         if not Options["Source-Only"]:
 266             # Source + Binary
 267             q = session.execute("""
 268                     SELECT b.package, b.version, a.arch_string, b.id, b.maintainer
 269                     FROM binaries b
 270                          JOIN bin_associations ba ON b.id = ba.bin
 271                          JOIN architecture a ON b.architecture = a.id
 272                          JOIN suite su ON ba.suite = su.id
 273                          JOIN archive ON archive.id = su.archive_id
 274                          JOIN files_archive_map af ON b.file = af.file_id AND af.archive_id = archive.id
 275                          JOIN component c ON af.component_id = c.id
 276                          JOIN source s ON b.source = s.id
 277                          JOIN src_associations sa ON s.id = sa.source AND sa.suite = su.id
 278                     WHERE TRUE %s %s %s %s""" % (con_packages, con_suites, con_components, con_architectures))
 279             to_remove.extend(q)
 280
 281     if not to_remove:
 282         print "Nothing to do."
 283         sys.exit(0)
 284
 285     # If we don't have a reason; spawn an editor so the user can add one
 286     # Write the rejection email out as the <foo>.reason file
 287     if not Options["Reason"] and not Options["No-Action"]:
 288         (fd, temp_filename) = utils.temp_filename()
 289         editor = os.environ.get("EDITOR","vi")
 290         result = os.system("%s %s" % (editor, temp_filename))
 291         if result != 0:
 292             utils.fubar ("vi invocation failed for `%s'!" % (temp_filename), result)
 293         temp_file = utils.open_file(temp_filename)
 294         for line in temp_file.readlines():
 295             Options["Reason"] += line
 296         temp_file.close()
 297         os.unlink(temp_filename)
 298
 299     # Generate the summary of what's to be removed
 300     d = {}
 301     for i in to_remove:
 302         package = i[0]
 303         version = i[1]
 304         architecture = i[2]
 305         maintainer = i[4]
 306         maintainers[maintainer] = ""
 307         if not d.has_key(package):
 308             d[package] = {}
 309         if not d[package].has_key(version):
 310             d[package][version] = []
 311         if architecture not in d[package][version]:
 312             d[package][version].append(architecture)
 313
 314     maintainer_list = []
 315     for maintainer_id in maintainers.keys():
 316         maintainer_list.append(get_maintainer(maintainer_id).name)
 317     summary = ""
 318     removals = d.keys()
 319     removals.sort()
 320     versions = []
 321     for package in removals:
 322         versions = d[package].keys()
 323         versions.sort(apt_pkg.version_compare)
 324         for version in versions:
 325             d[package][version].sort(utils.arch_compare_sw)
 326             summary += "%10s | %10s | %s\n" % (package, version, ", ".join(d[package][version]))
 327     print "Will remove the following packages from %s:" % (suites_list)
 328     print
 329     print summary
 330     print "Maintainer: %s" % ", ".join(maintainer_list)
 331     if Options["Done"]:
 332         print "Will also close bugs: "+Options["Done"]
 333     if carbon_copy:
 334         print "Will also send CCs to: " + ", ".join(carbon_copy)
 335     if Options["Do-Close"]:
 336         print "Will also close associated bug reports."
 337     print
 338     print "------------------- Reason -------------------"
 339     print Options["Reason"]
 340     print "----------------------------------------------"
 341     print
 342
 343     if Options["Rdep-Check"]:
 344         arches = utils.split_args(Options["Architecture"])
 345         reverse_depends_check(removals, suites[0], arches, session)
 346
 347     # If -n/--no-action, drop out here
 348     if Options["No-Action"]:
 349         sys.exit(0)
 350
 351     print "Going to remove the packages now."
 352     game_over()
 353
 354     whoami = utils.whoami()
 355     date = commands.getoutput('date -R')
 356
 357     # Log first; if it all falls apart I want a record that we at least tried.
 358     logfile = utils.open_file(cnf["Rm::LogFile"], 'a')
 359     logfile.write("=========================================================================\n")
 360     logfile.write("[Date: %s] [ftpmaster: %s]\n" % (date, whoami))
 361     logfile.write("Removed the following packages from %s:\n\n%s" % (suites_list, summary))
 362     if Options["Done"]:
 363         logfile.write("Closed bugs: %s\n" % (Options["Done"]))
 364     logfile.write("\n------------------- Reason -------------------\n%s\n" % (Options["Reason"]))
 365     logfile.write("----------------------------------------------\n")
 366
 367     # Do the same in rfc822 format
 368     logfile822 = utils.open_file(cnf["Rm::LogFile822"], 'a')
 369     logfile822.write("Date: %s\n" % date)
 370     logfile822.write("Ftpmaster: %s\n" % whoami)
 371     logfile822.write("Suite: %s\n" % suites_list)
 372     sources = []
 373     binaries = []
 374     for package in summary.split("\n"):
 375         for row in package.split("\n"):
 376             element = row.split("|")
 377             if len(element) == 3:
 378                 if element[2].find("source") > 0:
 379                     sources.append("%s_%s" % tuple(elem.strip(" ") for elem in element[:2]))
 380                     element[2] = sub("source\s?,?", "", element[2]).strip(" ")
 381                 if element[2]:
 382                     binaries.append("%s_%s [%s]" % tuple(elem.strip(" ") for elem in element))
 383     if sources:
 384         logfile822.write("Sources:\n")
 385         for source in sources:
 386             logfile822.write(" %s\n" % source)
 387     if binaries:
 388         logfile822.write("Binaries:\n")
 389         for binary in binaries:
 390             logfile822.write(" %s\n" % binary)
 391     logfile822.write("Reason: %s\n" % Options["Reason"].replace('\n', '\n '))
 392     if Options["Done"]:
 393         logfile822.write("Bug: %s\n" % Options["Done"])
 394
 395     dsc_type_id = get_override_type('dsc', session).overridetype_id
 396     deb_type_id = get_override_type('deb', session).overridetype_id
 397
 398     # Do the actual deletion
 399     print "Deleting...",
 400     sys.stdout.flush()
 401
 402     for i in to_remove:
 403         package = i[0]
 404         architecture = i[2]
 405         package_id = i[3]
 406         for suite_id in suite_ids_list:
 407             if architecture == "source":
 408                 session.execute("DELETE FROM src_associations WHERE source = :packageid AND suite = :suiteid",
 409                                 {'packageid': package_id, 'suiteid': suite_id})
 410                 #print "DELETE FROM src_associations WHERE source = %s AND suite = %s" % (package_id, suite_id)
 411             else:
 412                 session.execute("DELETE FROM bin_associations WHERE bin = :packageid AND suite = :suiteid",
 413                                 {'packageid': package_id, 'suiteid': suite_id})
 414                 #print "DELETE FROM bin_associations WHERE bin = %s AND suite = %s" % (package_id, suite_id)
 415             # Delete from the override file
 416             if not Options["Partial"]:
 417                 if architecture == "source":
 418                     type_id = dsc_type_id
 419                 else:
 420                     type_id = deb_type_id
 421                 # TODO: Again, fix this properly to remove the remaining non-bind argument
 422                 session.execute("DELETE FROM override WHERE package = :package AND type = :typeid AND suite = :suiteid %s" % (over_con_components), {'package': package, 'typeid': type_id, 'suiteid': suite_id})
 423     session.commit()
 424     print "done."
 425
 426     # If we don't have a Bug server configured, we're done
 427     if not cnf.has_key("Dinstall::BugServer"):
 428         if Options["Done"] or Options["Do-Close"]:
 429             print "Cannot send mail to BugServer as Dinstall::BugServer is not configured"
 430
 431         logfile.write("=========================================================================\n")
 432         logfile.close()
 433
 434         logfile822.write("\n")
 435         logfile822.close()
 436
 437         return
 438
 439     # read common subst variables for all bug closure mails
 440     Subst_common = {}
 441     Subst_common["__RM_ADDRESS__"] = cnf["Dinstall::MyEmailAddress"]
 442     Subst_common["__BUG_SERVER__"] = cnf["Dinstall::BugServer"]
 443     Subst_common["__CC__"] = "X-DAK: dak rm"
 444     if carbon_copy:
 445         Subst_common["__CC__"] += "\nCc: " + ", ".join(carbon_copy)
 446     Subst_common["__SUITE_LIST__"] = suites_list
 447     Subst_common["__SUBJECT__"] = "Removed package(s) from %s" % (suites_list)
 448     Subst_common["__ADMIN_ADDRESS__"] = cnf["Dinstall::MyAdminAddress"]
 449     Subst_common["__DISTRO__"] = cnf["Dinstall::MyDistribution"]
 450     Subst_common["__WHOAMI__"] = whoami
 451
 452     # Send the bug closing messages
 453     if Options["Done"]:
 454         Subst_close_rm = Subst_common
 455         bcc = []
 456         if cnf.find("Dinstall::Bcc") != "":
 457             bcc.append(cnf["Dinstall::Bcc"])
 458         if cnf.find("Rm::Bcc") != "":
 459             bcc.append(cnf["Rm::Bcc"])
 460         if bcc:
 461             Subst_close_rm["__BCC__"] = "Bcc: " + ", ".join(bcc)
 462         else:
 463             Subst_close_rm["__BCC__"] = "X-Filler: 42"
 464         summarymail = "%s\n------------------- Reason -------------------\n%s\n" % (summary, Options["Reason"])
 465         summarymail += "----------------------------------------------\n"
 466         Subst_close_rm["__SUMMARY__"] = summarymail
 467
 468         whereami = utils.where_am_i()
 469         Archive = get_archive(whereami, session)
 470         if Archive is None:
 471             utils.warn("Cannot find archive %s.  Setting blank values for origin" % whereami)
 472             Subst_close_rm["__PRIMARY_MIRROR__"] = ""
 473         else:
 474             Subst_close_rm["__PRIMARY_MIRROR__"] = Archive.primary_mirror
 475
 476         for bug in utils.split_args(Options["Done"]):
 477             Subst_close_rm["__BUG_NUMBER__"] = bug
 478             if Options["Do-Close"]:
 479                 mail_message = utils.TemplateSubst(Subst_close_rm,cnf["Dir::Templates"]+"/rm.bug-close-with-related")
 480             else:
 481                 mail_message = utils.TemplateSubst(Subst_close_rm,cnf["Dir::Templates"]+"/rm.bug-close")
 482             utils.send_mail(mail_message, whitelists=whitelists)
 483
 484     # close associated bug reports
 485     if Options["Do-Close"]:
 486         Subst_close_other = Subst_common
 487         bcc = []
 488         wnpp = utils.parse_wnpp_bug_file()
 489         versions = list(set([re_bin_only_nmu.sub('', v) for v in versions]))
 490         if len(versions) == 1:
 491             Subst_close_other["__VERSION__"] = versions[0]
 492         else:
 493             utils.fubar("Closing bugs with multiple package versions is not supported.  Do it yourself.")
 494         if bcc:
 495             Subst_close_other["__BCC__"] = "Bcc: " + ", ".join(bcc)
 496         else:
 497             Subst_close_other["__BCC__"] = "X-Filler: 42"
 498         # at this point, I just assume, that the first closed bug gives
 499         # some useful information on why the package got removed
 500         Subst_close_other["__BUG_NUMBER__"] = utils.split_args(Options["Done"])[0]
 501         if len(sources) == 1:
 502             source_pkg = source.split("_", 1)[0]
 503         else:
 504             utils.fubar("Closing bugs for multiple source packages is not supported.  Do it yourself.")
 505         Subst_close_other["__BUG_NUMBER_ALSO__"] = ""
 506         Subst_close_other["__SOURCE__"] = source_pkg
 507         merged_bugs = set()
 508         other_bugs = bts.get_bugs('src', source_pkg, 'status', 'open', 'status', 'forwarded')
 509         if other_bugs:
 510             for bugno in other_bugs:
 511                 if bugno not in merged_bugs:
 512                     for bug in bts.get_status(bugno):
 513                         for merged in bug.mergedwith:
 514                             other_bugs.remove(merged)
 515                             merged_bugs.add(merged)
 516             logfile.write("Also closing bug(s):")
 517             logfile822.write("Also-Bugs:")
 518             for bug in other_bugs:
 519                 Subst_close_other["__BUG_NUMBER_ALSO__"] += str(bug) + "-done@" + cnf["Dinstall::BugServer"] + ","
 520                 logfile.write(" " + str(bug))
 521                 logfile822.write(" " + str(bug))
 522             logfile.write("\n")
 523             logfile822.write("\n")
 524         if source_pkg in wnpp.keys():
 525             logfile.write("Also closing WNPP bug(s):")
 526             logfile822.write("Also-WNPP:")
 527             for bug in wnpp[source_pkg]:
 528                 # the wnpp-rm file we parse also contains our removal
 529                 # bugs, filtering that out
 530                 if bug != Subst_close_other["__BUG_NUMBER__"]:
 531                     Subst_close_other["__BUG_NUMBER_ALSO__"] += str(bug) + "-done@" + cnf["Dinstall::BugServer"] + ","
 532                     logfile.write(" " + str(bug))
 533                     logfile822.write(" " + str(bug))
 534             logfile.write("\n")
 535             logfile822.write("\n")
 536
 537         mail_message = utils.TemplateSubst(Subst_close_other,cnf["Dir::Templates"]+"/rm.bug-close-related")
 538         if Subst_close_other["__BUG_NUMBER_ALSO__"]:
 539             utils.send_mail(mail_message)
 540
 541
 542     logfile.write("=========================================================================\n")
 543     logfile.close()
 544
 545     logfile822.write("\n")
 546     logfile822.close()
 547
 548 #######################################################################################
 549
 550 if __name__ == '__main__':
 551     main()