Return-Path: Delivered-To: new-httpd-archive@hyperreal.org Received: (qmail 16959 invoked by uid 6000); 9 May 1998 02:56:05 -0000 Received: (qmail 16952 invoked from network); 9 May 1998 02:56:03 -0000 Received: from twinlark.arctic.org (204.62.130.91) by taz.hyperreal.org with SMTP; 9 May 1998 02:56:03 -0000 Received: (qmail 1232 invoked by uid 500); 9 May 1998 02:56:02 -0000 Date: Fri, 8 May 1998 19:56:02 -0700 (PDT) From: Dean Gaudet To: new-httpd@apache.org Subject: Re: when the tools work, they kick ass. In-Reply-To: <3.0.3.32.19980508195350.007ef730@hyperreal.org> Message-ID: X-Comment: Visit http://www.arctic.org/~dgaudet/legal for information regarding copyright and disclaimer. MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: new-httpd-owner@apache.org Precedence: bulk Reply-To: new-httpd@apache.org But still, if we were to change directory in the CGI child after the fork() it would simplify things immensely. Immensely! Like... all of those ap_chdir_file() calls could be removed. And it'd go faster to boot! (removing ap_chdir_file from mod_include is worth a nice gain, I don't remember the number now, but it's on the order of 10% to 20%.) 1.3.1... er, well I'll do it in NSPR for unix and folks can port it to win32 later. Dean On Fri, 8 May 1998, Brian Behlendorf wrote: > > So I was going through the discussions on the chdir/NT issue, and I saw > Dean question the chdir in handle_include in mod_include. I was hoping > it'd be as easy as just making it #ifndef WIN32 like all the others. So I > went to the CVS history viewer at > > _include.c> > > And found the patch that introduced this change. In looking at that patch, > I saw that it was done based on a bug report in the bug database, #742. I > went over to the bug database and saw what it was on, why it was there, and > decided it probably needed to stay there but could be safely elided on > WIN32. Anyways, it felt good to realize that we had a system where the > reasoning behind almost any part of the code could be researched relatively > easily; no more "why the hell did we add that?" "I dunno, go read the > archives". Yeehaw. > > Brian > > > --=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-- > pure chewing satisfaction brian@apache.org > brian@hyperreal.org >