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

[SCM] Debian package checker branch, master, updated. 2.5.10-18-g72dc4c3



The following commit has been merged in the master branch:
commit 72dc4c32cf9f4eca0933ee59448994f5144b4a29
Author: Niels Thykier <niels@thykier.net>
Date:   Wed Jul 11 17:59:01 2012 +0200

    lintian: Bail if a check leaves a zombie process behind
    
    This check should catch regressions like the one fixed in 3bbcc3b
    earlier.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/frontend/lintian b/frontend/lintian
index ae9d4f2..2d825c2 100755
--- a/frontend/lintian
+++ b/frontend/lintian
@@ -26,6 +26,7 @@ use strict;
 use warnings;
 
 use Getopt::Long;
+use POSIX qw(:sys_wait_h);
 
 # }}}
 
@@ -1174,6 +1175,10 @@ foreach my $gname (sort $pool->get_group_names()) {
     if ($action eq 'check'){
         process_group($group);
         clear_group_cache($group);
+        # Double check that no processes are running;  hopefully
+        # it will catch regressions like 3bbcc3b earlier.
+        fail ("Unreaped processes after running checks!?")
+            if waitpid (-1, WNOHANG) != -1;
     }
 }
 

-- 
Debian package checker


Reply to: