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

[SCM] Debian package checker branch, master, updated. 2.5.11-4-g8eb4c67



The following commit has been merged in the master branch:
commit 8eb4c673560888bf4e2c37d6a7436b0e00643ad9
Author: Niels Thykier <niels@thykier.net>
Date:   Thu Dec 13 20:49:39 2012 +0100

    harness: Fix handling of lintian exiting with 1
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/reporting/harness b/reporting/harness
index f28f162..ece233a 100755
--- a/reporting/harness
+++ b/reporting/harness
@@ -416,10 +416,11 @@ unless ($opt{'reports-only'}) {
                 # exit 1 (policy violations) happens all the time (sadly)
                 # exit 2 (broken packages) also happens all the time...
                 my $res = ($? >> 8) & 0xff;
+                my $sig = $? & 0xff;
                 if ($res != 1 and $res != 0) {
                     Log ("warning: executing lintian returned $res");
-                } else {
-                    Log ("Lintian terminated by signal: $?");
+                } elsif ($sig) {
+                    Log ("Lintian terminated by signal: $sig");
                     # If someone is sending us signals (e.g. SIGINT/Ctrl-C)
                     # don't start the next round.
                     #  - 35k / 512 => 700 rounds (or 700 SIGINTs to stop...)

-- 
Debian package checker


Reply to: