[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, lab-refactor, updated. 2.5.3-188-g35e4734



The following commit has been merged in the lab-refactor branch:
commit 35e47344877ddc5a080963a85d5e1f5c4d0d8802
Author: Niels Thykier <niels@thykier.net>
Date:   Wed Nov 2 11:25:30 2011 +0100

    Update the status file after auto-cleaning has been done
    
    Otherwise the status file will also include the auto-removed
    collections.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/frontend/lintian b/frontend/lintian
index 751f3f5..d4cfeb4 100755
--- a/frontend/lintian
+++ b/frontend/lintian
@@ -1531,13 +1531,6 @@ sub unpack_group {
             debug_msg(1, "Reap done jobs ... unpack $pkg_name $pkg_ver [$pkg_arch] ($pkg_type)");
         }
 
-        # All successful, make sure to record it so we do not unpack the same package
-        # in a later run (mostly for archive-wide checks).
-        unless ($lpkg->update_status_file) {
-            warning("could not create status file for package $pkg_name: $!");
-        }
-
-
         if ($action eq 'check') {
             # We only need this if we are checking the package later
             $proc->lab_pkg($lpkg);
@@ -1547,6 +1540,12 @@ sub unpack_group {
                 auto_clean_package($lpkg) or $exit_code = 2;
             }
         }
+        # All successful, make sure to record it so we do not unpack the same package
+        # in a later run (mostly for archive-wide checks).
+        unless ($lpkg->update_status_file) {
+            warning("could not create status file for package $pkg_name: $!");
+        }
+
     }
     return 1;
 }
@@ -1639,6 +1638,12 @@ sub process_group {
 
         if (!$keep_lab) {
             auto_clean_package($lpkg) or $exit_code = 2;
+            # Update the status file as auto_clean_package may have removed some
+            # collections
+            unless ($lpkg->update_status_file) {
+                warning("could not create status file for package $pkg_name: $!");
+            }
+
         }
 
     } # end foreach my $proc ($group->get_processable())

-- 
Debian package checker


Reply to: