[RELEASE CANDIDATE] Apache-Test-1.31 RC2

Greetings,

Please, download, test, and report back on success or failure for this
Apache::Test release candidate.

Development from 1.30 has been ongoing for about two years, so this is
as good a time as ever to make a release.

http://people.apache.org/~phred/Apache-Test-1.31-rc2.tar.gz

This candidate resolves an issue with httpd binaries that were
compiled without mod_version, and also adds a fix for the following RT
ticket:

https://rt.cpan.org/Ticket/History.html?id=21554

Re: [RELEASE CANDIDATE] Apache-Test-1.31 RC2

On Tue, 09 Feb 2010 22:30:32 +0100, Fred Moyer
wrote:

> Please, download, test, and report back on success or failure for this
> Apache::Test release candidate.

Seems to pass all the tests for me, even if it reports
missing modules (?).

cosimo@ubuntu:~/src/Apache-Test-1.31-rc2$ make test
/usr/bin/perl -Iblib/arch -Iblib/lib \
t/TEST -clean
[warning] setting ulimit to allow core files
ulimit -c unlimited; /usr/bin/perl
/home/cosimo/src/Apache-Test-1.31-rc2/t/TEST -clean
APACHE_TEST_GROUP= APACHE_TEST_HTTPD= APACHE_TEST_PORT= APACHE_TEST_USER=
APACHE_TEST_APXS= \
/usr/bin/perl -Iblib/arch -Iblib/lib \
t/TEST -bugreport -verbose=0
[warning] setting ulimit to allow core files
ulimit -c unlimited; /usr/bin/perl
/home/cosimo/src/Apache-Test-1.31-rc2/t/TEST -bugreport -verbose=0
/usr/sbin/apache2 -d /home/cosimo/src/Apache-Test-1.31-rc2/t -f
/home/cosimo/src/Apache-Test-1.31-rc2/t/conf/httpd.conf -D APACHE2 -D
PERL_USEITHREADS
using Apache/2.2.11 (prefork MPM)

waiting 60 seconds for server to start: .
waiting 60 seconds for server to start: ok (waited 0 secs)
server localhost:8529 started
t/bad_coding.............ok
t/cookies................skipped
all skipped: cannot find one of cgi or cgid, cannot find module
'mod_alias.c'
t/next_available_port....skipped
all skipped: cannot find module 'mod_cgi.c', cannot find module
'mod_cgid.c', cannot find module 'mod_env.c'
t/ping...................ok
t/redirect...............skipped
all skipped: cannot find module 'mod_alias.c'
t/request................ok
All tests successful, 3 tests skipped.
Files=6, Tests=13, 2 wallclock secs ( 2.14 cusr + 0.22 csys = 2.36 CPU)
[warning] server localhost:8529 shutdown


--
Cosimo

Re: [RELEASE CANDIDATE] Apache-Test-1.31 RC2

Passes all tests on 2.2.10 w/ mp 2.0.4 and perl 5.8.8 on linux (ubuntu
hardy).

Passes all tests on 2.2.14 w/ mp 2.0.4 and perl 5.8.8 on solaris 5.10

I tried to run it against our (extremely old) 1.3 setup at work, but i'm
getting weird errors from httpd when i start it up:

Ouch! ap_mm_create(1048576, "/var/httpd/httpd.mm.2591") failed
Error: MM: mm:core: failed to open semaphore file (Permission denied):
OS: No such file or directory

Not sure what that's about, but i think it has more to do with works
weird setup than anything else.

I'll try to remember to run this against some more combinations later
tonight.

Adam


Fred Moyer wrote:
> Greetings,
>
> Please, download, test, and report back on success or failure for this
> Apache::Test release candidate.
>
> Development from 1.30 has been ongoing for about two years, so this is
> as good a time as ever to make a release.
>
> http://people.apache.org/~phred/Apache-Test-1.31-rc2.tar.gz
>
> This candidate resolves an issue with httpd binaries that were
> compiled without mod_version, and also adds a fix for the following RT
> ticket:
>
> https://rt.cpan.org/Ticket/History.html?id=21554
>
> ------------------------------------------------------------ ---------
> To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
> For additional commands, e-mail: dev-help@perl.apache.org
>

Re: [RELEASE CANDIDATE] Apache-Test-1.31 RC2

On 10/02/10 08:30, Fred Moyer wrote:
> Please, download, test, and report back on success or failure for this
> Apache::Test release candidate.

Passes all tests on 2.2.13 w/ mp 2.0.4 and perl 5.10 on linux (fedora 11)
Passes all tests on 2.2.9 w/ mp 2.0.4 and perl 5.10 on linux (debian lenny)

Re: [RELEASE CANDIDATE] Apache-Test-1.31 RC2

I'll take that as a +1? :)

+1 from me, so that makes 2. Would like to get at least two more +1s
from mod_perl devs.

Thanks to everyone else who has taken a few minutes to test out this version.

On Tue, Feb 9, 2010 at 2:01 PM, Adam Prime wrote:
> Passes all tests on 2.2.10 w/ mp 2.0.4 and perl 5.8.8 on linux (ubuntu
> hardy).
>
> Passes all tests on 2.2.14 w/ mp 2.0.4 and perl 5.8.8 on solaris 5.10
>
> I tried to run it against our (extremely old) 1.3 setup at work, but i'm
> getting weird errors from httpd when i start it up:
>
> Ouch! ap_mm_create(1048576, "/var/httpd/httpd.mm.2591") failed
> Error: MM: mm:core: failed to open semaphore file (Permission denied): OS:
> No such file or directory
>
> Not sure what that's about, but i think it has more to do with works weird
> setup than anything else.
>
> I'll try to remember to run this against some more combinations later
> tonight.
>
> Adam
>
>
> Fred Moyer wrote:
>>
>> Greetings,
>>
>> Please, download, test, and report back on success or failure for this
>> Apache::Test release candidate.
>>
>> Development from 1.30 has been ongoing for about two years, so this is
>> as good a time as ever to make a release.
>>
>> http://people.apache.org/~phred/Apache-Test-1.31-rc2.tar.gz
>>
>> This candidate resolves an issue with httpd binaries that were
>> compiled without mod_version, and also adds a fix for the following RT
>> ticket:
>>
>> https://rt.cpan.org/Ticket/History.html?id=21554
>>
>> ------------------------------------------------------------ ---------
>> To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
>> For additional commands, e-mail: dev-help@perl.apache.org
>>
>
>
> ------------------------------------------------------------ ---------
> To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
> For additional commands, e-mail: dev-help@perl.apache.org
>
>

Re: [RELEASE CANDIDATE] Apache-Test-1.31 RC2

This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
--------------enig4407AC92165910A82C2BAB20
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On 10-02-09 16:30 , Fred Moyer wrote:
> Greetings,
>=20
> Please, download, test, and report back on success or failure for this
> Apache::Test release candidate.
>=20
> Development from 1.30 has been ongoing for about two years, so this is
> as good a time as ever to make a release.
>=20
> http://people.apache.org/~phred/Apache-Test-1.31-rc2.tar.gz

Passes all tests on OSX/10.6 Apache/2.2.13/prefork mod_perl/2.0.4
Perl/v5.10.0

One small nit, not sure why, but t/next_available_port.t was modified
and will only be run if mod_cgi *and* mod_cgid is installed, which is
not very likely. Shouldn't this be what it does instead ?

diff -rdu Apache-Test-1.31-rc2/t/next_available_port.t
Apache-Test-1.31-rc2.test/t/next_available_port.t
--- Apache-Test-1.31-rc2/t/next_available_port.t 2009-04-18
05:59:26.000000000 -0400
+++ Apache-Test-1.31-rc2.test/t/next_available_port.t 2010-02-09
21:20:26.000000000 -0500
@@ -7,8 +7,7 @@
use Apache::TestRequest;
use Apache::TestUtil;

-plan tests =3D> 1, need need_module('mod_cgi.c'),
- need_module('mod_cgid.c'),
+plan tests =3D> 1, need need_cgi,
need_module('mod_env.c');

my $url =3D '/cgi-bin/next_available_port.pl';


Looking at Changes, I can see

"Use need_module('mod_cgi.c') and need_module('mod_cgid.c') in
t/next_available_port.t instead of need_cgi. On case insensitive file
systems such as OS X, need_cgi will fulfill the requirement with cgi.pm,
when mod_cgi.c is the desired requirement.
[Fred Moyer ]"

Wouldn't the correct solution be to fix need_cgi then :

diff -ru Apache-Test-1.31-rc2.test/blib/lib/Apache/Test.pm
Apache-Test-1.31-rc2/blib/lib/Apache/Test.pm
--- Apache-Test-1.31-rc2.test/blib/lib/Apache/Test.pm 2010-02-09
13:21:39.000000000 -0500
+++ Apache-Test-1.31-rc2/blib/lib/Apache/Test.pm 2010-02-09
21:27:47.000000000 -0500
@@ -348,7 +348,7 @@
}

sub need_cgi {
- return _need_multi(qw(cgi cgid));
+ return _need_multi(qw(cgi.c cgid.c));
}

--=20
Philippe M. Chiasson GPG: F9BFE0C2480E7680 1AE53631CB32A107 88C3A5A5
http://gozer.ectoplasm.org/ m/gozer\@(apache|cpan|ectoplasm)\.org/


--------------enig4407AC92165910A82C2BAB20
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iD8DBQFLchykyzKhB4jDpaURAqN7AJ96pbN2lb2y6UpQ57MTx4sLl5F9QACa Air+
i7mPkk5dpuiKDB04SDdSPpk=
=9qKS
-----END PGP SIGNATURE-----

--------------enig4407AC92165910A82C2BAB20--

Re: [RELEASE CANDIDATE] Apache-Test-1.31 RC2

On Tue, Feb 9, 2010 at 6:40 PM, Philippe M. Chiasson
wrote:
>
> One small nit, not sure why, but t/next_available_port.t was modified
> and will only be run if mod_cgi *and* mod_cgid is installed, which is
> not very likely. Shouldn't this be what it does instead ?
>
> diff -rdu Apache-Test-1.31-rc2/t/next_available_port.t
> Apache-Test-1.31-rc2.test/t/next_available_port.t
> --- Apache-Test-1.31-rc2/t/next_available_port.t =A0 =A0 =A0 =A02009-04-1=
8
> 05:59:26.000000000 -0400
> +++ Apache-Test-1.31-rc2.test/t/next_available_port.t =A0 2010-02-09
> 21:20:26.000000000 -0500
> @@ -7,8 +7,7 @@
> =A0use Apache::TestRequest;
> =A0use Apache::TestUtil;
>
> -plan tests =3D> 1, need need_module('mod_cgi.c'),
> - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 need_module('mod_cgid.c'),
> +plan tests =3D> 1, need need_cgi,
> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0need_module('mod_env.c');

Yes, thanks for the spot there. Applied.


> Looking at Changes, I can see
>
> "Use need_module('mod_cgi.c') and need_module('mod_cgid.c') in
> t/next_available_port.t instead of need_cgi. =A0On case insensitive file
> systems such as OS X, need_cgi will fulfill the requirement with cgi.pm,
> when mod_cgi.c is the desired requirement.
> [Fred Moyer ]"
>
> Wouldn't the correct solution be to fix need_cgi then :
>
> diff -ru Apache-Test-1.31-rc2.test/blib/lib/Apache/Test.pm
> Apache-Test-1.31-rc2/blib/lib/Apache/Test.pm
> --- Apache-Test-1.31-rc2.test/blib/lib/Apache/Test.pm =A0 2010-02-09
> 13:21:39.000000000 -0500
> +++ Apache-Test-1.31-rc2/blib/lib/Apache/Test.pm =A0 =A0 =A0 =A02010-02-0=
9
> 21:27:47.000000000 -0500
> @@ -348,7 +348,7 @@
> =A0}
>
> =A0sub need_cgi {
> - =A0 =A0return _need_multi(qw(cgi cgid));
> + =A0 =A0return _need_multi(qw(cgi.c cgid.c));
> =A0}

Ah, this is a much better approach. I don't think I completely
understood that _need_multi was OR instead of AND last time I looked
at it.

Both changes applied r909744. Thanks Philippe!