[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: