[Patch] Apache::DBI Clean Transaction handling

2003-09-30 Thread Patrick Mulvany
Hi, Please can I have some feed back on this. This patch fixes the noted issue in the Apache::DBI POD. Note, that this CleanupHandler will only be used, if the initial data_source sets AutoCommit => 0. It will not be used, if AutoCommit will be turned off, after the conn

Re: t/SMOKE on win32

2003-09-30 Thread Steve Hay
Randy Kobes wrote: On Wed, 17 Sep 2003, Steve Hay wrote: Stas Bekman wrote: Steve Hay wrote: I tried running "perl t/SMOKE" but I just get this: = C:\Temp\modperl-2.0>perl t/SMOKE *** Using random number seed: 1012582729 (autogenerated) ***

Re: [Patch] Apache::DBI Clean Transaction handling

2003-09-30 Thread Stas Bekman
Patrick Mulvany wrote: Hi, Please can I have some feed back on this. This patch fixes the noted issue in the Apache::DBI POD. Note, that this CleanupHandler will only be used, if the initial data_source sets AutoCommit => 0. It will not be used, if AutoCommit will be tur

Re: t/SMOKE on win32

2003-09-30 Thread Stas Bekman
Randy Kobes wrote: On Wed, 17 Sep 2003, Steve Hay wrote: Stas Bekman wrote: Steve Hay wrote: I tried running "perl t/SMOKE" but I just get this: = C:\Temp\modperl-2.0>perl t/SMOKE *** Using random number seed: 1012582729 (autogenerated) *** -

Re: [PATCH] add PERL5LIB to @INC in the correct order

2003-09-30 Thread Stas Bekman
Simon Flack wrote: Problem: at the beggining of each request PERL5LIB is prepended to @INC in reverse order. E.g.: SetEnv PERL5LIB "one:two:three:four" results in this: @INC = ('four', 'three', 'two', 'one', '/usr', '...'); Environment: Tested with perl 5.6.1, mod_perl 1.27 an

Re: t/SMOKE on win32

2003-09-30 Thread Barrie Slaymaker
On Sep 30 2003, Stas Bekman wrote: Randy Kobes wrote: Randy, do you think it's going to be hard to fix IPC::Run do the right thing on win32? If IPC::Run is fixed, we don't need to add win32 cruft to our code. Sorry if I forgot the start of a thread; what's IPC::Run not doing right? The only open

Re: RE : [mp2] PERLIO_K_RAW in apr_perlio.c

2003-09-30 Thread Stas Bekman
Randy Kobes wrote: On Sun, 28 Sep 2003, Ayhan Ulusoy wrote: I am worried about this. Does 'PERLIO_K_RAW' also make the following call "do nothing" ? : binmode($apr_f, 'utf8'); That's a good point ... The binmode($apr_f) call (without a layer) seems to "do nothing", in the sense of not b

Re: t/SMOKE on win32

2003-09-30 Thread Stas Bekman
Barrie Slaymaker wrote: On Sep 30 2003, Stas Bekman wrote: Randy Kobes wrote: Randy, do you think it's going to be hard to fix IPC::Run do the right thing on win32? If IPC::Run is fixed, we don't need to add win32 cruft to our code. Sorry if I forgot the start of a thread; what's IPC::Run not

Re: [mp2 RELEASE CANDIDATE] 1.99_10 is available

2003-09-30 Thread Stas Bekman
Autrijus Tang wrote: On Fri, Sep 26, 2003 at 04:09:38PM -0400, Geoffrey Young wrote: Please test and report any problems. http://apache.org/~stas/mod_perl-1.99_10-dev.tar.gz works fine for me. Works fine for me with Win2k/perl5.8.1/httpd2.0.47/Mason/RT3. Thank you all for testing! _

Re: t/SMOKE on win32

2003-09-30 Thread Barrie Slaymaker
On Sep 30 2003, Stas Bekman wrote: Isn't IPC::Run3 based on IPC::Run? So if IPC::Run doesn't work... No, it's smaller, faster, lighter ;) It's "all new code", no select(). Uses File::Temp temporary files to optionally queue up input for the child, runs the child, optionally with any stdout and/o

Re: t/SMOKE on win32

2003-09-30 Thread Randy Kobes
On Tue, 30 Sep 2003, Stas Bekman wrote: > Barrie Slaymaker wrote: > > On Sep 30 2003, Stas Bekman wrote: > > > >> Randy Kobes wrote: Randy, do you think it's going to be > >> hard to fix IPC::Run do the right thing on win32? If > >> IPC::Run is fixed, we don't need to add win32 cruft to > >> our c

Re: t/SMOKE on win32

2003-09-30 Thread Stas Bekman
Barrie Slaymaker wrote: On Sep 30 2003, Stas Bekman wrote: Isn't IPC::Run3 based on IPC::Run? So if IPC::Run doesn't work... No, it's smaller, faster, lighter ;) It's "all new code", no select(). Uses File::Temp temporary files to optionally queue up input for the child, runs the child, optio

RE : [mp2] PERLIO_K_RAW in apr_perlio.c

2003-09-30 Thread Ayhan Ulusoy
It is normal that the "\015\012" stuff works on linux, precisely because of what the ":crlf" filter does. But the way the test is written, it is bound to fail on EBCDIC machines (if that matters at all for mod_perl), because CR LF doesn't spell out that way in EBCDIC. When the perl code writes o

Re: RE : [mp2] PERLIO_K_RAW in apr_perlio.c

2003-09-30 Thread Geoffrey Young
# Define the CRLF sequence. I can't use a simple "\r\n" because the meaning # of "\n" is different on different OS's (sometimes it generates CRLF, sometimes LF # and sometimes CR). The most popular VMS web server # doesn't accept CRLF -- instead it wants a LR. EBCDIC machines don't # use ASCII,

anon subs in httpd.conf

2003-09-30 Thread Geoffrey Young
hi all... I noticed today that this now works PerlLogHandler 'sub { warn shift->as_string }' (a favorite debugging line of mine), much to my surprise, since I know back in 1.99_07 (before I implemented as_string) I tried it and it didn't work - it threw the 'anonymous handlers not (yet) supp

[mp2] useless warning on ModPerl::Util::exit()

2003-09-30 Thread ydnar
Bug report follows.   Thanks a million. y     -8<-- Start Bug Report 8<--1. Problem Description:   Useless warning in error_log:     [Mon Sep 29 16:43:15 2003] -e: Use of uninitialized value.   This occurs whenever a handler script calls ModPerl::Util::ex

[mp2] trouble reading post using CGI.pm

2003-09-30 Thread christian
1. Problem Description: I have trouble getting iniformation from CGI-forms sent by HTTP/POST. To reproduce the problem; use the testpackage I've written and included in section 5 of this mail. I've also included what I've found relevant from my mod_perl/Apache-setup in section 4. My observatio

Useless use of a constant in void context at Makefile.PL line 467

2003-09-30 Thread Carl Brewer
Stas et al, current CVS pull (as of this morning) steel1: {173} perl mp2bug -8<-- Start Bug Report 8<-- 1. Problem Description: steel1: {167} perl Makefile.PL MP_APXS=/usr/local/apache2/bin/apxs Useless use of a constant in void context at Makefile.PL line

Re: Useless use of a constant in void context at Makefile.PL line 467

2003-09-30 Thread Stas Bekman
Carl Brewer wrote: current CVS pull (as of this morning) steel1: {167} perl Makefile.PL MP_APXS=/usr/local/apache2/bin/apxs Useless use of a constant in void context at Makefile.PL line 467. steel1: {168} The offending line is : $string .= Apache::Test::install::nuke_Apache__test_target() I

Re: Useless use of a constant in void context at Makefile.PL line 467

2003-09-30 Thread Carl Brewer
Stas Bekman wrote: Nothing has changed since the 1.99_10 release in the build sw. Do you get a different behavior if you try with 1.99_10? Also are you sure that the line is reported correctly? Try to verify that the reported line is correct: I'll have a play with this tonight when I get home

Re: t/SMOKE on win32

2003-09-30 Thread Randy Kobes
On Tue, 30 Sep 2003, Steve Hay wrote: > Randy Kobes wrote: [ ... ] > >Here's a first stab at a patch to get t/SMOKE running on > >Win32. I tried to localize the Win32 changes by putting > >them in their own sub. The full functionality is probably > >not all there - it still needs more testing - bu

Re: t/SMOKE on win32

2003-09-30 Thread Randy Kobes
On Tue, 30 Sep 2003, Stas Bekman wrote: > Barrie Slaymaker wrote: > > On Sep 30 2003, Stas Bekman wrote: > > > >> Isn't IPC::Run3 based on IPC::Run? So if IPC::Run doesn't work... > > > > No, it's smaller, faster, lighter ;) > > > > It's "all new code", no select(). Uses File::Temp > > temporary