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

[SCM] Debian package checker branch, lab-refactor, updated. 2.5.3-186-gd1570a8



The following commit has been merged in the lab-refactor branch:
commit 79e62bed57ac7368afaf20601904a5f2b61e6f94
Author: Niels Thykier <niels@thykier.net>
Date:   Wed Nov 2 10:05:17 2011 +0100

    lintian: Always write the entry status file after unpacking
    
    The status file keeps track of which collections have been run, so it
    is now unconditionally run after unpacking.
    
    If a collection errors out the status file is not updated.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/frontend/lintian b/frontend/lintian
index a6df760..751f3f5 100755
--- a/frontend/lintian
+++ b/frontend/lintian
@@ -1531,6 +1531,13 @@ 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);
@@ -1539,12 +1546,6 @@ sub unpack_group {
             if (!$keep_lab) {
                 auto_clean_package($lpkg) or $exit_code = 2;
             }
-
-            # All successful, make sure to record it so we do not recheck the same package
-            # in a later run (mostly for archive-wide checks).
-            if ($lpkg->update_status_file($LINTIAN_VERSION) < 1) {
-                warning("could not create status file for package $pkg_name: $!");
-            }
         }
     }
     return 1;
@@ -1640,12 +1641,6 @@ sub process_group {
             auto_clean_package($lpkg) or $exit_code = 2;
         }
 
-        # All successful, make sure to record it so we do not recheck the same package
-        # in a later run (mostly for archive-wide checks).
-        if ($lpkg->update_status_file($LINTIAN_VERSION) < 1) {
-            warning("could not create status file for package $pkg_name: $!");
-        }
-
     } # end foreach my $proc ($group->get_processable())
 
     return 1;

-- 
Debian package checker


Reply to: