On Tue, Jun 28, 2016 at 4:15 PM, William A Rowe Jr <wr...@rowe-clan.net> wrote: > On Tue, Jun 28, 2016 at 8:46 AM, William A Rowe Jr <wr...@rowe-clan.net> > wrote: >> >> >> I suppose this would have been the more accurate toggle, in the first >> place? >> Any reason we would build lbmethods without balancer? >> >> enable_lbmethod_byrequests=$enable_proxy_balancer >> enable_lbmethod_bytraffic=$enable_proxy_balancer >> enable_lbmethod_bybusyness=$enable_proxy_balancer >> enable_lbmethod_heartbeat=$enable_proxy_balancer >> >> Patch committed to trunk, r1750507, to depend-on proxy_balancer >> and to use it's build flag as our default (static/shared or 'yes') > > > Also, in one case... > > APACHE_MODULE(lbmethod_heartbeat, Apache proxy Load balancing from > Heartbeats, , , $enable_proxy_balancer, , proxy_balancer) > > Wouldn't this depend on both [proxy_balancer,heartmonitor] (using the > new multiple-dependency syntax)?
I don't see where lbmethod_heartbeat depends on mod_heartmonitor in the code, but it seems to require mod_slotmem_shm though.