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

[SCM] Debian package checker branch, master, updated. 2.5.13-3-gbb252c7



The following commit has been merged in the master branch:
commit bb252c784ed2ea344055350efaf612b8f3c92d3b
Author: Niels Thykier <niels@thykier.net>
Date:   Thu May 30 13:42:30 2013 +0200

    r/harness: Add a query to the worklist at most once
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/debian/changelog b/debian/changelog
index 7ebaf25..9f01aa3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,10 @@ lintian (2.5.14) UNRELEASED; urgency=low
   * lib/Lintian/{Path,Util}.pm:
     + [NT] Remove deprecated methods.
 
+  * reporting/harness:
+    + [NT] Fix bug where harness might schedule the same group
+      multiple times.  (Closes: #710380)
+
  -- Niels Thykier <niels@thykier.net>  Thu, 30 May 2013 10:38:26 +0200
 
 lintian (2.5.13) unstable; urgency=low
diff --git a/reporting/harness b/reporting/harness
index e971369..788d15f 100755
--- a/reporting/harness
+++ b/reporting/harness
@@ -313,6 +313,7 @@ unless ($opt{'reports-only'}) {
             my @res = $LAB->lab_query ($query);
             if (@res) {
                 # Group is still in the Lab - reprocess it together
+                push(@worklist, $query);
                 foreach my $entry (@res) {
                     # Remove old log entry for all entries in this group
                     # NB: This part is unable to replace the update to %skip
@@ -321,7 +322,6 @@ unless ($opt{'reports-only'}) {
                     # but we will see each of the entries removed in the
                     # loop above.
                     $skip{$entry->identifier} = 1;
-                    push @worklist, $query;
                 }
             }
         }

-- 
Debian package checker


Reply to: