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

[SCM] Debian package checker branch, master, updated. 2.5.5-18-ga498e42



The following commit has been merged in the master branch:
commit a498e42796804c817a4f49f451efec13ccc232ca
Author: Niels Thykier <niels@thykier.net>
Date:   Fri Mar 9 22:46:10 2012 +0100

    harness: Fix inverted logic involving dry-run
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/debian/changelog b/debian/changelog
index f41e2bf..73ceeb7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -43,6 +43,9 @@ lintian (2.5.6) UNRELEASED; urgency=low
     + [NT] Fixed issue where "Experimental: no" was handled as a
       "yes" when generating a tag description.
 
+  * reporting/harness:
+    + [NT] Corrected inverted logic when checking for dry-run.
+
  -- Niels Thykier <niels@thykier.net>  Fri, 24 Feb 2012 09:40:30 +0100
 
 lintian (2.5.5) unstable; urgency=low
diff --git a/reporting/harness b/reporting/harness
index 411a21d..35213af 100755
--- a/reporting/harness
+++ b/reporting/harness
@@ -277,12 +277,13 @@ unless ($opt{'reports-only'}) {
         Log('Updating lintian.log...');
         my $nfd;
         if ($opt{'dry-run'}) {
-            rename $lintian_log, $old_lintian_log;
-            open $nfd, '>', $lintian_log
-                or Die ("cannot open lintian.log $lintian_log for writing: $!");
-        } else {
             open $nfd, '>', '/dev/null'
                 or Die ("cannot open lintian.log /dev/null for writing: $!");
+        } else {
+            rename $lintian_log, $old_lintian_log
+                or Die ("cannot rename lintian.log to $old_lintian_log: $!");
+            open $nfd, '>', $lintian_log
+                or Die ("cannot open lintian.log $lintian_log for writing: $!");
         }
         open my $ofd, '<', $old_lintian_log
             or Die ("cannot open old lintian.log $old_lintian_log for reading: $!");

-- 
Debian package checker


Reply to: