I agree. Frank, can you revert and then we can either fix it ourselves or ask that guy to come up with a new patch.

Andi

At 04:11 PM 2/10/2005 -0800, Frank M. Kromann wrote:
I Suggest that we revert the patch, as it for sure breaks how IIS works
with CGI. I have not thested with ISAPI or Apache, but I could do that.

We can then add a patch that checks for SCRIPT_FILENAME and then falls
back to SCRIPT_NAME if the first one was undefined.

It seams to me that the patch does it the other way arround.

- Frank


> > Bug report suggested the other way around: CGI not being usable > under IIS _without_ the patch. :) > > (Yes, I didn't test it, I don't have IIS..) > > http://bugs.php.net/bug.php?id=28227&edit=1 > > --Jani > > On Tue, 8 Feb 2005, Frank M. Kromann wrote: > > > Hello Everyone, > > > > The patch on sapi/cgi_main.c from 1.259 to 1.260 (HEAD) breaks the cgi > > under IIS. No after this patch the server returns file not found on all > > requests. > > > > - Frank > > > > >

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php



Reply via email to