httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bishop <bish...@sco.com>
Subject Re: More on PR#6397
Date Fri, 29 Sep 2000 19:19:47 GMT


"William A. Rowe, Jr." wrote:
> 
> > From: Bishop [mailto:bishop@platypus.bc.ca]
> > Sent: Thursday, September 28, 2000 7:41 PM
> >
> > "William A. Rowe, Jr." wrote:
> > > Noted the alpha sort - very good, but need to use readdir and
> > > then sort it, not elegant, but at least it's a one shot.  Call it
> > > httpd.d by default and I'm happy, plus port this for 2.0 before
> > > mid-October.
> >
> > I'm mostly on track with ya, Bill, except for the last part.
> > What'm I to label as httpd.d?
> 
> I'm just suggesting that the default/example directory name should
> correspond to the .conf file we built.  Therefore, if we are building
> with httpd.conf  -  we should watch for httpd.d - likewise the user
> who reconfiged Apache to use apache.conf should be watching apache.d
> 
> Clearer?  I hope --- cool idea by the way.

I got you now.  

You're thinking something like this?

--- apache_1.3.12/src/include/httpd.h-old       Fri Sep 29 14:23:13 2000
+++ apache_1.3.12/src/include/httpd.h   Fri Sep 29 14:23:55 2000
@@ -208,7 +208,7 @@
 
 /* The name of the server config file */
 #ifndef SERVER_CONFIG_FILE
-#define SERVER_CONFIG_FILE "conf/httpd.conf"
+#define SERVER_CONFIG_FILE "conf/httpd.d"
 #endif
 
 /* The name of the document config file */

Imho, this should be an OEM patch, not a vendor patch.

I was thinking that it'd be keen to s/conf/d/ in SERVER_CONFIG_FILE
before doing the inital open (eg http_main.c:4955), but we have no idea
what the filesystem looks like before we actually read the conf file. 
It's almost a classic chicken/egg problem.

Then, and this is all after reading your latest note above, I thought
it'd be impressive to check for it on the initial load in
http_config.c:1481, but again the chicken and the egg.  How do we set a
switch to hint to the executable that we're loading a dir-based config?
I decided that maybe a switch on the command line would HAVE to be it,
because anything else would be after the problem.

I decided I liked -f.

OEMs should be using the patch above, imho, and sysadmins should be
using a -f switch.  I couldn't find any significant middle ground that
wouldn't involve a bucket of code-mucking.  



--
Bishop Clark                                          ISV Engineer
908.790.2280                              The Santa Cruz Operation

   "I've ALWAYS been intimidating." -- D Michael Drangula

Mime
View raw message