hi all

can somebody on win32 verify this patch for me?  IIRC the required fix was
committed in november, and 2.0.49 is the latest release that contains the
fix, so all should be ok now.

thanks

--Geoff
Index: t/response/TestAPR/finfo.pm
===================================================================
RCS file: /home/cvs/modperl-2.0/t/response/TestAPR/finfo.pm,v
retrieving revision 1.8
diff -u -r1.8 finfo.pm
--- t/response/TestAPR/finfo.pm	14 Jan 2004 21:27:41 -0000	1.8
+++ t/response/TestAPR/finfo.pm	8 Apr 2004 16:30:50 -0000
@@ -9,6 +9,8 @@
 use Apache::TestConfig;
 use constant WIN32 => Apache::TestConfig::WIN32;
 
+use constant APACHE_2_0_49 => have_apache_version('2.0.49');
+
 use Apache::RequestRec ();
 use APR::Finfo ();
 use APR::Const -compile => qw(SUCCESS FINFO_NORM REG
@@ -98,9 +100,9 @@
 
     # tests for stuff not in perl's stat
     {
-        # XXX r->finfo->fname requires on Win32 a patched cvs apr
-        if (WIN32) {
-            skip "finfo.fname not available yet on Win32", 0;
+        # BACK_COMPAT_MARKER - fixed as of 2.0.49.
+        if (WIN32 && !APACHE_2_0_49) {
+            skip "finfo.fname requires Apache 2.0.49 or later", 0;
         }
         else {
             ok t_cmp($file,

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

Reply via email to