summaryrefslogtreecommitdiff
path: root/Porting/Maintainers.pm
diff options
context:
space:
mode:
authorFlorian Ragwitz <rafl@debian.org>2011-07-07 15:44:45 +0200
committerFlorian Ragwitz <rafl@debian.org>2011-07-07 15:55:03 +0200
commit8ceba1a470d4a87a27f40e4a0187b7d28c6d4912 (patch)
tree79ce2487fede825885eeb87cc18768fe39842ca8 /Porting/Maintainers.pm
parentd8d7ae6469c0196eef4d10482329c6aded9eb169 (diff)
downloadperl-8ceba1a470d4a87a27f40e4a0187b7d28c6d4912.tar.gz
Make sure there's no superfluous Maintainer.pl entries
Diffstat (limited to 'Porting/Maintainers.pm')
-rw-r--r--Porting/Maintainers.pm20
1 files changed, 19 insertions, 1 deletions
diff --git a/Porting/Maintainers.pm b/Porting/Maintainers.pm
index f1016b44eb..c98413a241 100644
--- a/Porting/Maintainers.pm
+++ b/Porting/Maintainers.pm
@@ -314,8 +314,9 @@ sub show_results {
: sub { /\.(?:[chty]|p[lm]|xs)\z/msx },
@Files
);
- } else {
+ } else {
duplicated_maintainers();
+ superfluous_maintainers();
}
} elsif (@Files) {
my $ModuleByFile = files_to_modules(@Files);
@@ -391,6 +392,23 @@ sub missing_maintainers {
find sub { warn_maintainer($File::Find::name) if $check->() }, @dir if @dir;
}
+sub superfluous_maintainers {
+ maintainers_files();
+ for my $f (keys %files) {
+ if ($TapOutput) {
+ if ($MANIFEST{$f}) {
+ print "ok ".++$TestCounter." - Maintained file $f appears in MANIFEST\n";
+ } else {
+ print "not ok ".++$TestCounter." - File $f has has a maintainer but is not in MANIFEST\n";
+ }
+ } else {
+ if (!$MANIFEST{$f}) {
+ warn "File $f has has a maintainer but is not in MANIFEST\n";
+ }
+ }
+ }
+}
+
sub finish_tap_output {
print "1..".$TestCounter."\n";
}