> 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; + } + } } }