lintian: r1045 - in trunk: debian reporting

[rra@debian.org]

Author: rra
Date: 2007-12-04 22:03:32 +0100 (Tue, 04 Dec 2007)
New Revision: 1045

Modified:
   trunk/debian/changelog
   trunk/reporting/html_reports
Log:
* reporting/html_reports:
  + [RA] Change the layout of the per-maintainer lintian pages.  Now,
    each report will be in /reports/maintainer/<email>.html where
    <email> is the maintainer's e-mail address with unsafe characters
    replaced by _.  This should allow better links from the QA pages.
    Thanks, Stefano Zacchiroli.

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2007-12-04 19:47:32 UTC (rev 1044)
+++ trunk/debian/changelog	2007-12-04 21:03:32 UTC (rev 1045)
@@ -113,12 +113,19 @@
   * lib/Tags.pm:
     + [RA] Keep statistics on overrides by severity.
 
+  * reporting/html_reports:
+    + [RA] Change the layout of the per-maintainer lintian pages.  Now,
+      each report will be in /reports/maintainer/<email>.html where
+      <email> is the maintainer's e-mail address with unsafe characters
+      replaced by _.  This should allow better links from the QA pages.
+      Thanks, Stefano Zacchiroli.
+
   * unpack/list-srcpkg:
     + [RA] Escape ; and newline characters in all package fields before
       writing them to the index (uncovered when someone accidentally
       appended a package description to Standards-Version).
 
- -- Russ Allbery <rra@debian.org>  Tue, 04 Dec 2007 00:59:20 -0800
+ -- Russ Allbery <rra@debian.org>  Tue, 04 Dec 2007 13:02:52 -0800
 
 lintian (1.23.36) unstable; urgency=low
 

Modified: trunk/reporting/html_reports
===================================================================
--- trunk/reporting/html_reports	2007-12-04 19:47:32 UTC (rev 1044)
+++ trunk/reporting/html_reports	2007-12-04 21:03:32 UTC (rev 1045)
@@ -76,6 +76,8 @@
 # Create output directories
 mkdir($HTML_TMP_DIR,0777)
     or die "cannot create output directory $HTML_TMP_DIR: $!";
+mkdir("$HTML_TMP_DIR/maintainer",0777)
+    or die "cannot create output directory $HTML_TMP_DIR/maintainer: $!";
 
 my ($num_errors, $num_warnings, $num_experimental, $num_overridden, $num_info);
 $num_errors = $num_warnings = $num_experimental = $num_overridden = $num_info = 0;
@@ -478,16 +480,15 @@
 
     $maint = $_[0];
 
-    my $file = $maint;
+    # The maintainer page should be maintainer/<email>.html where <email> is
+    # their email address with all characters other than a-z A-Z 0-9 - _ . @ =
+    # + replaced with _.  Don't change this without coordinating with QA.
+    my ($file) = ($maint =~ /<([^>]+)>/);
     if ($file) {
-	$file =~ s/^(.+)\<.*$/$1/;
-	$file =~ tr/A-Za-z0-9_.,/_/c;
-	$file =~ s/^_+//g;
-	$file =~ s/_+$//g;
-
-	$file = "m$file.html";
+	$file =~ tr/a-zA-Z0-9_.@=+-/_/c;
+	$file = "maintainer/$file.html";
     } else {
-	$file = "munsorted.html";
+	$file = "maintainer/unsorted.html";
     }
 
     open_maintainer($file);