> I've been thinking a bit about this and I think the attached is a better
> patch.

ugh, wrong patch.  sorry

--Geoff
Index: lib/Apache/TestConfigParse.pm
===================================================================
--- lib/Apache/TestConfigParse.pm	(revision 267203)
+++ lib/Apache/TestConfigParse.pm	(working copy)
@@ -345,8 +345,18 @@
             my $default_conf = $self->{httpd_defines}->{SERVER_CONFIG_FILE};
             $default_conf ||= catfile qw(conf httpd.conf);
             $file = catfile $base, $default_conf;
+
             # SERVER_CONFIG_FILE might be an absolute path
-            $file = $default_conf if !-e $file and -e $default_conf;
+            if (! -e $file && -e $default_conf) {
+                $file = $default_conf;
+            }
+            else {
+              # try a little harder
+              if (my $root = $self->{httpd_defines}->{HTTPD_ROOT}) {
+                  debug "using HTTPD_ROOT to resolve $default_conf";
+                  $file = catfile $root, $default_conf;
+              }
+            }
         }
     }
 

Reply via email to