From eecdfc84a7e605b42adf68c848ed2e16e08e963c Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 27 Oct 2020 07:54:40 +0000 Subject: Makefile.PL: do not build manpage if POD is missing But warn on it, this lets us test new or throwaway commands more easily if we don't have to start a new POD for everything we want to dump in script/. --- Makefile.PL | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) (limited to 'Makefile.PL') diff --git a/Makefile.PL b/Makefile.PL index f6b7abb6..56679598 100644 --- a/Makefile.PL +++ b/Makefile.PL @@ -31,7 +31,18 @@ my @syn = (@EXE_FILES, grep(m!^lib/.*\.pm$!, @manifest), @scripts); @syn = grep(!/SaPlugin/, @syn) if !eval { require Mail::SpamAssasin }; $v->{syn_files} = \@syn; $v->{my_syntax} = [map { "$_.syntax" } @syn]; -$v->{-m1} = [ map { (split('/'))[-1] } @EXE_FILES ]; +my @no_pod; +$v->{-m1} = [ map { + my $x = (split('/'))[-1]; + my $pod = "Documentation/$x.pod"; + if (-f $pod) { + $x; + } else { + warn "W: $pod missing\n"; + push @no_pod, $x; + (); + } + } @EXE_FILES ]; $v->{-m5} = [ qw(public-inbox-config public-inbox-v1-format public-inbox-v2-format) ]; $v->{-m7} = [ qw(public-inbox-overview public-inbox-tuning) ]; @@ -109,6 +120,7 @@ my %man3 = map {; # semi-colon tells Perl this is a BLOCK (and not EXPR) $mod =~ s/\.\w+\z//; "lib/PublicInbox/$_" => "blib/man3/PublicInbox::$mod.3" } qw(Git.pm Import.pm WWW.pod SaPlugin/ListMirror.pod); +my $warn_no_pod = @no_pod ? "\n\t\@echo W: missing .pod: @no_pod\n" : ''; WriteMakefile( NAME => 'PublicInbox', # n.b. camel-case is not our choice @@ -172,6 +184,8 @@ $VARS -include Documentation/include.mk $TGTS +check-man ::$warn_no_pod + # syntax checks are currently GNU make only: %.syntax :: % @\$(PERL) -w -I lib -c \$< -- cgit v1.2.3-24-ge0c7