Re: t/SMOKE points the finger at filter/out_bbs_filebucket.t

2004-09-22 Thread Joe Schaefer
Stas Bekman <[EMAIL PROTECTED]> writes: > The error comes from > return $filter->next->pass_brigade($bb2); > > in sub handler in TestFilter::out_bbs_filebucket; > > So it's some Apache filter that throws an error, It's wierder than that- the return value is tainted somehow: my $stat

Re: [mp2 api] log_reason

2004-09-22 Thread Jie Gao
Yeah, that was a very handy func. -Jie On Wed, 22 Sep 2004, Stas Bekman wrote: > Date: Wed, 22 Sep 2004 22:13:27 -0400 > From: Stas Bekman <[EMAIL PROTECTED]> > To: mod_perl Dev <[EMAIL PROTECTED]> > Subject: [mp2 api] log_reason > > Over irc we have decided to dump log_reason. But now that I've

[mp2 api] log_reason

2004-09-22 Thread Stas Bekman
Over irc we have decided to dump log_reason. But now that I've looked at it, I'm not sure whether we should do that. If you call: $r->log_reason("What's the reason?"); you get: [Wed Sep 22 22:06:23 2004] [error] access to /My__Bug failed for 127.0.0.1, reason: What's the reason? and there is

Re: t/SMOKE points the finger at filter/out_bbs_filebucket.t

2004-09-22 Thread Stas Bekman
The error comes from return $filter->next->pass_brigade($bb2); in sub handler in TestFilter::out_bbs_filebucket; So it's some Apache filter that throws an error, but as Apache filter provide no error handling, we don't know who does that and what's the error. One needs to step through with th

Re: t/SMOKE points the finger at filter/out_bbs_filebucket.t

2004-09-22 Thread Stas Bekman
Joe Schaefer wrote: Stas Bekman <[EMAIL PROTECTED]> writes: Stas Bekman wrote: t/SMOKE is hard at work and found the following sequence: t/TEST -v t/apr/string.t t/protocol/echo_bbs2.t t/filter/out_bbs_filebucket.t with only one server, the sequence is reduced to just two tests: t/TEST -v -maxclie

Re: t/SMOKE points the finger at filter/out_bbs_filebucket.t

2004-09-22 Thread Joe Schaefer
Stas Bekman <[EMAIL PROTECTED]> writes: > Stas Bekman wrote: > > t/SMOKE is hard at work and found the following sequence: > > t/TEST -v t/apr/string.t t/protocol/echo_bbs2.t t/filter/out_bbs_filebucket.t > > with only one server, the sequence is reduced to just two tests: > t/TEST -v -maxclients

Re: t/SMOKE points the finger at filter/out_bbs_filebucket.t

2004-09-22 Thread Stas Bekman
Stas Bekman wrote: t/SMOKE is hard at work and found the following sequence: t/TEST -v t/apr/string.t t/protocol/echo_bbs2.t t/filter/out_bbs_filebucket.t with only one server, the sequence is reduced to just two tests: t/TEST -v -maxclients 1 t/protocol/echo_bbs2.t t/filter/out_bbs_filebucket.t -

mp2 static segfault handling

2004-09-22 Thread Stas Bekman
Philippe, Under static modperl, when we get a segfault, it suggests to run: gdb /home/stas/apache.org/httpd-2.0/httpd -core /home/stas/apache.org/mp2-static/t/core.28076 which will fail, since httpd is a script. Instead it should say: gdb /home/stas/apache.org/httpd-2.0/.libs/lt-httpd -core /hom

t/SMOKE points the finger at filter/out_bbs_filebucket.t

2004-09-22 Thread Stas Bekman
t/SMOKE is hard at work and found the following sequence: t/TEST -v t/apr/string.t t/protocol/echo_bbs2.t t/filter/out_bbs_filebucket.t ... t/filter/out_bbs_filebucket1..10 # Running under perl version 5.008005 for linux # Current time local: Wed Sep 22 07:33:33 2004 # Current time GMT: Wed S

mp2: static build shutdown segfault

2004-09-22 Thread Stas Bekman
Got a core when running the test suite on the static server (probably at the server shutdown). couldn't reproduce it with (t/SMOKE -bug). #0 0x4043d67f in find_entry (ht=0x6c2f742f, key=0x407538e6, klen=14, val=0x0) at apr_hash.c:235 235 for (hep = &ht->array[hash & ht->max], he = *h

Re: [Patch mp2] APR::OS::thread_current => APR::OS::current_thread_id

2004-09-22 Thread Philippe M. Chiasson
thanks and done. Stas Bekman wrote: Philippe M. Chiasson wrote: One more item off todo/release for today. This patch changes my $tid_obj = APR::OS::thread_current(); my $tid = $$tid_obj; to my $tid = APR::OS::current_thread_id(); +1 with 2 fixes below: and don't forget to fix the manpage. Thanks.

Re: [Patch mp2] APR::OS::thread_current => APR::OS::current_thread_id

2004-09-22 Thread Stas Bekman
Philippe M. Chiasson wrote: One more item off todo/release for today. This patch changes my $tid_obj = APR::OS::thread_current(); my $tid = $$tid_obj; to my $tid = APR::OS::current_thread_id(); +1 with 2 fixes below: and don't forget to fix the manpage. Thanks. Index: Changes ==

[Patch mp2] APR::OS::thread_current => APR::OS::current_thread_id

2004-09-22 Thread Philippe M. Chiasson
One more item off todo/release for today. This patch changes my $tid_obj = APR::OS::thread_current(); my $tid = $$tid_obj; to my $tid = APR::OS::current_thread_id(); -- Philippe M. Chiasson m/gozer\@(apache|cpan|ectopl