Apache::Test not finding global config file

[Geoffrey Young]

$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

[Geoffrey Young]

Post by Geoffrey Young
I've been thinking a bit about this and I think the attached is a better
patch.

ugh, wrong patch. sorry

--Geoff

[William McKee]

if you can do me the favor of trying a few different configuration scenarios
(including 1.3, non-apxs, /usr/local/apache, etc) and making sure nothing
breaks, that would be great. the box where my matrix used to live had a
hard drive failure, so I'm slowly rebuilding it as I find the time.

Besides testing under Apache2, I've tested with /usr/local/apache (this
was an Apache1 server and was non-apxs). That's all I have at my
disposal.

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?

I removed your original patch and added this one then tested it under
the two systems above. Looks good.


Thanks,
William

[Geoffrey Young]

Post by William McKee
I removed your original patch and added this one then tested it under
the two systems above. Looks good.

committed.

--Geoff