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

[SCM] Debian package checker branch, master, updated. 2.2.15-1-g3a79f6b



The following commit has been merged in the master branch:
commit 3a79f6bad3a4b9f9fc02f53fa3ee7853877f3431
Author: Russ Allbery <rra@debian.org>
Date:   Mon Sep 14 18:37:31 2009 -0700

    Include all tag types in the qa-list.txt report
    
    * reporting/html_reports:
      + [RA] Include all tag types in the qa-list.txt file instead of just
        errors and warnings.  The new order of columns is errors, warnings,
        info tags, pedantic tags, experimental tags, and overridden tags.

diff --git a/debian/changelog b/debian/changelog
index 316ea8e..d43d6d4 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
+lintian (2.2.16) UNRELEASED; urgency=low
+
+  * reporting/html_reports:
+    + [RA] Include all tag types in the qa-list.txt file instead of just
+      errors and warnings.  The new order of columns is errors, warnings,
+      info tags, pedantic tags, experimental tags, and overridden tags.
+
+ -- Russ Allbery <rra@debian.org>  Mon, 14 Sep 2009 18:37:26 -0700
+
 lintian (2.2.15) unstable; urgency=low
 
   The "BTS cleanup" release.
diff --git a/reporting/html_reports b/reporting/html_reports
index ba75f78..01f887d 100755
--- a/reporting/html_reports
+++ b/reporting/html_reports
@@ -341,20 +341,20 @@ for my $maintainer (@maintainers) {
     delete $clean{$id};
 
     # For each of this maintainer's packages, add statistical information
-    # about warnings and errors to the QA list and build the packages hash
-    # used for the package index.  We only do this for the maintainer
-    # packages, not the uploader packages, to avoid double-counting.
+    # about the number of each type of tag to the QA data and build the
+    # packages hash used for the package index.  We only do this for the
+    # maintainer packages, not the uploader packages, to avoid
+    # double-counting.
     for my $source (keys %{ $by_maint{$maintainer} }) {
-        my ($errors, $warnings) = (0, 0);
+        my %count;
         for my $version (keys %{ $by_maint{$maintainer}{$source} }) {
             my $tags = $by_maint{$maintainer}{$source}{$version};
             for my $tag (@$tags) {
-                $errors++ if $tag->{code} eq 'E';
-                $warnings++ if $tag->{code} eq 'W';
+                $count{$tag->{code}}++;
                 $packages{$tag->{package}} = $tag->{xref};
             }
         }
-        $qa{$source} = [ $errors, $warnings ];
+        $qa{$source} = \%count;
     }
 
     # Determine if the maintainer's page is clean.  Check all packages for
@@ -415,11 +415,18 @@ my %data = (
 );
 output_template ('maintainers.html', $templates{maintainers}, \%data);
 
-# Write out the QA package list.
+# Write out the QA package list.  This is a space-delimited file that contains
+# the package name and then the error count, warning count, info count,
+# pedantic count, experimental count, and overridden tag count.
 open (QA, '>', "$HTML_TMP_DIR/qa-list.txt")
     or die "cannot create qa-list.txt: $!\n";
 for my $source (sort keys %qa) {
-    print QA "$source $qa{$source}[0] $qa{$source}[1]\n";
+    print QA $source;
+    for my $code (qw/E W I P X O/) {
+        my $count = $qa{$source}{$code} || 0;
+        print QA " $count";
+    }
+    print QA "\n";
 }
 close QA or die "cannot write to qa-list: $!\n";
 

-- 
Debian package checker


Reply to: