Randy Kobes wrote:
For making the tests for apr/apr-ext share a common
base under modperl-2.0/t/lib/TestAPRlib/, one must
at present 'use lib q(.....)' in the appropriate
places for both the t/apr/ and t/apr-ext/ tests.
Would the following patch be OK to add this to
httpd.conf, so one doesn't have to repeat it?

You are reading my mind, Randy :)

Ideally it shouldn't go into Apache-Test, but into mod_perl specific config. But I think quite a few people may find t/lib to be useful, so +1 for that change

(but please commit from within Apache-Test/ so it'll be posted to the right cvs commit list and add a Changes entry. Thanks.)

===================================================
Index: lib/Apache/TestConfigPerl.pm

             next unless $_;
             print $fh "use lib '$_';\n";
         }
+        my $tlib = catfile $self->{vars}->{t_dir}, 'lib';

I think that should be catdir (though I never could quite figure what's the difference between the two, I guess I need to look at the source code :)


+        if (-d $tlib) {
+            print $fh "use lib '$tlib';\n";
+        }



-- __________________________________________________________________ Stas Bekman JAm_pH ------> Just Another mod_perl Hacker http://stason.org/ mod_perl Guide ---> http://perl.apache.org mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com http://modperlbook.org http://apache.org http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to