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...@gmail.com> on 2012/01/10 16:43:18 UTC

Re: svn commit: r1229024 - in /httpd/httpd/trunk/docs/manual/mod: mod_heartbeat.xml mod_heartmonitor.xml

On Sun, Jan 8, 2012 at 11:20 PM,  <sc...@apache.org> wrote:
> Author: sctemme
> Date: Mon Jan  9 04:20:46 2012
> New Revision: 1229024
>
> URL: http://svn.apache.org/viewvc?rev=1229024&view=rev
> Log:
> Changes to mod_heart* docs upon review by rjung
>
> Modified:
>    httpd/httpd/trunk/docs/manual/mod/mod_heartbeat.xml
>    httpd/httpd/trunk/docs/manual/mod/mod_heartmonitor.xml
>
> Modified: httpd/httpd/trunk/docs/manual/mod/mod_heartbeat.xml
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/docs/manual/mod/mod_heartbeat.xml?rev=1229024&r1=1229023&r2=1229024&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/docs/manual/mod/mod_heartbeat.xml (original)
> +++ httpd/httpd/trunk/docs/manual/mod/mod_heartbeat.xml Mon Jan  9 04:20:46 2012
> @@ -43,7 +43,7 @@ of <directive module="mod_proxy">ProxyPa
>     <note type="warning">
>         To use <module>mod_heartbeat</module>,
>         <module>mod_status</module> and <module>mod_watchdog</module>
> -        must be either a static modules or, if a dynamic module, it must
> +        must be either a static modules or, if a dynamic module, must
>         be loaded before <module>mod_heartbeat</module>.

But why is mod_status required?  mod_status doesn't export any symbols
besides status_module and ap_run_status_hook, and mod_heartbeat
doesn't reference those.  (If it has a behavior requirement, which
isn't clear to me, there's no LoadModule ordering issue.

(yeah, why don't I/luser just try it myself :) )