And another place to kick nfu

diff --git a/ChangeLog b/ChangeLog
index d29d7f7427aa728baaca0b5694936be374ecfb76..9ee80a8c94ac771e87ec13576fbc1a8debedf8c4 100644 (file)
--- 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 8640ea6163e02581f86885717ce1742d61520763..fab47bf056d72bf910fd774ed48c45e584487ce6 100755 (executable)
--- 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)