On 4/15/2010 10:08 PM, Fred Moyer wrote:
RT: https://rt.cpan.org/Ticket/Display.html?id=32993
Please take a look at this revised patch here and give a +1, 0, or -1. Thanks!
Index: lib/Apache/TestHarnessPHP.pm
===================================================================
--- lib/Apache/TestHarnessPHP.pm (revision 934663)
+++ lib/Apache/TestHarnessPHP.pm (working copy)
@@ -25,29 +25,13 @@
use vars qw(@ISA);
@ISA = qw(Apache::TestHarness);
+use TAP::Formatter::Console;
+use TAP::Harness;
-# Test::Harness didn't start using Test::Harness::Straps until 2.38
-# everything except t/foo.php with earlier versions, so let things go
-# on without it
-my $phpclient = eval {
- require Test::Harness;
- Test::Harness->VERSION(2.38);
+my $phpbin = `which php`;
I forget where off the top of my head, but A-T has a function to search
$PATH for binaries.
+chomp($phpbin);
+my $phpclient = -x `$phpbin`;
--
------------------------------------------------------------------------
1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70 3F8C 75B8 8FFB DB9B 8C1C
Philip M. Gollucci (pgollu...@p6m7g8.com) c: 703.336.9354
VP Apache Infrastructure; Member, Apache Software Foundation
Committer, FreeBSD Foundation
Consultant, P6M7G8 Inc.
Sr. System Admin, Ridecharge Inc.
Work like you don't need the money,
love like you'll never get hurt,
and dance like nobody's watching.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org
For additional commands, e-mail: dev-h...@perl.apache.org
