Re: Bug in Apache::Test - extra.conf included before modperl_inc.pl

[Matt Sergeant]

On Friday, Jul 18, 2003, at 19:04 Europe/London, Stas Bekman wrote:

Matt Sergeant wrote:
On Fri, 18 Jul 2003, Stas Bekman wrote:
As for the prompt, it's probably only useful from within CPAN shell. 
From the
command line it's much better to have people pass this argument to
Makefile.PL. Again if everybody sets their APXS env var in ~/.login, 
then all
the future testing using Apache::Test just works. Do you think it's 
not worth
to make people do this instead having an annoying prompt on each 
module build?
Can't you store the value prompted for when you install Apache::Test,
rather than every time you generate a new t/TEST? That way you get
prompted *once* and never again (of course you have to provide a way 
to
change it if you move your httpd).
If I have httpd-1.3 and httpd-2.0 on the same system, which one should 
be chosen? Or do you suggest to just let the user to make a single 
choice assuming that they will use only one most of the time and can 
still override the default if they want to change it?

Let's say that we store that value. Should this value be used as a 
default suggestion in the prompt, when a user hasn't supplied an 
explicit argument or should it be used without any prompt?
I would go with the latter. The situation of working with >1 httpd on 
the same system is reserved for guys like you and I, not the majority 
of people installing Apache::Test because it was a pre-requisite for 
things like Apache::Request. So just allow it to be overridden with a 
Makefile.PL argument.

As for your earlier question, the docs for Apache::Test should be in 
the Apache::Test distribution, in the .pm file preferably. I would 
never think to look anywhere else, especially not the mod_perl guide 
for module docs.

Matt.

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