httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <>
Subject RE: cvs commit: apache-2.0 STATUS
Date Sun, 11 Jun 2000 06:20:25 GMT
> From: Greg Stein []
> Sent: Sunday, June 11, 2000 12:38 AM
> On Sun, Jun 11, 2000 at 01:19:19AM -0400, Manoj Kasichainula wrote:
> > On Sat, Jun 10, 2000 at 05:21:46PM -0000, 
> wrote:
> > >   +
> > >   +    * Go through ap_config.h and namespace-protect the 
> symbols (e.g. USE_*).
> > >   +      Some symbols can/should move to mpm_common.h 
> where possible.
> > 
> > This discussion was had before. The original intention was that
> > ap_config.h shouldn't be included, directly or indirectly, outside
> > Apache and its bundled modules. If it is being included, why?
> The following headers each include ap_config.h:
>     ap_base64.h
>     ap_hooks.h
>     ap_sha1.h
>     httpd.h
> The above files include ap_config because they need 
> API_EXPORT and friends.
> Wow. There is a ton of crap in there. For example, if HAVE_SYS_RESOURCE_H is
> defined, then it just includes the header. Why? 
> Whothehellknows. It just does.
> Feh... all that should disappear. The modules that need a header should
> include it themselves. ap_config.h (and httpd.h in turn) should not be an
> "include the world" header.

Agreed here too... ap_config.h should be the root of all apache sources,
defining (but not including) what we aught to be doing.  So I included
ap_config from those headers under the (mistaken) belief that ap_config
represents the cornerstone of apache sources, not (as it turns out) the 
ultimate bag of tricks and traps.

likewise, httpd.h should tell us what and how we go about compiling a server,
but not doing it all itself :-)  It (by default) is based on how we compile
our sources (ap_config) and aught to include that, but not the rest of all
headers we ever invent.

Would you like to suggest the rules of engagement, Greg, for how and why
headers are included, and what aught to include what else?


View raw message