You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by "Gregg L. Smith" <li...@glewis.com> on 2010/07/31 22:11:59 UTC

Re: svn commit: r981060 - /httpd/httpd/branches/2.2.x/STATUS

Hello Rainer,

I am +1 for this, as a matter of fact I lobbied for this 10 months ago.

http://marc.info/?l=apache-httpd-dev&m=125421199606305&w=2

If it is going to be done however, please do it completely. Attached is
the rest of what is needed. [mod_proxy_scgi_project.patch]

mod_proxy_scgi.dsp <- must be there for the module to really build in
IDE, otherwise if you add to Apache.dsw and do not include the project
file, vc++ is going to complain and never build the module. Instead, it
will just show the project as unloaded I think. I do see you covered
installwinconf.awk which was overlooked when adding this module to trunk.

Big thanks, this is one less patch I need to use and maintain.

Gregg


rjung@apache.org wrote:
> Author: rjung
> Date: Sat Jul 31 13:49:52 2010
> New Revision: 981060
> 
> URL: http://svn.apache.org/viewvc?rev=981060&view=rev
> Log:
> Add proposal.
> 
> Modified:
>     httpd/httpd/branches/2.2.x/STATUS
> 
> Modified: httpd/httpd/branches/2.2.x/STATUS
> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=981060&r1=981059&r2=981060&view=diff
> ==============================================================================
> --- httpd/httpd/branches/2.2.x/STATUS (original)
> +++ httpd/httpd/branches/2.2.x/STATUS Sat Jul 31 13:49:52 2010
> @@ -250,6 +250,11 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
>       2.2 patch: trunk patch works with offset
>       +1: sf, rpluem
>  
> +   * mod_proxy_scgi: Add module to build on Windows.
> +     It already gets build for other platforms and for Windows in trunk.
> +     2.2 patch: http://people.apache.org/~rjung/patches/mod_proxy_scgi-win-build-2_2.patch
> +     +1: rjung
> +
>  PATCHES/ISSUES THAT ARE STALLED
>  
>    * core: Support wildcards in both the directory and file components of
> 
> 
> 




Re: svn commit: r981060 - /httpd/httpd/branches/2.2.x/STATUS

Posted by Rainer Jung <ra...@kippdata.de>.
Hi Gregg,

On 31.07.2010 22:11, Gregg L. Smith wrote:
> Hello Rainer,
>
> I am +1 for this, as a matter of fact I lobbied for this 10 months ago.
>
> http://marc.info/?l=apache-httpd-dev&m=125421199606305&w=2
>
> If it is going to be done however, please do it completely. Attached is
> the rest of what is needed. [mod_proxy_scgi_project.patch]
>
> mod_proxy_scgi.dsp <- must be there for the module to really build in
> IDE, otherwise if you add to Apache.dsw and do not include the project
> file, vc++ is going to complain and never build the module. Instead, it
> will just show the project as unloaded I think. I do see you covered
> installwinconf.awk which was overlooked when adding this module to trunk.
>
> Big thanks, this is one less patch I need to use and maintain.

Thanks to you and sorry for the long response time of 10 months. My 
proposal actually was triggered by your original message, although 
somewhat late.

And yes, I forgot mod_proxy_scgi.dsp. I updated the patch to contain it.

Regards,

Rainer

Re: svn commit: r981060 - /httpd/httpd/branches/2.2.x/STATUS

Posted by "Gregg L. Smith" <li...@glewis.com>.
I see this statement I made to be incorrect now since you added this today.

Regards,

Gregg

Gregg L. Smith wrote:
> installwinconf.awk which was overlooked when adding this module to trunk.