use 5.008;
require "Maintainers.pl";
-use vars qw(%Modules %Maintainers);
+our (%Modules, %Maintainers);
-use vars qw(@ISA @EXPORT_OK $VERSION);
-@ISA = qw(Exporter);
-@EXPORT_OK = qw(%Modules %Maintainers
+our @ISA = qw(Exporter);
+our @EXPORT_OK = qw(%Modules %Maintainers
get_module_files get_module_pat
show_results process_options files_to_modules
finish_tap_output
reload_manifest);
-$VERSION = 0.08;
+our $VERSION = 0.13;
require Exporter;
$manifest_path = "../MANIFEST";
}
- if (open(my $manfh, $manifest_path )) {
+ if (open(my $manfh, '<', $manifest_path )) {
while (<$manfh>) {
if (/^(\S+)/) {
$MANIFEST{$1}++;
split ' ', $Modules{$m}{FILES};
}
-# exand dir/ or foo* into a full list of files
+# expand dir/ or foo* into a full list of files
#
sub expand_glob {
sort { lc $a cmp lc $b }
if (@ToDo) {
# Try prefix matching.
- # Need to try longst prefixes first, else lib/CPAN may match
+ # Need to try longest prefixes first, else lib/CPAN may match
# lib/CPANPLUS/... and similar
my @OrderedModuleByPat
sub duplicated_maintainers {
maintainers_files();
- for my $f (keys %files) {
+ for my $f (sort keys %files) {
cmp_ok($files{$f}, '<=', 1, "File $f appears $files{$f} times in Maintainers.pl");
}
}
sub warn_maintainer {
my $name = shift;
- ok($files{$name}, "$name has a maintainer");
+ ok($files{$name}, "$name has a maintainer (see Porting/Maintainer.pl)");
}
sub missing_maintainers {
sub superfluous_maintainers {
maintainers_files();
- for my $f (keys %files) {
+ for my $f (sort keys %files) {
ok($MANIFEST{$f}, "File $f has a maintainer and is in MANIFEST");
}
}