Re: svn commit: r797647 - /httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c

[Jeff Trawick <tr...@gmail.com>]

On Fri, Jul 24, 2009 at 4:54 PM, <wr...@apache.org> wrote:

> Author: wrowe
> Date: Fri Jul 24 20:54:46 2009
> New Revision: 797647
>
> URL: http://svn.apache.org/viewvc?rev=797647&view=rev
> Log:
> use our apr specific methods of merging this server config table and array
>
> Modified:
>    httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c
>
> Modified: httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c
> URL:
> http://svn.apache.org/viewvc/httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c?rev=797647&r1=797646&r2=797647&view=diff
>
> ==============================================================================
> --- httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c (original)
> +++ httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c Fri Jul 24 20:54:46 2009
> @@ -99,6 +100,71 @@
>     return APR_SUCCESS;
>  }
>
> +
> +/* This should be proposed as a stand-alone improvement to the httpd
> module,
> + * either in the arch/ platform-specific modules or util_script.c from
> whence
> + * it came.
> + */
> +static void default_proc_env(apr_table_t *e)


do you have any code yet to call this function?

Re: svn commit: r797647 - /httpd/mod_fcgid/trunk/mod_fcgid/mod_fcgid.c

["William A. Rowe, Jr." <wr...@rowe-clan.net>]

Jeff Trawick wrote:
> 
> do you have any code yet to call this function?

Once again, compiler warnings are our friend.  Yes, this needs to be
invoked, but it's a NTP, and the code I tossed was a new direction
I wanted to go in, which would pick up all the PassEnv and other
overrides, and set these all aside in a way that it would reinvoke
the program again identically if a worker is lost.  I then looked
at the possibility of using httpd's API but that wasn't suitable either.

So... back to the drawing board to flesh out this table before we
go and invoke apr_proc_create.  Thanks for drawing it to my attention.