httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <>
Subject Re: CHANGES-FCGID is incorrect
Date Tue, 10 Apr 2012 16:11:28 GMT
On Tue, Apr 10, 2012 at 12:35 AM, William A. Rowe Jr.
<> wrote:
> On 4/9/2012 11:30 PM, William A. Rowe Jr. wrote:
>> 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.
> Gregg, and Jeff and company, if this is reviewed and everyone is happy with
> an end result, I'd like to ensure 2.4.2 compatibility and get tagging.


My gut feel is that the feature will help in some real world
circumstances, but I need to stare at Windows-specific code for a
while to really understand the problem (where third-party modules can
intervene, where a timeout comes into play, etc.).  At first glance it
seems to simply be the case when apr_proc_kill(SIGKILL) doesn't work
for whatever reason (third-party code, but not the usual code we worry

> We seem well overdue for a release.
> There is a group of issues with spaces in command lines, spaces in file names
> that need to be resolved for the typical windows scenario (and obviously also
> unix, just more unusual there).  I think we are best off calling 2.3.7 about
> baked right now, tagging, reviewing and releasing, and calling the
> flavor which fixes these command/file string argument quirks.


View raw message