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

[lintian] 01/01: harness: Always update the log under incremental runs



This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit db3b423109c7ac5264c69402eb1acf994a6f6e48
Author: Niels Thykier <niels@thykier.net>
Date:   Thu Sep 26 09:00:21 2013 +0200

    harness: Always update the log under incremental runs
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 debian/changelog  |    3 +++
 reporting/harness |    5 ++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 5615e9f..30f93ed 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -37,6 +37,9 @@ lintian (2.5.19) UNRELEASED; urgency=low
       run lintian.
     + [NT] Remove LINTIAN_BIN_DIR; it is no longer used to
       avoid signature checking.
+    + [NT] Fix a bug where harness would not remove entries in
+      the Lintian log for removed packages if no packages were
+      to be processed.
   * reporting/lintian-dummy.cfg:
     + [NT] Remove file as it is now obsolete.
 
diff --git a/reporting/harness b/reporting/harness
index bf477e9..5a3d038 100755
--- a/reporting/harness
+++ b/reporting/harness
@@ -321,8 +321,11 @@ unless ($opt{'reports-only'}) {
     # Throw away the lab to avoid it hogging memory.
     $LAB = undef;
 
-    if (@worklist and $opt{'incremental-mode'}) {
+    if ($opt{'incremental-mode'}) {
         # Extra work for the incremental run
+        # - do this regardless of @worklist, since we could in theory
+        #   have an "only removals"-run.  In that case, their entries
+        #   should still be removed from the log.
 
         die "Old Lintian log file $lintian_log not found!\n"
           unless -f $lintian_log;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: