The following commit has been merged in the master branch:
commit 64c17bda8481d6e51355bea107a5ebe61314d67f
Author: Adam D. Barratt <[EMAIL PROTECTED]>
Date:   Fri Nov 7 19:26:55 2008 +0000

    Don't emit unused-override in situations where we can't know it's unused
    
    Don't complain about unused overrides if the check script for the
    overriden tag hasn't been run (or --tags was used and the overriden
    tag not specified).  (Closes: #501722)

diff --git a/debian/changelog b/debian/changelog
index d57119c..0681838 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -33,6 +33,9 @@ lintian (2.1.0) unstable; urgency=low
     + [FL] Convert to use Lintian::Output.
     + [CW] Add jaunty as an allowable distribution for Ubuntu, and remove
       the EOLed edgy and feisty.
+    + [ADB] Don't complain about unused overrides if the check script
+      for the overriden tag hasn't been run (or --tags was used and
+      the overriden tag not specified).  (Closes: #501722)
   
   * lib/Lintian/Output:
     + [FL] New module to bundle output methods.
diff --git a/frontend/lintian b/frontend/lintian
index cc65e35..fe9a823 100755
--- a/frontend/lintian
+++ b/frontend/lintian
@@ -1544,6 +1544,16 @@ foreach my $pkg_info ($schedule->get_all) {
            my $overrides = Tags::get_overrides( $file );
 
            for my $tag (sort keys %$overrides) {
+               my $taginfo = Tags::get_tag_info{$tag};
+               if (defined $taginfo) {
+                   # Did we run the check script containing the tag?
+                   next unless $checks{$taginfo->{'script'}};
+
+                   # If only checking specific tags, is this one of them?
+                   next unless (scalar keys %Tags::only_issue_tags == 0)
+                       or exists $Tags::only_issue_tags{$tag};
+               }
+
                for my $extra (sort keys %{$overrides->{$tag}}) {
                    next if $overrides->{$tag}{$extra};
 

-- 
Debian package checker


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

Reply via email to