The following commit has been merged in the lab-refactor branch:
commit 79e62bed57ac7368afaf20601904a5f2b61e6f94
Author: Niels Thykier <[email protected]>
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 <[email protected]>

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


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to