[patch]avoid closing fd 0/1

2010-03-30 Thread Torsten Förtsch
eview! One thing that I don't understand is the difference between IoIFP and IoOFP. Why does perl need 2 such structures to hold 1 file handle? Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net Index: src/modules/perl/

Re: [patch]avoid closing fd 0/1

2010-03-30 Thread Torsten Förtsch
mode, 0, NULL) == 0) { err = get_sv("!", TRUE); } do_close(handle, FALSE); (void)hv_delete(gv_stashpv("Apache2::RequestIO", TRUE), GvNAME(handle), GvNAMELEN(handle), G_DISCARD); if (err != Nullsv) { Perl

Re: [patch]avoid closing fd 0/1

2010-03-31 Thread Torsten Förtsch
On Tuesday 30 March 2010 19:10:36 Torsten Förtsch wrote: > Previously the code did something similar to > > open SAVEFH, '<&STDIN'; > close STDIN; > ... > open STDIN, '<&SAVEFH'; > > The idea is to change that int

Re: [patch]avoid closing fd 0/1

2010-03-31 Thread Torsten Förtsch
On Wednesday 31 March 2010 16:43:04 Torsten Förtsch wrote: > I believe the IoFLUSH_off in the override function can be omitted since > this is standard for a new handle. > This appears to be true. This is now the final version of the patch. Are there any objections against applying it

simplify mod_perl.c a bit

2010-03-31 Thread Torsten Förtsch
modperl_response_handler_run(r, TRUE) do something modperl_response_finish() it now looks like modperl_response_handler_run(r) { do something } 1st usage: modperl_response_handler_run(r) modperl_response_finish() 2nd usage: modperl_response_handler_run(r) do something modperl_response_finish() Torsten

Re: [patch]avoid closing fd 0/1

2010-04-01 Thread Torsten Förtsch
destio=GvIO(handle_orig); -do_close(handle, FALSE); + tmp=SvANY(destio); + SvANY(destio)=SvANY(srcio); + SvANY(srcio)=tmp; + (void)hv_delete(gv_stashpv("Apache2::RequestIO", TRUE), GvNAME(handle), GvNAMELEN(handle), G_DIS

modperl_filter_t question

2010-04-02 Thread Torsten Förtsch
not possible that both fields are used at the same time since the result of modperl_filter_new is either an output or an input filter but not both. So, why are there 2 fields? Am I missing something? Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like

Re: simplify mod_perl.c a bit

2010-04-04 Thread Torsten Förtsch
hat I committed your version. K&R C didn't. But K&R C didn't allow for function parameters to be declared in parentheses. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net --

Re: [patch]avoid closing fd 0/1

2010-04-04 Thread Torsten Förtsch
t;', $INC{'TestModperl/stdfd2.pm'} + or die "Cannot open $INC{'TestModperl/stdfd2.pm'}"; +scalar readline STDIN for(1..2); + +my $expected=$.; + +$r->lookup_uri($r->uri)->run; + +$r->print("1..1\n"); +$r->print(($.

Re: [patch]avoid closing fd 0/1

2010-04-04 Thread Torsten Förtsch
On Wednesday 31 March 2010 16:43:04 Torsten Förtsch wrote: > This one is much simpler. > This is another idea to solve the problem. We used to save and restore the file handle. Why not simply localize it? The caller of the override/restore functions already creates the opening and c

what appeared to be a small typo

2010-04-06 Thread Torsten Förtsch
The patch also eliminates the useless apr_strdup in name = apr_pstrcat(p, HvNAME(GvSTASH(gv)), "::", GvNAME(gv), NULL); return modperl_handler_new(p, apr_pstrdup(p, name)); Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fa

Re: what appeared to be a small typo

2010-04-06 Thread Torsten Förtsch
On Tuesday 06 April 2010 12:51:11 Torsten Förtsch wrote: > Any objections against this patch? > I have separated the previous patch into 2 1) to address the problem. It calls now modperl_mgv_resolve instead of modperl_code_attrs. Index: src/modules/perl/modperl_fi

Re: [patch]avoid closing fd 0/1

2010-04-07 Thread Torsten Förtsch
On Sunday 04 April 2010 18:43:29 Torsten Förtsch wrote: > This is another idea to solve the problem. We used to save and restore the > file handle. Why not simply localize it? > [...] > Opinions? > > How do I build a modperl that uses tied IO? > This version compiles c

adding t_{start,read,finish}_file_watch to Apache::TestUtil

2010-04-08 Thread Torsten Förtsch
meter. If omitted +or undefined the C is opened. Relative file name are +evaluated relative to the directory containing C. + +If the specified file does not exist (yet) no error is returned. It is +assumed that it will appear soon. In this case C +will open the file silently and read from the beginn

Re: [mp2] make test fails with a debugging perl

2010-04-13 Thread Torsten Förtsch
.apache.org/repos/asf/perl/modperl/branches/threading \ mod_perl-2.0 I'd recommend the latter. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To u

Re: [mp2] make test fails with a debugging perl

2010-04-13 Thread Torsten Förtsch
On Tuesday 13 April 2010 12:00:05 Torsten Förtsch wrote: > svn checkout \ > https://svn.apache.org/repos/asf/perl/modperl/branches/threading \ > mod_perl-2.0 > Oops, don't do that. It one requires a threaded perl. Yours is not. useithreads=undef, usemultiplicity=undef

Re: [mp2] make test fails with a debugging perl

2010-04-13 Thread Torsten Förtsch
rst thing. SvGROW then allocates memory and stores it in SvPVX. The patch is really only needed to make -DDEBUGGING happy. You really might want to try trunk. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy

Re: RT 32992

2010-04-16 Thread Torsten Förtsch
-le 'my $v1="6.06"; my $v2="6.06_01"; print "less" if 0+$v1<0+$v2' $ perl -Mversion -le 'my $v1="6.06"; my $v2="6.06_01"; print "less" if version->new($v1)new($v2)' less There is a difference. Without version it is

Re: svn commit: r935519 - in /perl/modperl/trunk: t/response/TestAPR/socket.pm xs/APR/Socket/APR__Socket.h xs/maps/apr_functions.map xs/tables/current/ModPerl/FunctionTable.pm

2010-04-20 Thread Torsten Förtsch
apr_os_sock_t; typedef int apr_os_sock_t;/**< native dir */ The first result is windows. So, as long as no other systems are added it should work. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http:/

Re: Another A:T patch for review

2010-04-21 Thread Torsten Förtsch
; > > + > +sub is_root { > +my $user = getpwuid($>) || ''; > + > +return unless $user eq 'root'; why not C<< $>==0 >>? This is what apache checks, not the name. Torsten Förtsch -- Need professional modperl support? Hire me! (htt

[RFC] How to implement stuff that makes sense only under certain conditions

2010-05-08 Thread Torsten Förtsch
;> * or leave it conditionally unimplemented like this: Index: xs/maps/apr_functions.map === ... +#_if_ $^O ne 'linux' mpxs_APR__Socket_fileno | | apr_socket_t *:socket +#_end_ ... +#_if_ $^O ne 'linux' ~apr_os_sock_get +#_else_

Re: perl5.10.1 and mod_perl 2.0.4 break on "my_perl"

2010-08-12 Thread Torsten Förtsch
On Thursday, August 12, 2010 20:31:42 Randal L. Schwartz wrote: > useithreads=undef, usemultiplicity=define This is your problem. Either both or none should be defined but not only one of them. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fant

Trunk: A::R and A::SL test failures

2010-08-13 Thread Torsten Förtsch
::SL is failing because it expects a httpd.conf with an activated mod_perl to inherit from. These are suspicions only. Perhaps they ring a bell or two. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net --

Re: Trunk: A::R and A::SL test failures

2010-08-14 Thread Torsten Förtsch
On Saturday, August 14, 2010 05:23:09 Fred Moyer wrote: > 2010/8/13 Torsten Förtsch : > > I am trying to compile/test trunk with a freshly compiled perl and > > apache. There hasn't been installed a modperl nor A::T/A::R/A::SL for > > this perl. > > Hmm, nice cat

Found in Apache-Reload/Makefile.PL

2010-08-15 Thread Torsten Förtsch
, $ENV{MP_APXS}; or push @ARGV, "-apxs=$ENV{MP_APXS}"; Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-u

Re: Trunk: A::R and A::SL test failures

2010-08-15 Thread Torsten Förtsch
On Friday, August 13, 2010 20:55:50 Torsten Förtsch wrote: > I believe A::R is failing because it uses the standard @INC but A::T is > not yet installed there. > > A::SL is failing because it expects a httpd.conf with an activated mod_perl > to inherit from. Just committed

Re: [VOTE] Apache-Reload-0.11-rc2

2010-08-16 Thread Torsten Förtsch
bitint=define, use64bitall=define, uselongdouble=undef usemymalloc=n, bincompat5005=undef All tests successful. +1 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net -

Failed to find a config file to save the custom configuration in

2010-08-19 Thread Torsten Förtsch
APACHE_TEST_TRACE_LEVEL PerlPassEnv HARNESS_PERL_SWITCHES) ]); } = Shouldn't APACHE_TEST_NO_STICKY_PREFERENCES be added to this list? Torsten Förtsch -- Need professional modperl support? Hi

Re: Fwd: svn commit: r987890 - in /perl/Apache-Test/trunk: Changes lib/Apache/TestConfigPerl.pm

2010-08-22 Thread Torsten Förtsch
t through unnoticed and in the end the package contains a config that does not match the system. So, had I noticed I'd given you a -1 anyway. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.na

Re: Fwd: svn commit: r987890 - in /perl/Apache-Test/trunk: Changes lib/Apache/TestConfigPerl.pm

2010-08-22 Thread Torsten Förtsch
On Sunday, August 22, 2010 21:46:58 Fred Moyer wrote: > 2010/8/22 Torsten Förtsch : > > On Sunday, August 22, 2010 20:02:05 Fred Moyer wrote: > >> I put out a release candidate for Apache-Test 1.33 yesterday. Did you > >> see the email come through? An archive

Re: Fwd: svn commit: r987890 - in /perl/Apache-Test/trunk: Changes lib/Apache/TestConfigPerl.pm

2010-08-22 Thread Torsten Förtsch
Apache::TestRequest::hostport; t_debug "Using HTTP @ $hostport"; Now, it happens that when the server is started one set of ports is allocated when the test runs t/conf/apache_test_config.pm is overwritten with another set of ports. Don't know if it's my fault o

2 A-T patches and a bug without patch

2010-08-23 Thread Torsten Förtsch
his happens in the current A-T test suite. A possible fix could be to check defined(&Test::plan) and defined(&Test::More::plan) and reinitialize both if both are loaded. The same should be done for plan() ($real_plan). Thoughts? Torsten Förtsch -- Need professional modperl support?

Checking for "usemultiplicity xor usethreads"

2010-08-31 Thread Torsten Förtsch
ile Perl with either -Duseithreads and ". + "-Dusemultiplicity or -Uuseithreads and -Uusemultiplicity"; + } +exit 1; +} } sub system_sanity_check { Is that worth committing? Torsten Förtsch -- Need professional modperl support? Hire me! (http

Re: [RELEASE CANDIDATE] Apache-Test-1.33 RC1

2010-09-11 Thread Torsten Förtsch
:55 Server's Module Magic Number: 20051115:24 Server loaded: APR 1.4.2, APR-Util 1.3.9 Compiled using: APR 1.4.2, APR-Util 1.3.9 Architecture: 64-bit Server MPM: Prefork Thanks, Fred. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fanta

Re: [RELEASE CANDIDATE] Apache-SizeLimit 0.92 RC2

2010-09-20 Thread Torsten Förtsch
On Monday, September 20, 2010 19:44:54 Fred Moyer wrote: > http://people.apache.org/~phred/Apache-SizeLimit-0.92-rc2.tar.gz > > Please take this for a test and report back. +1, linux x86_64, perl 5.12.1, Apache/2.2.16 (prefork MPM) Torsten Förtsch -- Need professional modperl support

Re: [RELEASE CANDIDATE] Apache-SizeLimit 0.93 RC1

2010-09-29 Thread Torsten Förtsch
2::SizeLimit Apache2::SizeLimit->set_max_process_size(150_000); Apache2::SizeLimit->set_min_shared_size(10_000); Apache2::SizeLimit->set_max_unshared_size(120_000); PerlModule is executed later. Torsten Förtsch -- Need professional modperl support? Hire me! (http://fo

Re: [patch] Apache::Test - taking the pain away

2010-09-30 Thread Torsten Förtsch
even on us...@httpd if there is anyone out there using them consciously. I'll have a closer look on the weekend. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net

Re: [patch] Apache::Test - taking the pain away

2010-10-04 Thread Torsten Förtsch
sting : called Apache::Test::ok() # expected: 1 # received: 1 ok 2 All 1 subtests passed t/more/04testmore.t .. 1..2 # Using Apache/Test.pm version 1.34 ok 1 - called Test::More::is() ok 2 - called Test::More::like() ok Torsten Förtsch -- Need professional modperl support? Hire me! (http://foe

Re: Fwd: svn commit: r1023553 - in /perl/modperl/trunk: Changes Makefile.PL lib/Apache2/Build.pm lib/ModPerl/BuildOptions.pm

2010-10-19 Thread Torsten Förtsch
smatch: sub Apache::Test::ok ($;$$) vs ($;$) at /home/r2/work/mp2/trunk/Apache- Test/t/../lib/Apache/Test.pm line 88 Prototype mismatch: sub Apache::Test::skip ($;$$$) vs none at /home/r2/work/mp2/trunk/Apache- Test/t/../lib/Apache/Test.pm line 88 Torsten Förtsch -- Need professional modperl su

Re: [vote] Release cycle

2010-10-26 Thread Torsten Förtsch
On Tuesday, October 26, 2010 06:40:48 Fred Moyer wrote: > mod_perl committers, please vote +1 or -1 to proceed with this release > cycle. +1 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatin

Re: [vote] Release cycle

2010-10-26 Thread Torsten Förtsch
failures that I saw > before? > > The other possibility was that it was a failure with perl 5.12.1 vs > 5.12.2. I am using 5.12.1 (w/o threads) here and am not able to reproduce the problem. Torsten Förtsch -- Need professional modperl support? Hire me! (h

You can call me code elf (was: [RELEASE CANDIDATE] Apache-Test-1.34 RC1)

2010-12-27 Thread Torsten Förtsch
t mean much either because it is quite limited. So, please test it and provide test cases if possible. A bit late but, Merry Christmas and a Happy New Year, Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatin

Re: You can call me code elf (was: [RELEASE CANDIDATE] Apache-Test-1.34 RC1)

2010-12-30 Thread Torsten Förtsch
led 1/47 subtests That's the old perl version. Could you please retest revision 1053949 (or later). The following command shows the bug: $ perl -Mstrict -e 'my $re=qr/^a$/m; printf "%vd: %s\n", $^V, "a\nx"=~$re ? "match" : "no match"' 5.1

Re: Fwd: [rt.cpan.org #64999] GvCV and GvGP lvalue changes in perl core

2011-01-21 Thread Torsten Förtsch
vCV_set(gv, cv) (GvCV(gv)=(cv)) # define GvGP_set(gv, gp) (GvGP(gv)=(gp)) Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-uns

Re: [RELEASE CANDIDATE] Apache-Test-1.35 RC1

2011-01-22 Thread Torsten Förtsch
g] server localhost.localdomain:8529 shutdown looks good, clean error_log +1 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev

Re: svn commit: r1062443 - in /perl/modperl/trunk/src/modules/perl: modperl_const.c modperl_perl.c modperl_perl.h

2011-01-23 Thread Torsten Förtsch
l get a few "Attempt to free unreferenced scalar: SV 0x1d75960, Perl interpreter: 0x28f89b0 during global destruction." messages in the error_log. But otherwise all tests succeed. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.nam

modperl - quo vadis?

2011-01-28 Thread Torsten Förtsch
s and documentation even longer. Last year I couldn't afford that. I doubt this year. It's quite a large chunk of my yearly income. Thoughts? Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net ---

Re: modperl - quo vadis?

2011-01-31 Thread Torsten Förtsch
2005 (VC8), VS2003 (VC7) and VC6 used to work > too and probably still do (but some of the earlier free Express versions > have various dependencies on SDKs etc: the VS2008 or VS2010 versions are > better because they are self-contained). Thank you very much! I'll try that out next wee

Re: [RELEASE CANDIDATE] Apache-Test-1.36 RC1

2011-02-02 Thread Torsten Förtsch
On Tuesday, February 01, 2011 07:54:43 Fred Moyer wrote: > http://people.apache.org/~phred/Apache-Test-1.36-rc1.tar.gz +1 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net === Special Tests Sequence Failure Fin

Re: [RELEASE CANDIDATE]: mod_perl-2.0.5 RC1

2011-02-02 Thread Torsten Förtsch
ded() returns false without any limits set I have corrected the problem in commit 1066417. See svn log -c 1066417 for more information. Not sure if localizing the limits is the right solution. Perhaps better to make sure the other tests do not change global settings. What do you think? Tors

Re: modperl - quo vadis?

2011-02-02 Thread Torsten Förtsch
most are ok. But porting/buildtoc.t fails and now the test suite is stalled in cpan/Archive-Extract/t/01_Archive-Extract.t ... 342/? So, again I am failing on the preliminaries, :-( With perl 5.12.3 it stalled a bit earlier. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foer

Re: modperl - quo vadis?

2011-02-04 Thread Torsten Förtsch
l: drop the tied-IO code and refactor/simplify the perlio stuff. > > Yes! > > >>> - implement a way to untie %ENV after fork() in a handler > > Thats a rough one b/c of the way its coded. see also local %ENV bombing. I have feared someone would say that. I had a look

Re: [RELEASE CANDIDATE]: mod_perl-2.0.5 RC2

2011-02-06 Thread Torsten Förtsch
Built under linux Compiled at Jan 22 2011 17:46:40 %ENV: PERL5LIB="/opt/mod_perl" PERLDOC="-oman" PERLDOC_PAGER="more" @INC: /opt/mod_perl /opt/perl/lib/site_perl/5.12.3/x86_64-linux /opt/perl/lib/site_perl/5.12.3 /opt/perl/lib/vendor_p

Re: modperl - quo vadis?

2011-02-13 Thread Torsten Förtsch
test. But then nmake test gives you a lot of errors of the type "program C:\Dokumente not found". So, there are 2 points that I have found: - don't use vista, haven't tried win7 - don't use directory names with blanks Torsten Förtsch -- Need professional modperl suppor

Re: mod_perl 2.0.6

2011-02-17 Thread Torsten Förtsch
g on Windows needs to be done. I have managed to compile and test perl on a XP VM. That's a 1st step. But I won't mind if someone else would jump in. svn co https://svn.eu.apache.org/repos/asf/perl/modperl/branches/threading Torsten Förtsch -- Need professiona

Re: Retiring mod_perl 1.3.x

2011-02-19 Thread Torsten Förtsch
gt; I'll leave 5 days for comments. +1 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org

PERL_SET_CONTEXT

2011-03-03 Thread Torsten Förtsch
modperl_interp_t *interp = \ modperl_interp_select(r, r->connection, r->server); \ dTHXa(interp->perl);\ PERL_SET_CONTEXT(aTHX) #else # define MP_dTHX dNOOP #endif Thoughts? Torsten Förtsch -

Re: PERL_SET_CONTEXT

2011-03-03 Thread Torsten Förtsch
On Thursday, March 03, 2011 20:29:43 Torsten Förtsch wrote: > Thoughts? Better to think first, modperl_interp_select does that already, sorry for the noise. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.

pnotes and pool cleanups

2011-03-04 Thread Torsten Förtsch
d we support them separately as APR::Pool->pre_cleanup_register or should we simply move our pool cleanups to the pre_cleanup stage? I think the former is more appropriate. In the mean time use $r->notes in such cases or perhaps $r->connection->pnotes. Torsten Förtsch -- Ne

Is there anything wrong with the mail archives?

2011-03-05 Thread Torsten Förtsch
Hi, normally mails sent to the list appear in the archives quite on time. At the moment, however, neither gossamer-threads nor mail-archives.apache.org show anything more recent than Mar 3. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy

Re: Fwd: [rt.cpan.org #66894] Apache::SizeLimit: Linux::Smaps detection broken

2011-03-25 Thread Torsten Förtsch
On Friday, March 25, 2011 17:58:26 Fred Moyer wrote: > -if (eval { require Linux::Smaps } && Linux::Smaps->new($$)) { > +if (eval { require Linux::Smaps && Linux::Smaps->new($$) }) { +1 Torsten Förtsch -- Need professional modperl support? Hire me!

Anyone working on preparing modperl for httpd 2.4?

2011-04-06 Thread Torsten Förtsch
laced by server_rec->log.level Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org For additional comma

Apache2::RequestUtil weirdness

2011-04-08 Thread Torsten Förtsch
and not be hardcoded, even as default values. I'll remove the lines: if (!entry->ap_auth_type) { entry->ap_auth_type = "Basic"; } if (!entry->ap_auth_name) { entry->ap_auth_name = apr_pstrdup(p, location);

modperl_module_insert

2011-04-17 Thread Torsten Förtsch
same as modp->next = perl_module.next; perl_module.next = modp; without the need to search for perl_module? Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net -

Re: Apache2::RequestUtil weirdness

2011-04-19 Thread Torsten Förtsch
I am not sure yet how to support that with httpd 2.4. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apac

Re: Apache::SizeLimit new RT issue

2011-04-19 Thread Torsten Förtsch
On Tuesday, April 19, 2011 09:47:23 Fred Moyer wrote: > Torsten have you seen this one? > > https://rt.cpan.org/Public/Bug/Display.html?id=66894 Yes, and at a first glance it looks very reasonable. Please test and commit. Torsten Förtsch -- Need professional modperl support? Hire

mp2 & perl 5.14

2011-05-05 Thread Torsten Förtsch
, argv=0x7fff5a3f6fc8) at main.c:679 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org For

perl 5.14 & modperl - please test

2011-05-20 Thread Torsten Förtsch
Hi, users of a threaded modperl (esp. Windows), please test modperl trunk with perl 5.14! On my linux box it works now (rev 1125476). Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net

Apache::Test & LWP 6.x - don't

2011-05-20 Thread Torsten Förtsch
s for the body and LWP waits for god knows what. I got back to LWP 5. Now all the tests work again. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net ---

Re: Apache::Test & LWP 6.x - don't

2011-06-04 Thread Torsten Förtsch
mpiles. A few segfaults have also been fixed. But it still crashes at startup in modperl_pcw.c. Tackle one problem at a time, that's also the reason why I went back to LWP 5. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.na

Re: mod_perl 2.0.4: $r->uri

2011-06-06 Thread Torsten Förtsch
s sense I think the answer is yes. Anyway, this is rather a httpd API than a modperl question. Modperl only provides access to the "uri" field of the "struct request_rec" here. So, for more information ask here: d...@httpd.apache.org us...@httpd.apache.org Torsten Förtsc

Re: mod_perl 2.0.4: $r->uri

2011-06-06 Thread Torsten Förtsch
sHandler "sub { \ my ($r)=@_; \ $r->uri(parse_uri $r->unparsed_uri); \ -1; \ }" parse_uri() is to be defined by you. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foert

Re: Anything left before 2.0.6?

2011-06-27 Thread Torsten Förtsch
more attention (hopefully). Anyway, I think it's time. Thanks Fred! Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-un

First test run with Apache 2.3 (trunk) without core dump

2011-07-25 Thread Torsten Förtsch
tests fail. I had to kill the apache worker that ran t/filter/both_str_req_mix.t. The aaa tests were expected to fail as well as logging. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net

Re: [vote] Submit a motion to the ASF board to exempt Apache::* from the official release process

2011-07-25 Thread Torsten Förtsch
s to > do a major mod_perl release can reach 12, which is 4x as much as some > other ASF projects. Apache::Test continues to have participation from > httpd core members, and as such is consistently able to get enough > votes for releases, and does not need this exemption.' +1

Re: Embperl & STDOUT/IN in mod_perl 2.0.5

2011-08-05 Thread Torsten Förtsch
dy tell me how I get the filehandle that mod_perl > has setup in my XS code? The commit in question is 932875. The functions to do the above are C and C in F. They use C to get the handle. Note, all this is only done if the handler is C. For C handlers STD handles are not set up. Torsten Förts

Re: Embperl & STDOUT/IN in mod_perl 2.0.5

2011-08-05 Thread Torsten Förtsch
think I am an expert on this. I think what I have said is correct according to the code. Unfortunately this piece of perl is quite undocumented. If in doubt ask on p5p. For example perlapi several times explains other functions in terms of gv_fetchpv but never explains gv_fetchpv itself. Tor

Re: 2.0.6?

2011-11-22 Thread Torsten Förtsch
;s not necessary. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org For additional comman

Re: Fwd: 2012 updates

2011-11-23 Thread Torsten Förtsch
cated and >apreq will be 'closed down' I'd add 7) Have a look at the futex problem recently reported by Max Barry. 8) sort out the large file support reported by Marco Walther in the thread "[mp 2.0.5] Early core dump:-( OpenSolaris-x86, Apache 2.2.21 & Perl 5

Re: 2.0.6?

2011-12-08 Thread Torsten Förtsch
eck_n_requests.t which is expected. I further see another such message for every 3rd run of the whole test suite on average which puzzels me a bit but is probably okay given the nature of what is tested. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.nam

developer with 32bit linux needed to review a patch

2011-12-18 Thread Torsten Förtsch
idea of all the implications of the patch. I don't think it could be applied as is for the general case. Joe, I have added as a recipient because according to a comment in the code you are the author of the stuff that is removed by the patch. What do you think about it? Torsten Förtsch

Re: developer with 32bit linux needed to review a patch

2011-12-19 Thread Torsten Förtsch
ver, this thread: http://www.gossamer-threads.com/lists/modperl/modperl/103514#103514 So, it seems the problem is not Ububtu/Debian specific. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http

Re: [RELEASE CANDIDATE] Apache-Test-1.37 RC1

2012-01-04 Thread Torsten Förtsch
at prefork.c:903 #25 ap_mpm_run (_pconf=, plog=, s=) at prefork.c:1107 #26 0x004286d4 in main (argc=7, argv=0x7fffceabde38) at main.c:751 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net --

Re: [RELEASE CANDIDATE] Apache-Test-1.37 RC1

2012-01-24 Thread Torsten Förtsch
er if 1.36 behaves the same. > > Torsten, were you able to reproduce it in 1.36? yes. But I haven't had the time yet to figure out the reason. So, it's no regression. Torsten Förtsch -- Need professional modperl s

Re: [RELEASE CANDIDATE] Apache-Test-1.37 RC1

2012-01-28 Thread Torsten Förtsch
at one installation I haven't seen any problems. Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apa

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-01-31 Thread Torsten Förtsch
.pm(revision 1145161) @@ -134,6 +134,8 @@ TestCmdParms "Location" - -TestCmdParms "Limit" - + + +TestCmdParms "Limit" + + looks quite innocent. Except without the change the limit is part of the server's base config. With it it will be me

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-01-31 Thread Torsten Förtsch
On Tuesday, 31 January 2012 15:32:44 Torsten Förtsch wrote: > I can also confirm that r1145161 is > the first commit that shows this behavior. Working with r1145161, the minimal set of tests to trigger the bug is this (so far): t/TEST t/apache/add_config.t \ t/apache/conf

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-01 Thread Torsten Förtsch
mand directive code - not so sure. It can be basically everywhere in the code. The change in cmdparms.pm just modifies the memory layout a bit and now the stray pointer access changes a vital bit. That is what I suspect happens. Torsten Förtsch -- Need professional modperl support?

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-07 Thread Torsten Förtsch
On Tuesday, 31 January 2012 15:32:44 Torsten Förtsch wrote: > worker: I see the same behavior as Steve. I can also confirm that r1145161 > is the first commit that shows this behavior. Blame on me! > > $ svn diff -c1145161 > Index: t/response/TestDirect

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-10 Thread Torsten Förtsch
On Tuesday, 07 February 2012 18:01:50 Torsten Förtsch wrote: > What to do now? I'd suggest to revert change 1145161 and get 2.0.6 out > (perhaps with Steve's latest patch). Steve, do you use perl 5.14? If > yes, can you try if you see the "panic: free from wrong po

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-10 Thread Torsten Förtsch
On Friday, 10 February 2012 10:36:57 Fred Moyer wrote: > Should we add a Changes entry for this? Don't think so. What do we have svn commit messages for? I think an entry in the Changes file should be a bit more substantial. Torsten Förtsch -- Need professional modperl support? Hire m

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-14 Thread Torsten Förtsch
Another note, I have seen quite a few pieces of code reading: if (...) { /* should not happen */ return NULL; } This is really bad because it hides bugs. It does not fix anything. Wouldn't it be better to abort

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-14 Thread Torsten Förtsch
gi-bin/bugreport.cgi?msg=50;filename=0002-Fix-another- reference-counting-bug-uncovered-by-Perl.patch;att=3;bug=650675 Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-14 Thread Torsten Förtsch
me=0002-Fix-an > > other- reference-counting-bug-uncovered-by-Perl.patch;att=3;bug=650675 > > That's right those are the two patches. Feedback welcome, cc'ing > Niko. Take a look if you can, I'm ready to roll RC2. Committed revision 1244184. Thanks, Niko, for sor

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC1

2012-02-15 Thread Torsten Förtsch
md5sum -c mod_perl-2.0.6-rc2.tar.gz: OK Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.name) Like fantasy? http://kabatinte.net - To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org For addi

[RELEASE CANDIDATE]: mod_perl-2.0.6 RC2

2012-02-18 Thread Torsten Förtsch
I found such a script in build/. It tries to fetch the archive http://perl.apache.org/dist/win32- bin/apxs_win32.tar.gz via LWP. The newest files in this directory from 2007-04-18 03:32. Do we really need this? Torsten Förtsch -- Need professional modperl support? Hire me! (http://foertsch.na

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC2

2012-02-19 Thread Torsten Förtsch
.org/dist/win32-bin and provide a working modern combination of perl, httpd and modperl for win32 (and perhaps 64bit). Or we could drop the stuff. Keeping it as is is a bit embarrassing, I feel. > (I again haven't applied my patch myself because

Re: [RELEASE CANDIDATE]: mod_perl-2.0.6 RC2

2012-02-20 Thread Torsten Förtsch
On Sunday, 19 February 2012 22:35:21 Steve Hay wrote: > Torsten Förtsch wrote on 2012-02-19: > > How to proceed from here is a question we'd have to ask Fred. I see 3 > > options: > > > > 1) push out RC2 as is as 2.0.6 > > ... > Not so long ago (mod_perl

Re: [PATCH 2.0.5 futex] Fix to Children stuck on futex problem

2012-03-07 Thread Torsten Förtsch
see what happens. With the broken system one of the requests should hang forever. Ideally, the test client would abort the request and continue after a few seconds. Thanks again, Torsten Förtsch -- Need professional modperl support? Hire me

Re: [PATCH 2.0.5 futex] Fix to Children stuck on futex problem

2012-03-11 Thread Torsten Förtsch
On Wednesday, 07 March 2012 11:15:36 Torsten Förtsch wrote: > On Sunday, 04 March 2012 10:29:45 SalusaSecondus wrote: > > Broken behavior: > > 4 threads, 2 free interpreters > > 1: A (1,1) > > 2: A (2,0) > > 3: W > > 4: W > > 1: P (1,1) B > &g

  1   2   >