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

[lintian] 10/13: r/html_reports: Stop warning about maintainers with "two names"



This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit 7a8a34567890cb7eeacb4bc18ddfb6affc558301
Author: Niels Thykier <niels@thykier.net>
Date:   Sat Feb 21 19:12:40 2015 +0100

    r/html_reports: Stop warning about maintainers with "two names"
    
    No body reads the warnings and for the past many years there has not
    been a complaint about how html_reports "picks" between the "two
    names".
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 reporting/html_reports | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/reporting/html_reports b/reporting/html_reports
index c12afd4..d18b0db 100755
--- a/reporting/html_reports
+++ b/reporting/html_reports
@@ -842,26 +842,20 @@ sub parse_lintian_log {
 }
 
 # Deduplicate maintainers.  Maintains a cache of the maintainers we've seen
-# with a given e-mail address, issues a warning if two maintainers have the
-# same e-mail address, and returns the maintainer string that we should use
-# (which is whatever maintainer we saw first with that e-mail).
+# with a given e-mail address and returns the maintainer string that we
+# should use (which is whatever maintainer we saw first with that e-mail).
 
 # PerlCritic apparently does not recognise this way of "encapsulating"
 # variables.  Though, these could probably be replaced by "state"
 # variables.
 ## no critic (ControlStructures::ProhibitUnreachableCode)
 {
-    my (%urlmap, %warned);
+    my (%urlmap);
 
     sub map_maintainer {
         my ($maintainer) = @_;
         my $url = maintainer_url($maintainer);
         if ($urlmap{$url} && $urlmap{$url} ne $maintainer) {
-            warn "$maintainer has the same page as $urlmap{$url}\n"
-              unless ($warned{$maintainer}
-                || lc($maintainer) eq lc($urlmap{$url})
-                || $maintainer =~ /\@lists\.(alioth\.)?debian\.org>/);
-            $warned{$maintainer}++;
             $maintainer = $urlmap{$url};
         } else {
             $urlmap{$url} = $maintainer;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: