Re: [A-T patch] give a hint how to raise the timeout

2004-03-05 Thread David Wheeler
On Mar 4, 2004, at 4:15 PM, Stas Bekman wrote: The patch below attempts to give users an idea to try before they rush to report a bug. For example: waiting 120 seconds for server to start: not ok giving up after 121 secs. If you think that your system is slow or overloaded try again with a

Re: [A-T patch] give a hint how to raise the timeout

2004-03-05 Thread Stas Bekman
David Wheeler wrote: On Mar 4, 2004, at 4:15 PM, Stas Bekman wrote: The patch below attempts to give users an idea to try before they rush to report a bug. For example: waiting 120 seconds for server to start: not ok giving up after 121 secs. If you think that your system is slow or overloaded

Re: [A-T patch] give a hint how to raise the timeout

2004-03-05 Thread David Wheeler
On Mar 4, 2004, at 5:09 PM, Stas Bekman wrote: Not in this case. T-H wasn't called yet. It gets called only after server successfully starts. Ah, right, okay. If you they did: % make test but you'd suggest to run: % ./Build test not only it'll confuse the user, who aren't aware of Build or t/TEST

Re: [A-T patch] give a hint how to raise the timeout

2004-03-05 Thread Stas Bekman
David Wheeler wrote: On Mar 4, 2004, at 5:09 PM, Stas Bekman wrote: Not in this case. T-H wasn't called yet. It gets called only after server successfully starts. Ah, right, okay. If you they did: % make test but you'd suggest to run: % ./Build test not only it'll confuse the user, who aren't

Re: [A-T patch] give a hint how to raise the timeout

2004-03-05 Thread Stas Bekman
Stas Bekman wrote: Well, you could still say Or repeat the last command with the -startup_timeout=420 option. That would work, if the last command was 'make test' or any other wrapper. s/would/won't/ -- __ Stas Bekman

Re: cvs commit: httpd-test/perl-framework/Apache-Test/lib/Apache TestServer.pm

2004-03-05 Thread Stas Bekman
Philippe M. Chiasson wrote: On Thu, 2004-03-04 at 22:21 +, [EMAIL PROTECTED] wrote: geoff 2004/03/04 14:21:27 Modified:perl-framework/Apache-Test Changes perl-framework/Apache-Test/lib/Apache TestServer.pm Log: $ENV{APACHE_TEST_STARTUP_TIMEOUT} now supersedes

Re: cvs commit: httpd-test/perl-framework/Apache-Test/lib/Apache TestServer.pm

2004-03-05 Thread Geoffrey Young
$ENV{APACHE_TEST_STARTUP_TIMEOUT} now supersedes -startup_timeout [...] What's the reason for this change? it was a follow up from something on modperl@ I almost always expect that command line arguments supercede env variables.. i.e. $ CVSROOT=foo cvs -d bar co foo Bad CVSROOT:

Re: cvs commit: httpd-test/perl-framework/Apache-Test/lib/Apache TestConfigC.pm

2004-03-05 Thread Geoffrey Young
+my $dversion = $self-server-dversion; +if ($dversion eq '-DAPACHE1') { that's '-D APACHE1' in current cvs, no? --Geoff

Re: cvs commit: httpd-test/perl-framework/Apache-Test/lib/Apache TestConfigC.pm

2004-03-05 Thread Rodent of Unusual Size
Geoffrey Young wrote: +my $dversion = $self-server-dversion; +if ($dversion eq '-DAPACHE1') { that's '-D APACHE1' in current cvs, no? so i saw, when my tests blew up. -- #kenP-)} Ken Coar, Sanagendamgagwedweinini http://Ken.Coar.Org/ Author, developer, opinionist