httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe Jr." <>
Subject Re: CHANGES-FCGID is incorrect
Date Tue, 10 Apr 2012 04:30:22 GMT
I have corrected the 1236319 entry which should not have been recorded yet.

The patch does have value to a limited number of applications.  I even went
as far as to put caviats in the docs, and a see-docs note to the directive
cmd commentary.  I hope it dissuades the casual user from throwing it on there
unless they know it won't corrupt their app and know it solves their bug.

Please reconsider your veto.  I agree that it -should not- occur in any
real world scenario.  But there are unreal scenarios of server configs
and third party modules which spend minutes, not seconds, tearing down on
shutdown.  Those are the bugs.  But users just want some help and I think
this patch is some help in rare cases.  Disabled by default and won't be
used by much of anyone, we hope.  I'll revert (yet again) if you really
want to make a case that we should never support this, even as a workaround.

I presume you meant 51560 borrows the 51020 patch.  Please recheck, we might
be working a record for the number of transpositions in one bug resolution.

On 3/2/2012 6:57 PM, William A. Rowe Jr. wrote:
> Gregg,
> I reached the same conclusion Wednesday when I stumbled on a dirty mod_fcgid
> checkout here.
> I'll review your comments and then determine how to proceed (commit the feature
> which could be useful, port the feature to apr, or concur it isn't needed.)
> Because of the way the Windows Service Control manager works, it may still have
> value.
> On 3/2/2012 4:34 PM, Gregg Smith wrote:
>> Hi,
>> First,
>> The patch in PR 51078 was never commited. However, I am -1 on committing this right
>> because I believe the root cause of this orphan described in PR 51078 is what has
>> fixed now by the commit of PR 50309. Without the PR 50309 fix I have run into this
>> after stopping httpd and having to go into processes and kill it manually.  It's
>> problem with graceful restarts, the fcgid process never got killed and it would stop
>> httpd parent because it was holding onto the port and logs. PR 50309 most likely
will fix
>> PR 48949 as well, since Mario describes the same issue on Ubuntu and the same patch
>> for him on it.
>> Second,
>> The patch applied is from PR 51020, so you have killed 2 PRs with the one stone,
50120 &
>> 51520
>> The latter's a nit pick I know :)
>> Gregg

View raw message