ok :) "Philip M. Gollucci" <[EMAIL PROTECTED]> wrote in message news:[EMAIL PROTECTED] > Nikolay Ananiev wrote: > > + " LoadModule perl_module modules/$build->{MODPERL_LIB_DSO}", > > + > > + if($build->{MP_APXS}) { > > + warning "", > > + "depending on your build, mod_perl might not live in", > > + "the modules/ directory - check the results of", "", > > + " \$ $build->{MP_APXS} -q LIBEXECDIR", "", > > + "and adjust the LoadModule directive accordingly.\n"; > > + } > > Thanks Nick, but I think I'd move the if a bit: > Index: Makefile.PL > =================================================================== > --- Makefile.PL (revision 234145) > +++ Makefile.PL (working copy) > @@ -393,9 +393,13 @@ > "You'll need to add the following to httpd.conf:", "", > " LoadModule perl_module modules/$build->{MODPERL_LIB_DSO}", "", > "depending on your build, mod_perl might not live in", > - "the modules/ directory - check the results of", "", > - " \$ $build->{MP_APXS} -q LIBEXECDIR", "", > - "and adjust the LoadModule directive accordingly.\n"; > + "the modules/ directory.\n"; > + if ($build->{MP_APXS}) { > + warning > + "Check the results of", "", > + " \$ $build->{MP_APXS} -q LIBEXECDIR", "", > + "and adjust the LoadModule directive accordingly.\n"; > + } > } > > $build->save; > > > -- > END > ------------------------------------------------------------ > What doesn't kill us can only make us stronger. > Nothing is impossible. > > Philip M. Gollucci ([EMAIL PROTECTED]) 301.254.5198 > Consultant / http://p6m7g8.net/Resume/ > Senior Developer / Liquidity Services, Inc. > http://www.liquidityservicesinc.com > http://www.liquidation.com > http://www.uksurplus.com > http://www.govliquidation.com > http://www.gowholesale.com
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
