Geoffrey Young wrote:
   my $timeout = $vars->{startup_timeout} ||
                 $ENV{APACHE_TEST_STARTUP_TIMEOUT} ||
                 60;


I swapped the order in CVS. thanks.

No, Geoff, the order was correct. Command line options override env vars.


The correct patch seems to be:

Index: lib/ModPerl/TestRun.pm
===================================================================
RCS file: /home/cvs/modperl-2.0/lib/ModPerl/TestRun.pm,v
retrieving revision 1.19
diff -u -r1.19 TestRun.pm
--- lib/ModPerl/TestRun.pm      4 Mar 2004 06:01:06 -0000       1.19
+++ lib/ModPerl/TestRun.pm      4 Mar 2004 22:27:08 -0000
@@ -32,6 +32,7 @@
     # default timeout in secs (threaded mpms are extremely slow to
     # startup, due to a slow perl_clone operation)
     $self->{conf_opts}->{startup_timeout} ||=
+        $ENV{APACHE_TEST_STARTUP_TIMEOUT} ||
         Apache::Build->build_config->mpm_is_threaded() ? 300 : 120;

$self->{conf_opts}->{maxclients} ||= MIN_MAXCLIENTS;


__________________________________________________________________ Stas Bekman JAm_pH ------> Just Another mod_perl Hacker http://stason.org/ mod_perl Guide ---> http://perl.apache.org mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com http://modperlbook.org http://apache.org http://ticketmaster.com

--
Report problems: http://perl.apache.org/bugs/
Mail list info: http://perl.apache.org/maillist/modperl.html
List etiquette: http://perl.apache.org/maillist/email-etiquette.html



Reply via email to