Re: Apache::DBI new version?

2010-02-04 Thread Adam Prime
Is ask even on this list anymore? Fred Moyer wrote: I saw some commits yesterday for Apache::DBI, and then saw a new version out: http://search.cpan.org/~abh/Apache-DBI-1.08/lib/Apache/DBI.pm For what it is worth, I would have been happy to test a release candidate and vote on it. It would ha

Re: [Fwd: Re: Apache::DBI]

2009-12-14 Thread Adam Prime
. > 3 out of 5 hunks FAILED -- saving rejects to file lib/Apache/DBI.pm.rej > >> Thanks, >> >> Adam >> >> >> >> Original Message >> Subject: Re: Apache::DBI >> Date: Mon, 27 Jul 2009 00:06:07 -0400 >> From: Adam Prime >> To: Fr

Re: [Fwd: Re: Apache::DBI]

2009-12-14 Thread Fred Moyer
rejects to file lib/Apache/DBI.pm.rej > > Thanks, > > Adam > > > > ---- Original Message > Subject: Re: Apache::DBI > Date: Mon, 27 Jul 2009 00:06:07 -0400 > From: Adam Prime > To: Fred Moyer > CC: dev@perl.apache.org > References: <4a65c9a1.

[Fwd: Re: Apache::DBI]

2009-12-14 Thread Adam Prime
, Adam Original Message Subject: Re: Apache::DBI Date: Mon, 27 Jul 2009 00:06:07 -0400 From: Adam Prime To: Fred Moyer CC: dev@perl.apache.org References: <4a65c9a1.70...@utoronto.ca> <4a65ec64.4060...@redhotpenguin.com> <4a65fe8e.1020...@p6m7g8.com&

Re: Apache::DBI

2009-07-26 Thread Adam Prime
Adam Prime wrote: > Fred Moyer wrote: >> On Tue, Jul 21, 2009 at 8:39 PM, Adam Prime wrote: >>> Adam Prime wrote: What I wanted to do was fix this issue: http://rt.cpan.org/Public/Bug/Display.html?id=36346 >> I was just looking over your original code where you connect() in >> startu

restart_count (was: Re: Apache::DBI)

2009-07-26 Thread Adam Prime
> Looks like we have a unit test for part of restart_count: > > > ph...@harpua ~/dev/svn/modperl/mod_perl-2.0 $ ack restart_count t/ > t/conf/post_config_startup.pl > 110:#my $cnt = Apache2::ServerUtil::restart_count(); > > t/htdocs/vhost/post_config.pl > 6:$TestVhost::config::restart_c

Re: Apache::DBI

2009-07-22 Thread Adam Prime
Fred Moyer wrote: > On Tue, Jul 21, 2009 at 8:39 PM, Adam Prime wrote: >> Adam Prime wrote: >>> What I wanted to do was fix this issue: >>> >>> http://rt.cpan.org/Public/Bug/Display.html?id=36346 > > I was just looking over your original code where you connect() in > startup.pl. Won't that cause

Re: Apache::DBI

2009-07-22 Thread Fred Moyer
On Tue, Jul 21, 2009 at 8:39 PM, Adam Prime wrote: > Adam Prime wrote: >> What I wanted to do was fix this issue: >> >> http://rt.cpan.org/Public/Bug/Display.html?id=36346 I was just looking over your original code where you connect() in startup.pl. Won't that cause issues since the database hand

Re: Apache::DBI

2009-07-22 Thread Fred Moyer
On Tue, Jul 21, 2009 at 8:39 PM, Adam Prime wrote: > Adam Prime wrote: >> What I wanted to do was fix this issue: >> >> http://rt.cpan.org/Public/Bug/Display.html?id=36346 >> >> either by checking the various server starting variables for each of the >> apis, or simply throwing an eval around the p

Re: Apache::DBI

2009-07-21 Thread Adam Prime
Adam Prime wrote: > What I wanted to do was fix this issue: > > http://rt.cpan.org/Public/Bug/Display.html?id=36346 > > either by checking the various server starting variables for each of the > apis, or simply throwing an eval around the push_handler. I've been looking into this bug a bit tonig

Re: Apache::DBI

2009-07-21 Thread Adam Prime
Continuing discussion from off list. Philip M. Gollucci wrote: Fred Moyer wrote: On 7/21/09 6:58 AM, Adam Prime wrote: If it get a new release ready, can / will one of you guys release it? You both have co-maint on cpan. I'd like to see some patches hit the list with test results first. I

Re: Apache::DBI 1.07 requires global $r

2008-07-14 Thread Philip M. Gollucci
Coll, I will roll with global $r for now. Fred, if you want to patch it go for, you should have SVN commits to it. If not, let me know I'll commit the patch. Its been on my back burner for a while. I'd rather release though unless you feel compelled to do so. ---

Re: Apache::DBI 1.07 requires global $r

2008-07-13 Thread Fred Moyer
Perrin Harkins wrote: On Sun, Jul 13, 2008 at 5:40 PM, Fred Moyer <[EMAIL PROTECTED]> wrote: I see a couple issues with the patch that caused this, but I'm not sure what problem the patch was trying to solve. Looks like it was trying to solve the rollback cleanup not working at all for MP2. P

Re: Apache::DBI 1.07 requires global $r

2008-07-13 Thread Perrin Harkins
On Sun, Jul 13, 2008 at 5:40 PM, Fred Moyer <[EMAIL PROTECTED]> wrote: > I see a couple issues with the patch that caused this, but I'm not sure what > problem the patch was trying to solve. Looks like it was trying to solve the rollback cleanup not working at all for MP2. Pretty serious bug. >