[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: