You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Jeff Trawick <tr...@attglobal.net> on 2003/05/14 12:40:36 UTC

Re: cvs commit: httpd-2.0/modules/mappers config9.m4

aaron@apache.org wrote:
> aaron       2003/05/13 22:11:38
> 
>   Modified:    .        Tag: APACHE_2_0_BRANCH CHANGES
>                modules/mappers Tag: APACHE_2_0_BRANCH config9.m4
>   Log:
>   mod_so can only be built statically. If the user wants modules to
>   be built as DSOs by default (eg. ./configure --enable-mods-shared=most)
>   then we must override the default, otherwise none of the modules will
>   end up being installed (even though they'll get built).

>   1.12.2.5  +7 -0      httpd-2.0/modules/mappers/config9.m4
>   
>   Index: config9.m4
>   ===================================================================
>   RCS file: /home/cvs/httpd-2.0/modules/mappers/config9.m4,v
>   retrieving revision 1.12.2.4
>   retrieving revision 1.12.2.5
>   diff -u -r1.12.2.4 -r1.12.2.5
>   --- config9.m4	13 May 2003 16:53:07 -0000	1.12.2.4
>   +++ config9.m4	14 May 2003 05:11:38 -0000	1.12.2.5
>   @@ -34,6 +34,13 @@
>            ;;
>    esac
>    
>   +dnl mod_so can only be built statically. If the user wants modules to
>   +dnl be built as DSOs by default (eg. ./configure --enable-mods-shared=most)
>   +dnl then we must override the default here.
>   +if test "x$enable_so" = "xyes"; then
>   +    enable_so="static"
>   +fi

This will break the following logic in configure.in:

if test "$enable_so" = "yes"; then
   case $host in
     *-ibm-aix*)
       good stuff
       ;;
     *beos)
       good stuff

I guess we should now change the check in configure.in, and make sure 
that enable_so is either "static" or "no".  Unfortunately, while I tried 
a number of testcases with the code I committed, I did not think to try 
"--enable-mods-shared=most" :(  Thanks!