Greg Ames <[EMAIL PROTECTED]> writes:

> Jeff Trawick wrote:
> > 
> > With this newer patch, the behavior w.r.t. pid file cleanup and
> > SIGTERM messages should be consistent between prefork and worker.
> > Neither pid file cleanup nor the SIGTERM message is issued.
> 
> Personally, I like the idea of pid file cleanup when bad stuff happens. 
> It's probably not done everywhere it needs to be done within worker,  It
> isn't consistant between MPMs, as you point out, and isn't consistant
> with 1.3 behavior.  But so what?  Cleaning up the pid file just seems
> like the right way to go, IMO.

I think that if you really want the pid file cleaned up, then a
cleanup should be registered (or maybe the file left open, with
the APR_DELONCLOSE flag?  dunno).  If you really want it removed, try
harder than just removing it for this path.

The absense of httpd with presence of httpd.pid means something hit
the fan, which is what this type of exit is.

but the bottom line:

However the code looks w.r.t. to httpd.pid when I commit, I'll still
buy coffee Monday if you change it :)

-- 
Jeff Trawick | [EMAIL PROTECTED] | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...

Reply via email to