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

[SCM] Debian package checker branch, master, updated. 2.0.0-36-g64c17bd



The following commit has been merged in the master branch:
commit 64c17bda8481d6e51355bea107a5ebe61314d67f
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
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


Reply to: