The following commit has been merged in the master branch:
commit 66f927e62c0a0f62309c83549d96631c7d59fe3c
Author: Adam D. Barratt <[EMAIL PROTECTED]>
Date:   Sat Nov 29 18:26:49 2008 +0000

    Don't output a "start package" message for a .changes files.
    
    Don't output a "Processing binary package foo.changes (version )" message.
    Thanks Sandro Tosi for pointing the issue out.
    
    .changes files aren't really packages, but we pretend they're binary
    packages so that we can issue tags against them.
    
    We already output a message indicating we're processing a .changes
    file so the "start package" message is spurious and somewhat confusing.
    Maybe we should have a "changes" type rather than faking them as binary
    packages, but in the meantime this gets us back to the pre-::Output
    output.

diff --git a/debian/changelog b/debian/changelog
index 920c05c..fea113e 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -21,6 +21,9 @@ lintian (2.1.1) unstable; urgency=low
 
   * lib/Lab.pm:
     + [FL] Fix breakage of populate_with_dist() I introduced in 2.1.0.
+  * lib/Tags.pm:
+    + [ADB] Don't output a "Processing binary package foo.changes" message.
+      Thanks Sandro Tosi for pointing the issue out.
 
  -- Adam D. Barratt <[EMAIL PROTECTED]>  Fri, 28 Nov 2008 21:47:33 +0000
 
diff --git a/lib/Tags.pm b/lib/Tags.pm
index 310a8a1..51f30c3 100644
--- a/lib/Tags.pm
+++ b/lib/Tags.pm
@@ -131,7 +131,9 @@ sub select_pkg {
        $Lintian::Output::GLOBAL->print_end_pkg($info{$current});
     }
     $current = $file;
-    $Lintian::Output::GLOBAL->print_start_pkg($info{$current});
+    if ($file !~ /.changes$/) {
+       $Lintian::Output::GLOBAL->print_start_pkg($info{$current});
+    }
     return 1;
 }
 

-- 
Debian package checker


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to