Re: cvs commit: modperl-2.0/src/modules/perl modperl_env.c

Stas Bekman wrote:
> Geoffrey Young wrote:
>
> > >  move modperl_env_default_populate to where it was before (after
> > >  modperl_env_table_populate), so that GATEWAY_INTERFACE will be
> > >  CGI-Perl/1.1 and not CGI/1.1.
> >
> >
> >
> > whoops - I probably moved that at some point and didn't realize it.
>
>
> I wonder why modules/cgi.t wasn't failing for you. I haven't tried to 
> run the patch, assuming that it worked for you ;)
>
> > once we get the other issue sorted out, I'll specifically add a test for
> > GATEWAY_INTERFACE someplace.
>
>
> I think it's there already just fine, all you need is this patch:
>
> Index: t/response/TestModules/cgi.pm
> ===================================================================
> RCS file: /home/cvs/modperl-2.0/t/response/TestModules/cgi.pm,v
> retrieving revision 1.11
> diff -u -r1.11 cgi.pm
> --- t/response/TestModules/cgi.pm       22 May 2002 18:53:33 -0000      
> 1.11
> +++ t/response/TestModules/cgi.pm       14 Feb 2004 04:51:53 -0000
> @@ -55,4 +55,4 @@
>  1;
>  __END__
>  SetHandler perl-script
> -
> +PerlOptions -SetupEnv

OK, per our irc discussion we resolve this thread by dropping 
GATEWAY_INTERFACE on the floor, as it's totally unnecessary.

__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@xxxxxxxxxx http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com