From: Joerg Jaspert <joerg@debian.org>
Date: Thu, 7 Aug 2008 14:18:26 +0000 (+0200)
Subject: And another place to kick nfu
X-Git-Url: https://git.decadent.org.uk/gitweb/?a=commitdiff_plain;h=e0c19fd3f6c344ca20dd9e067ae76cfb624df467;p=dak.git

And another place to kick nfu
---

diff --git a/ChangeLog b/ChangeLog
index d29d7f74..9ee80a8c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,8 @@
 	* dak/cruft_report.py (parse_nfu): call utils.warn instead of warn
 	(main): Only do the nfu stuff if nfu is a check we want to run
 	later.
+	(main): And another place where we dont want to do nfu foo unless
+	we need nfu
 
 	* dak/make_suite_file_list.py (main): Fix a bug that has been
 	there for ages, but "just" never triggered.
diff --git a/dak/cruft_report.py b/dak/cruft_report.py
index 8640ea61..fab47bf0 100755
--- a/dak/cruft_report.py
+++ b/dak/cruft_report.py
@@ -472,9 +472,10 @@ def main ():
                         duplicate_bins.setdefault(key, [])
                         if package not in duplicate_bins[key]:
                             duplicate_bins[key].append(package)
-                    if package in nfu_entries and \
-                        version != source_versions[source]: # only suggest to remove out-of-date packages
-                        nfu_packages[architecture].append((package,version,source_versions[source]))
+                    if "nfu" in checks:
+                        if package in nfu_entries and \
+                               version != source_versions[source]: # only suggest to remove out-of-date packages
+                            nfu_packages[architecture].append((package,version,source_versions[source]))
                     
             packages.close()
             os.unlink(temp_filename)