Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
tree: b6d5fd8602
Fetching contributors…

Octocat-spinner-32-eaf2f5

Cannot retrieve contributors at this time

file 182 lines (154 sloc) 5.74 kb
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181
package App::LintPrereqs;

use 5.010;
use strict;
use warnings;
use Log::Any qw($log);

use Config::IniFiles;
use File::Find;
use File::Which;
use Sort::Versions;

our %SPEC;
require Exporter;
our @ISA = qw(Exporter);
our @EXPORT_OK = qw(lint_prereqs);

# VERSION

$SPEC{lint_prereqs} = {
    v => 1.1,
    summary => 'Check extraneous/missing prerequisites in dist.ini',
    description => <<'_',

Check [Prereqs / *] sections in your dist.ini against what's actually being used
in your Perl code (using Perl::PrereqScanner) and what's in Perl core list of
modules. Will complain if your prereqs is not actually used, or already in Perl
core. Will also complain if there are missing prereqs.

Designed to work with prereqs that are manually written. Does not work if you
use AutoPrereqs.

Sometimes there are prerequisites that you know are used but can't be detected
by scan_prereqs, or you want to include anyway. If this is the case, you can
instruct lint_prereqs to assume the prerequisite is used.

;!lint-prereqs assume-used # even though we know it is not currently used
Foo::Bar=0
;!lint-prereqs assume-used # we are forcing a certain version
Baz=0.12

Sometimes there are also prerequisites that are detected by scan_prereqs, but
you know are already provided by some other modules. So to make lint-prereqs
ignore them:

[Extras / lint-prereqs / assume-provided]
Qux::Quux=0

_
args => {
default_perl_version => {
schema => [str => {default=>'5.010000'}],
summary => 'Perl version to use when unspecified',
},
},
};
sub lint_prereqs {
my %args = @_;

(-f "dist.ini")
or return [412, "No dist.ini found, ".
"is your dist managed by Dist::Zilla?"];

my $cfg = Config::IniFiles->new(-file => "dist.ini", -fallback => "ALL");
$cfg or return [
500, "Can't open dist.ini: ".join(", ", @Config::IniFiles::errors)];

my %mods_from_ini;
my %assume_used;
my %assume_provided;
for my $section (grep {
m!^(prereqs|extras \s*/\s* lint[_-]prereqs \s*/\s*
assume-(?:provided|used))!ix}
$cfg->Sections) {
for my $param ($cfg->Parameters($section)) {
my $v = $cfg->val($section, $param);
my $cmt = $cfg->GetParameterComment($section, $param) // "";
#$log->tracef("section=$section, param=$param, v=$v, cmt=$cmt");
$mods_from_ini{$param} = $v unless $section =~ /assume-provided/;
$assume_provided{$param} = $v if $section =~ /assume-provided/;
$assume_used{$param} = $v if $section =~ /assume-used/ ||
$cmt =~ /^;!lint-prereqs\s+assume-used\b/m;
}
}
$log->tracef("mods_from_ini: %s", \%mods_from_ini);
$log->tracef("assume_used: %s", \%assume_used);
$log->tracef("assume_provided: %s", \%assume_provided);

# assume package names from filenames, should be better and scan using PPI
my %pkgs;
find({
#no_chdir => 1,
wanted => sub {
return unless /\.pm$/;
my $pkg = $File::Find::dir;
$pkg =~ s!^lib/!!;
$pkg =~ s!/!::!g;
$pkg .= "::$_";
$pkg =~ s/\.pm$//;
$pkgs{$pkg}++;
},
}, "lib");
$log->tracef("Packages: %s", \%pkgs);

my %mods_from_scanned;
my $sppath = which("scan_prereqs")
or return [412, "Can't find scan_prereqs in PATH"];
my $spcmd = "$sppath --combine .";
$spcmd .= " t/*.t" if <t/*.t>;
$spcmd .= " bin/*" if <bin/*>;
$spcmd .= " examples/*" if <examples/*>;
for (`$spcmd`) {
chomp;
/^([\w:]+)\s*=\s*(.+)/ or do {
warn "Invalid line from $sppath: $_, skipped";
next;
};
$mods_from_scanned{$1} = $2;
}
$log->tracef("mods_from_scanned: %s", \%mods_from_scanned);

my $perlv = $mods_from_ini{perl} // $mods_from_scanned{perl} // '5.010000';

my %core_mods;
my $clpath = which("corelist")
or return [412, "Can't find corelist in PATH"];
for (`$clpath -v $perlv`) {
chomp;
/^([\w:]+)(?:\s+(\S+))?\s*$/ or next;
#do {
# warn "Invalid line from $clpath: $_, skipped";
# next;
#};
$core_mods{$1} = $2 // 0;
}

my $err;

for my $mod (keys %mods_from_ini) {
next if $mod eq 'perl';
$log->tracef("Checking mod from dist.ini: %s", $mod);
if (exists($core_mods{$mod}) &&
versioncmp($core_mods{$mod}, $mods_from_ini{$mod}) >= 0) {
$log->warnf("Module is core, but mentioned in dist.ini: %s", $mod);
$err++;
}
unless (exists($mods_from_scanned{$mod}) ||
exists($assume_used{$mod})) {
$log->warnf("Module doesn't seem to be used, ".
"but mentioned in dist.ini: %s", $mod);
$err++;
}
}

for my $mod (keys %mods_from_scanned) {
next if $mod eq 'perl';
        $log->tracef("Checking mod from scanned: %s", $mod);
        next if exists $core_mods{$mod}; # XXX check version
        next if exists $pkgs{$mod};
        unless (exists($mods_from_ini{$mod}) ||
                    exists($assume_provided{$mod})) {
            $log->errorf("Module is used, but not mentioned in dist.ini: %s",
                         $mod);
            $err++;
        }
    }

    $err ?
        [500, "Extraneous/missing dependencies", undef,
         {"cmdline.display_result"=>0}] :
            [200, "OK"];
}

1;
#ABSTRACT: Check extraneous/missing prerequisites in dist.ini

=head1 SYNOPSIS

# Use via lint-prereqs CLI script

=cut
Something went wrong with that request. Please try again.