Re: [RELEASE CANDIDATE] libapreq 1.34-RC4

2008-11-26 Thread Philip M. Gollucci

Philip M. Gollucci wrote:

Issac Goldstand wrote:

http://people.apache.org/~issac/libapreq-1.34-RC4.tar.gz
Unit tests blow up spectacularly on solaris 2.10 but I don't think we 
support that and is related to Request.so failing to load.


It does compile.

I'll get a freebsd test for some sanity in the nearish future here.

I wouldn't worry about the solaris blow ups (1.33 doesn't work either)



Nothing liked getting pissed off to get things to work.
(I believe the only difference I did was -httpd vs -apxs)

All tests successful.
Files=4, Tests=25,  3 wallclock secs ( 1.22 cusr +  0.17 csys =  1.39 CPU)

Solaris 2.10
apache 1.3.41
mod_perl 1.30
perl 5.8.8

so thats a +1

Neither of Steve's changes are to apreq itself so they don't block the 
release.


+1: stevenhay, pgollucci
+0:
-0:
-1:

ISSAC did you vote ? if you do we get the required votes.

If do the release, make sure you send the e-mails from an @apache.org 
e-mail.


--

Philip M. Gollucci ([EMAIL PROTECTED]) c: 703.336.9354
Consultant - P6M7G8 Inc.  http://p6m7g8.net
Senior System Admin - RideCharge, Inc.  http://ridecharge.com
1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70  3F8C 75B8 8FFB DB9B 8C1C

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


Re: Should we release 2.10?

2008-11-26 Thread Adam Prime
Bojan Smojver wrote:
 On Fri, 2008-11-14 at 09:10 -0500, Adam Prime wrote:
   
 I was reminded of a documentation omission by an email on the mod_perl 
 list this morning.  Can something be added into the porting warnings here:

 http://httpd.apache.org/apreq/docs/libapreq2/group__apreq__xs__request.html

 mentioning that my @params = $r-param() no long returns a unique list 
 of the params.  IE that ?a=ba=c will return (a, a), not (a), which is 
 how it worked in libapreq1.
 

 Sure. Do you have a patch?

   

Does the attached patch work?

Adam
Index: glue/perl/lib/Apache2/Request.pm
===
--- glue/perl/lib/Apache2/Request.pm(revision 721076)
+++ glue/perl/lib/Apache2/Request.pm(working copy)
@@ -417,6 +417,10 @@
 =item * Cparam includes the functionality of Cparms() and Cparams(), so
 they are now deprecated and may be removed from a future 2.X release.
 
+=item * Cparam called in a list context no longer returns a unique list of
+paramaters.  The returned list contains multiple instances of the 
+parameter name for multivalued fields.
+
 =back
 
 


Re: Should we release 2.10?

2008-11-26 Thread Philip M. Gollucci

Adam Prime wrote:

Bojan Smojver wrote:

http://httpd.apache.org/apreq/docs/libapreq2/group__apreq__xs__request.html

Does the attached patch work?

Committed revision 721096.
Backported to branches/2_10 721099.


you forgot to bump the =over value :)

--

Philip M. Gollucci ([EMAIL PROTECTED]) c: 703.336.9354
Consultant - P6M7G8 Inc.  http://p6m7g8.net
Senior System Admin - RideCharge, Inc.  http://ridecharge.com
1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70  3F8C 75B8 8FFB DB9B 8C1C

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