William McKee wrote:
> On Wed, Aug 31, 2005 at 11:54:57AM -0400, Geoffrey Young wrote:
> 
>>$self->{httpd_basedir}, which looks to be calculated based on the location
>>of the httpd binary.  but it looks like it ought to be using HTTPD_ROOT
>>instead.  at least if it's defined.
>>
>>try this (completely untested)
> 
> 
> Perfect! That worked just fine. How can we get it added to A::T?

I've been thinking a bit about this and I think the attached is a better
patch.  basically, I think A-T goes through a lot of effort to separate the
current filesystem configuration from the hard-coded stuff in httpd in an
effort to make it easy to roll up an httpd install, move it someplace else,
and still have A-T be able to create a worthy configuration.  using
HTTPD_ROOT like that kinda breaks all of that for people expecting the
current behavior, so I'd rather use it as a last-ditch effort than off the bat.

can you give it a whirl?

--Geoff



Index: lib/Apache/TestConfigParse.pm
===================================================================
--- lib/Apache/TestConfigParse.pm	(revision 265514)
+++ lib/Apache/TestConfigParse.pm	(working copy)
@@ -341,7 +341,7 @@
     my $extra_file = $self->{vars}->{httpd_conf_extra};
 
     unless ($file and -e $file) {
-        if (my $base = $self->{httpd_basedir}) {
+        if (my $base = $self->{httpd_defines}->{HTTPD_ROOT} || $self->{httpd_basedir}) {
             my $default_conf = $self->{httpd_defines}->{SERVER_CONFIG_FILE};
             $default_conf ||= catfile qw(conf httpd.conf);
             $file = catfile $base, $default_conf;

Reply via email to