[PATCH 0/2] minor build/test tweaks

[PATCH 0/2] minor build/test tweaks

[Eric Wong]

Eric Wong (2):
      Makefile.PL: allow `N' variable to be set in local config.mak
      Makefile.PL: enable prove warnings

 Makefile.PL | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH 1/2] Makefile.PL: allow `N' variable to be set in local config.mak

[Eric Wong]

This can useful for limiting test resource use without relying
on remembering the variable command-line.
---
 Makefile.PL | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile.PL b/Makefile.PL
index de0c49f..f79697b 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -53,9 +53,9 @@ 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 = \$\$(( \$\$(nproc 2>/dev/null || gnproc 2>/dev/null || echo 2) + 1 ))
 SCRIPTS := scripts/ssoma-replay
 my_syntax := \$(addsuffix .syntax, $PM_FILES \$(EXE_FILES) \$(SCRIPTS))
 
-- 
EW

[PATCH 2/2] Makefile.PL: enable prove warnings

[Eric Wong]

We already "use warnings" everywhere, but could miss some spots.
This ought to cover that, and usually Perl module authors are
consistent about avoiding warnings that we won't clutter our
test outputs.
---
 Makefile.PL | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile.PL b/Makefile.PL
index f79697b..b1274ad 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -69,7 +69,7 @@ check-manifest :: MANIFEST
 	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 \$@ \$?
-- 
EW