Steve Hay wrote:
As I reported recently, compat/send_fd.t and directive/setupenv.t both currently (2.0.49 / 5.8.4 / 1.99_14) fail in the test suite on Win32, but work when tried in isolation.

After a little trial & error, I've found that running apr/pool.t before them make them fail

That what t/SMOKE is for. It does the job of finding such sequences for you.

, i.e.

    perl t/TEST compat/send_fd
    perl t/TEST directive/setupenv

both run OK, but

    perl t/TEST apr/pool compat/send_fd
    perl t/TEST apr/pool directive/setupenv

both fail (apr/pool.t itself runs OK, though).

Does this patch make any difference?

Index: t/response/TestAPR/pool.pm
===================================================================
RCS file: /home/cvs/modperl-2.0/t/response/TestAPR/pool.pm,v
retrieving revision 1.12
diff -u -r1.12 pool.pm
--- t/response/TestAPR/pool.pm  14 May 2004 07:58:21 -0000      1.12
+++ t/response/TestAPR/pool.pm  24 May 2004 18:32:13 -0000
@@ -377,16 +377,16 @@
     # errors. They are logged though. So as usual, one has to always
     # watch error_log (things like CGI::Carp's fatalsToBrowser) won't
     # quite be able to catch those.
-    {
-        my $p = APR::Pool->new;
-        t_server_log_error_is_expected();
-        $p->cleanup_register('some_bogus_non_existing', 1);
-    }
-    {
-        my $p = APR::Pool->new;
-        t_server_log_error_is_expected();
-        $p->cleanup_register(\&non_existing1, 1);
-    }
+#    {
+#        my $p = APR::Pool->new;
+#        t_server_log_error_is_expected();
+#        $p->cleanup_register('some_bogus_non_existing', 1);
+#    }
+#    {
+#        my $p = APR::Pool->new;
+#        t_server_log_error_is_expected();
+#        $p->cleanup_register(\&non_existing1, 1);
+#    }

     ### $p->clear ###
     {

--
__________________________________________________________________
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