httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r726068 - /httpd/httpd/trunk/modules/proxy/Makefile.in
Date Sat, 13 Dec 2008 15:04:14 GMT

On Dec 12, 2008, at 5:45 PM, Ruediger Pluem wrote:

>
> The following patch seems to fix this:
>
> Index: modules/proxy/balancers/config2.m4
> ===================================================================
> --- modules/proxy/balancers/config2.m4  (Revision 726122)
> +++ modules/proxy/balancers/config2.m4  (Arbeitskopie)
> @@ -1,11 +1,4 @@
> APACHE_MODPATH_INIT(proxy/balancers)
> -if test "$enable_proxy" = "shared"; then
> -  proxy_mods_enable=shared
> -elif test "$enable_proxy" = "yes"; then
> -  proxy_mods_enable=yes
> -else
> -  proxy_mods_enable=no
> -fi
>
> proxy_lb_br_objs="mod_lbmethod_byrequests.lo"
> proxy_lb_bt_objs="mod_lbmethod_bytraffic.lo"
>
>
> The reason is that $enable_proxy is empty by the point of time the  
> above is
> executed. Since the proxy modules get configured earlier we know that
> proxy_mods_enable should already have a sane value. But I am no build
> system specialist. So some remote eyes please,
>

Ummm... I'm not seeing that at all... where is $enable_proxy being
cleared in configure?

Are you sure you did a 'make extraclean; ./buildconf' after
r726068

In any case, removing the duplication is good, but I don't see
it being required...

Mime
View raw message