This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Don't call CPAN to install any "extra" modules for the install-notify target.
authorNicholas Clark <nick@ccl4.org>
Wed, 25 Apr 2012 20:00:09 +0000 (22:00 +0200)
committerNicholas Clark <nick@ccl4.org>
Sat, 13 Jul 2013 10:32:14 +0000 (12:32 +0200)
Previously the Makefile's install-notify target built extra.install just like
the other 5 install targets. However, the other 5 actually install perl,
whereas install-notify (a.k.a. no-install) runs installperl and installman
with the --notify option (formerly -n), to cause them to print what they would
do but take no action. Hence if -Dextras was used, make no-install would
actually install the extra modules, but nothing else. This is inconsistent.

Makefile.SH

index da00514..ef30479 100755 (executable)
@@ -1020,9 +1020,10 @@ for name in all notify silent strip verbose; do
 
 install_$name install-$name: \$(INSTALL_DEPENDENCE) installperl all installman
        \$(RUN_PERL) installperl --destdir=\$(DESTDIR) ${flags%--all} \$(INSTALLFLAGS) \$(STRIPFLAGS)
-       -@test ! -s extras.lst || PATH="`pwd`:\${PATH}" PERL5LIB="`pwd`/lib" \$(RUN_PERL) -Ilib -MCPAN -e '@ARGV&&install(@ARGV)' \`cat extras.lst\`
        \$(RUN_PERL) installman --destdir=\$(DESTDIR) ${flags%--all}
 EOT
+
+    test $name = notify || echo '      -@test ! -s extras.lst || PATH="`pwd`:${PATH}" PERL5LIB="`pwd`/lib" $(RUN_PERL) -Ilib -MCPAN -e '\''@ARGV&&install(@ARGV)'\'' `cat extras.lst`' >>$Makefile
 done
 
 $spitshell >>$Makefile <<'!NO!SUBS!'