[SCM] Debian package checker branch, master, updated. 2.3.4-33-g4d7501e

["Adam D. Barratt" <adam@adam-barratt.org.uk>]

The following commit has been merged in the master branch:
commit 5e2cfc4af3861ff18f660ff397785afcdd543a7c
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date:   Wed Jan 6 09:43:14 2010 +0000

    Fix .changes scheduling logic so that subfiles actually get scheduled
    
    Each of the add_*() functions return 1 on success, so the net effect of
    using ||= to accumulate the exit status of each subfile's scheduling was
    that as soon as one file was successfully scheduled all remaining files
    were skipped.

diff --git a/lib/Lintian/Schedule.pm b/lib/Lintian/Schedule.pm
index a61d1a6..7ffdd4f 100644
--- a/lib/Lintian/Schedule.pm
+++ b/lib/Lintian/Schedule.pm
@@ -131,15 +131,15 @@ sub add_changes {
 
 	my ($md5sum,$size,$section,$priority,$file) = split(/\s+/o, $_);
 	if ($file =~ /\.deb$/) {
-	    $status ||= $self->add_deb('b', "$arg_dir/$file");
+	    $status += $self->add_deb('b', "$arg_dir/$file");
 	} elsif ($file =~ /\.udeb$/) {
-	    $status ||= $self->add_deb('u', "$arg_dir/$file");
+	    $status += $self->add_deb('u', "$arg_dir/$file");
 	} elsif ($file =~ /\.dsc$/) {
-	    $status ||= $self->add_dsc("$arg_dir/$file");
+	    $status += $self->add_dsc("$arg_dir/$file");
 	}
     }                                    
 
-    return $status;
+    return ($status ? 0 : 1);
 }
 
 # for each package (the sort is to make sure that source packages are

-- 
Debian package checker