This all sounds excellent.Having an APACHE_INST would be a good idea, as it's then more intuitive of what it represents ... Perhaps a combination of APACHE_INST and INSTALL_LIB would be good, in order to keep the flavour of the current build procedure:
- APACHE_INST: the Apache install directory, or - APACHE_SRC, which can be either the Apache build or install directory,
With this, - INSTALL_DLL: where to put mod_perl.so. If not given, defaults to APACHE_INST/modules, or to APACHE_SRC/modules, if these exist; - INSTALL_LIB: where to put mod_perl.lib. If not given, defaults to APACHE_INST/libexec, or to APACHE_SRC/libexec, if these exist;
In Apache::MyConfig, the location of the mod_perl lib would then be the value of INSTALL_LIB, if that exists or was set; if not, then it would stay at the current value of the mod_perl source location.
Setting MODPERL_INC to the installed location rather than the build location certainly makes a lot of sense. A definite thumbs-up to that.
As for the location of the mod_perl header files in
Apache::MyConfig, right now for Win32 they're set as
/Path/to/mod_perl/sources/src/modules/perl. One instead
could use Apache::src->new->inc, which gives the installed
path (under the Perl tree). However, this wouldn't be a
compatible change with the current behaviour, as
Apache::src->new->inc is a string including the '-I' before
the directories, so that it can be used directly in a $(CC)
command. But I think it would be more convenient to use
Apache::src->new->inc, for the same reason as specifying an
installed location for the mod_perl.lib - that way, people
can delete the mod_perl build directory after installation.
What does this sound like?
But I'm not so sure about introducing the '-I' if that breaks backwards compatibility. Depends how much you think people would be affected by it; I don't really know.
Would it be possible to play it safe, and set MODPERL_INC to the installed location, but without the '-I's?
- Steve
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
