Makefile.PL: add test scripts to syntax checks

[public-inbox.git] / Makefile.PL

diff --git a/Makefile.PL b/Makefile.PL
index 3bb00725a4d8672fbc0ec1227284e63d25760f4d..238220726cab993dc6b0d5e61ae17ddebf018247 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -28,10 +28,20 @@ WriteMakefile(
                # We also depend on git.
                # Keep this sorted and synced to the INSTALL document
                'Date::Parse' => 0,
+
+               # libperl$PERL_VERSION,
+               # `perl5' on FreeBSD
+               # perl-Digest-SHA on RH-based
+               'Digest::SHA' => 0,
                'Email::MIME' => 0,
+
+               # the following should be pulled in by Email::MIME:
                'Email::MIME::ContentType' => 0,
                'Email::Simple' => 0,
-               'Encode::MIME::Header' => 0,
+
+               # libperl$PERL_VERSION or libencode-perl on Debian,
+               # `perl5' on FreeBSD
+               'Encode' => 0,
 
                # TODO: these should really be made optional...
                'Plack' => 0,
@@ -39,6 +49,14 @@ WriteMakefile(
 
                # We have more test dependencies, but do not force
                # users to install them.  See INSTALL
+
+               # All Perl installs I know about have these, but RH-based
+               # distros make them separate even though 'perl' pulls them in
+               'File::Path' => 0,
+               'File::Temp' => 0,
+               'Getopt::Long' => 0,
+               'Exporter' => 0,
+               # ExtUtils::MakeMaker # this file won't run w/o it...
        },
        MAN3PODS => \%man3,
 );
@@ -48,26 +66,29 @@ sub MY::postamble {
 # support using eatmydata to speed up tests (apt-get install eatmydata):
 # https://www.flamingspork.com/projects/libeatmydata/
 EATMYDATA =
+N = \$\$(( \$\$(nproc 2>/dev/null || gnproc 2>/dev/null || echo 2) + 1 ))
 -include config.mak
 -include Documentation/include.mk
-N ?= \$(shell echo \$\$(( \$\$(nproc 2>/dev/null || echo 2) + 1)))
 SCRIPTS := scripts/ssoma-replay
-my_syntax := \$(addsuffix .syntax, $PM_FILES \$(EXE_FILES) \$(SCRIPTS))
-
+syn_files = $PM_FILES \$(EXE_FILES) \$(SCRIPTS) \$(wildcard t/*.t)
+my_syntax = \$(addsuffix .syntax, \$(syn_files))
+changed = \$(shell git ls-files -m)
 
 %.syntax ::
-       @\$(PERL) -I lib -c \$(subst .syntax,,\$@)
+       @\$(PERL) -w -I lib -c \$(subst .syntax,,\$@)
 
 syntax:: \$(my_syntax)
 
+dsyn :: \$(addsuffix .syntax, \$(filter \$(changed), \$(syn_files)))
+
 check-manifest :: MANIFEST
-       if git ls-files >\$<.gen 2>&1; then diff -u \$< \$<.gen; fi
+       if git ls-files >\$?.gen 2>&1; then diff -u \$? \$?.gen; fi
 
 check:: pure_all check-manifest
-       \$(EATMYDATA) prove -lv -j\$(N)
+       \$(EATMYDATA) prove -lvw -j\$(N)
 
 lib/PublicInbox/UserContent.pm :: contrib/css/216dark.css
-       @\$(PERL) -I lib \$@ \$<
+       \$(PERL) -I lib \$@ \$?
 
 EOF
 }