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

[SCM] Debian package checker branch, master, updated. 2.2.8-8-g95a3cde



The following commit has been merged in the master branch:
commit 95a3cdedcdce07bec67ebde294313520d2950eab
Author: Russ Allbery <rra@debian.org>
Date:   Thu Mar 26 19:43:37 2009 -0700

    Really fix the return code from Lintian::Command::kill
    
    We want to return success only if all the kill_kill calls returned
    success.

diff --git a/debian/changelog b/debian/changelog
index f9db1dd..17d71e3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -20,7 +20,7 @@ lintian (2.2.9) UNRELEASED; urgency=low
     + [RA] Belatedly bump the lab format version for many recent changes.
 
   * lib/Lintian/Command.pm:
-    + [RA] In kill, initialize the result status and use ||= instead of |=.
+    + [RA] In kill, initialize the result status and use &&= instead of |=.
       Partly addresses #521257.
 
  -- Frank Lichtenheld <djpig@debian.org>  Mon, 16 Mar 2009 00:13:30 +0100
diff --git a/lib/Lintian/Command.pm b/lib/Lintian/Command.pm
index 1f935d2..05fb3ad 100644
--- a/lib/Lintian/Command.pm
+++ b/lib/Lintian/Command.pm
@@ -303,9 +303,9 @@ all the executions of kill_kill.
 =cut
 
 sub kill {
-    my $status = 0;
+    my $status = 1;
     while (my $opts = shift @_) {
-	$status ||= kill_kill($opts->{'harness'}, grace => 2);
+	$status &&= kill_kill($opts->{'harness'}, grace => 2);
     }
     return $status;
 }

-- 
Debian package checker


Reply to: