[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 3bbcc3b499845bbe7d7ba7e3ed1685af7c273f4e
Author: Niels Thykier <niels@thykier.net>
Date:   Wed Jul 11 17:40:14 2012 +0200

    c/manpages: Close the right handle and reap man process
    
    Otherwise, the manpage pids will remain zombies until the unpack loop
    for the next group reaps them.  However, it cannot properly handle it,
    causing Lintian to break with process multiple processes.
    
    This fixes a regression introduced in 3c351b6.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/checks/manpages b/checks/manpages
index 4b27afb..daa53e6 100644
--- a/checks/manpages
+++ b/checks/manpages
@@ -288,7 +288,9 @@ foreach my $file ($info->sorted_index) {
             tag 'manpage-has-errors-from-man', $file, $_;
             last;
         }
-        close(MANERRS);
+        close $read;
+        # reap man process
+        waitpid $pid, 0;
         # Now we search through the whole man page for some common errors
         my $lc = 0;
         my $hc = 0;

-- 
Debian package checker


Reply to: