Re: Bug#509457: [BUG] mod_perl2 test failures (when building Debian package)

2008-12-23 Thread Adam Prime

Philip M. Gollucci wrote:

Niko Tyni wrote:

On Mon, Dec 22, 2008 at 10:47:18AM -0600, Gunnar Wolf wrote:
The attached patch works around the problem by providing a credentials()
wrapper too. It's probably not quite correct, but the tests pass with
this on both libwww-perl 5.813 and 5.822.


Committed revision 728811.
http://svn.apache.org/viewvc?rev=728811view=rev

Thanks!



mod_perl 1 is affected by this as well.  This is actually the same bug i 
ran reported in this thread:


http://marc.info/?l=apache-modperlm=122886595126742w=2

Attached is a patch against the 1.x svn branch which resolves the 
problem there as well using the same technique the Niko used for 
TestRequest.pm


Adam

Index: t/internal/auth.t
===
--- t/internal/auth.t	(revision 729005)
+++ t/internal/auth.t	(working copy)
@@ -28,6 +28,7 @@
 @creds = qw(dougm mod_perl);  
 
 sub get_basic_credentials { shift-creds }
+sub credentials { shift-creds }
 
 sub creds {
 shift;


Re: [mp2] undefined symbol in make test with threaded Apache 2.2.11

2008-12-23 Thread craig

As Adam Prime notes, 5.8.9 became the current stable Perl recently.
It is the other new ingredient besides Apache 2.2.11.  I don't
normally introduce so many new things at once, but happen to be
working through a new sandbox/developmental version of our website.

One of the Apache developers suggested I should wait for 5.10.1, so
that's why I'm futzing with 5.8.9.  I don't know why the bug-report-
generating script would lie to us about what Perl I'm using.

Oooh your os?  Was that your machine doing something bad as you were  
typing?


I won't be doing development for most of a week, but if you have any
other questions, I'll be monitoring email.  Is there anything else
I should get through so that you will take a deeper look at this?
If so please advise.

Happy Holidays,
cmac

On Dec 22, 2008, at 11:25 PM, Philip M. Gollucci wrote:


PL_markstack_ptr
Thats a perl function its irrelevant of httpd versions.  Are you  
use you are using the /usr/local/bin/perl below ?



*** /usr/local/bin/perl -V
Summary of my perl5 (revision 5 version 8 subversion 9)  
configuration:

  Platform:
osname=freebsd, osvers=6.3-release,

Oooh my os, now you're in trouble.


-Dusethreads -Dusemultiplicity -Duseithreads -Duse64bitint'
usethreads=define use5005threads=undef useithreads=define  
usemultiplicity=define


perl 5.8.9 is not in the ports tree yet, how did you install perl,  
its also not a GA release, its a Release Candidate (RC)


If you get through all that, I'll take a deeper look.  Also, does  
this not happen with 2.2.10 though that should be irrelevant.




Re: [mp2] undefined symbol in make test with threaded Apache 2.2.11

2008-12-23 Thread Philip M. Gollucci

cr...@animalhead.com wrote:
Oooh your os?  Was that your machine doing something bad as you were 
typing?

/me is a FreeBSD person (see the .sig below)



I won't be doing development for most of a week, but if you have any
other questions, I'll be monitoring email.  Is there anything else
I should get through so that you will take a deeper look at this?
If so please advise.

I might suggest you try 5.8.8 perl as I'm 99.99% certain its not apache 
anything.


--

1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70  3F8C 75B8 8FFB DB9B 8C1C
Philip M. Gollucci (pgollu...@p6m7g8.com) c: 703.336.9354
Consultant  - P6M7G8 Inc.http://p6m7g8.net
Director IT - RideCharge, Inc.   http://ridecharge.com
Contractor  - PositiveEnergyUSA  http://positiveenergyusa.com
ASF Member  - Apache Software Foundation http://apache.org
FreeBSD Committer   - FreeBSD Foundation http://freebsd.org

Work like you don't need the money,
love like you'll never get hurt,
and dance like nobody's watching.


Re: Bug#509457: [BUG] mod_perl2 test failures (when building Debian package)

2008-12-23 Thread Philip M. Gollucci

Adam Prime wrote:
mod_perl 1 is affected by this as well.  This is actually the same bug i 
ran reported in this thread:


Attached is a patch against the 1.x svn branch which resolves the 
problem there as well using the same technique the Niko used for 
TestRequest.pm

Committed 729185

http://svn.apache.org/viewvc?rev=729185view=rev
--

1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70  3F8C 75B8 8FFB DB9B 8C1C
Philip M. Gollucci (pgollu...@p6m7g8.com) c: 703.336.9354
Consultant  - P6M7G8 Inc.http://p6m7g8.net
Director IT - RideCharge, Inc.   http://ridecharge.com
Contractor  - PositiveEnergyUSA  http://positiveenergyusa.com
ASF Member  - Apache Software Foundation http://apache.org
FreeBSD Committer   - FreeBSD Foundation http://freebsd.org

Work like you don't need the money,
love like you'll never get hurt,
and dance like nobody's watching.


Re: [mp2] undefined symbol in make test with threaded Apache 2.2.11

2008-12-23 Thread Philip M. Gollucci

cr...@animalhead.com wrote:

Neither mod_perl 2.0.4 nor the current build modperl-2.0_20081223052020
If you're going to do that 'current build', I'd use revision numbers of SVN 
instead of the data.



PL_markstack_ptr
This isn't perl 5.8.8 vs perl 5.8.9 related, if you diff the trees, only 1 
line is changed

-   *PL_markstack_ptr = (p) - PL_stack_base;\
+   *PL_markstack_ptr = (I32)((p) - PL_stack_base);\


  MP_APXS= /usr/local/apache2/bin/apxs
*** /usr/local/apache2/bin/httpd -V

You did not install www/apache22 port, but rather compiled by hand ?


*** /usr/local/bin/perl -V
Summary of my perl5 (revision 5 version 8 subversion 9) configuration:

So this isn't in the ports tree yet, so you also compiled perl by hand


  @INC:
/usr/local/lib/perl5/site_perl/5.8.9
/usr/local/lib/perl5/site_perl/5.8.8
/usr/local/lib/perl5/site_perl/5.8.7
/usr/local/lib/perl5/vendor_perl/5.8.9
/usr/local/lib/perl5/vendor_perl/5.8.7
Looks like your perl tree has some history, I'd try with a clean tree .. my 
money is here.



*** Packages of interest status:
mod_perl   : 1.29
mod_perl2  : 2.03, 2.05

Speaking of clean trees, that can't be good.
you have multiple versions of mod_perl2 installed which almost certainly means
that you'll get a mix match between .pm and .so files.  I'd clean that up


FWIW, you really shouldn't install your own perl into /usr/local/bin/perl on 
FreeBSD you should put yours somewhere else.





1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70  3F8C 75B8 8FFB DB9B 8C1C
Philip M. Gollucci (pgollu...@p6m7g8.com) c: 703.336.9354
Consultant  - P6M7G8 Inc.http://p6m7g8.net
Director IT - RideCharge, Inc.   http://ridecharge.com
Contractor  - PositiveEnergyUSA  http://positiveenergyusa.com
ASF Member  - Apache Software Foundation http://apache.org
FreeBSD Committer   - FreeBSD Foundation http://freebsd.org

Work like you don't need the money,
love like you'll never get hurt,
and dance like nobody's watching.


svn commit: r729185 - /perl/modperl/branches/1.x/t/internal/auth.t

2008-12-23 Thread pgollucci
Author: pgollucci
Date: Tue Dec 23 17:03:11 2008
New Revision: 729185

URL: http://svn.apache.org/viewvc?rev=729185view=rev
Log:
Fix broken tests due to a bug in libwww-perl 5.820.

Reported by: Gunnar Wolf gw...@gwolf.org
Submited by: Niko Tyni nt...@debian.org
Submited by: Adam Prime adam.pr...@utoronto.ca
Reviewed by: pgollucci

Message ID: 49510b72.80...@utoronto.ca

Modified:
perl/modperl/branches/1.x/t/internal/auth.t

Modified: perl/modperl/branches/1.x/t/internal/auth.t
URL: 
http://svn.apache.org/viewvc/perl/modperl/branches/1.x/t/internal/auth.t?rev=729185r1=729184r2=729185view=diff
==
--- perl/modperl/branches/1.x/t/internal/auth.t (original)
+++ perl/modperl/branches/1.x/t/internal/auth.t Tue Dec 23 17:03:11 2008
@@ -28,6 +28,7 @@
 @creds = qw(dougm mod_perl);  
 
 sub get_basic_credentials { shift-creds }
+sub credentials { shift-creds }
 
 sub creds {
 shift;




svn commit: r729189 - /perl/Apache-DBI/branches/

2008-12-23 Thread pgollucci
Author: pgollucci
Date: Tue Dec 23 17:34:56 2008
New Revision: 729189

URL: http://svn.apache.org/viewvc?rev=729189view=rev
Log:
create branches


Added:
perl/Apache-DBI/branches/



svn commit: r729190 - /perl/Apache-DBI/branches/AuthDBI_rewrite/

2008-12-23 Thread pgollucci
Author: pgollucci
Date: Tue Dec 23 17:36:46 2008
New Revision: 729190

URL: http://svn.apache.org/viewvc?rev=729190view=rev
Log:
branch to clean up AuthDBI so its actually testable


Added:
perl/Apache-DBI/branches/AuthDBI_rewrite/   (props changed)
  - copied from r725274, perl/Apache-DBI/trunk/

Propchange: perl/Apache-DBI/branches/AuthDBI_rewrite/
--
svn:mergeinfo =