httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Roy T. Fielding" <>
Subject mod_fcgid
Date Tue, 17 Mar 2009 23:09:39 GMT
On Mar 17, 2009, at 3:44 PM, Chris Darroch wrote:

> Roy T. Fielding wrote:
>> Done.  I set all of Ryan Pan's commits to svn:author=pqf
>> (for consistency), loaded the whole thing under httpd/mod_fcgid
>> and fixed the eol-style to native.
>> Please relicense the directory and files first before making any
>> other changes.
>   Thanks!  I think the relicensing is all done; anyone who wants
> to check for errors will get a thank-you.

I think most of the stuff in NOTICE is documentation that belongs
in README.  The only things we put in NOTICE files are copyright
or attribution lines required by the original copyright owners.

>> After that, we should remove the generated artifacts
>> and consider moving the subdirectory to a module dir under the
>> server trunk.
>   Which artifacts are you concerned about?  I'm not 100% sure
> what you'd like done here.

Makefile and .deps should be generated files.

>   I'm +1 on the idea of moving toward inclusion in httpd trunk
> as a module, at least as a longer-term goal.  My thoughts were:
> - Start by branching httpd/mod_fcgid/branches/2.x/mod_fcgid based on
>  the current relicensed code, just in case an "emergency" turns up
>  (security bug?) and we'd like to do a release based on the existing
>  code; most likely, we won't use this going forward.

We have not done any releases of this code, so we are not responsible
for "fixing" releases that might already exist.  In any case, a branch
can be created at any time by copying a specific revision number; there
is no reason to create it before the branch is needed.

> - Fix the whitespace; currently if you set your tab stops to 4 chars
>  it looks "right" and close to the httpd style, so it's just a
>  conversion from tabs to spaces that remains.  I've got a reformatted
>  version ready to commit.

Go ahead.  It might be worthwhile to create a tag after the whitespace
changes (same as a branch but without implying later mods) that we can
use for easier functional diffs in the future.

> - Before we can add to httpd trunk, we need to look at the directive
>  names.  There are a large number of config directives with names
>  that don't imply FastCGI and in some cases are a little mysterious.
>  I don't think we can drop 32 new directives into httpd with generic
>  names like "SocketPath" and "PassHeader".
>  I have an old patch I can dust off to rename these; we should choose
>  a prefix first, though.  "FCGI" or "FastCGI"?  The latter might
>  conflict with mod_fastcgi, but then, why would anyone run both at
>  the same time?  Still, perhaps "FCGI" is best?

FCGI_ is good.  What about docs?


View raw message