Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:29507
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain m.gmane.org designates 80.91.229.2 as permitted sender)
To: internals@lists.php.net
Date:  Fri, 18 May 2007 01:26:50 +0200
Lines: 292
Message-ID: <f2iobs$klm$1@sea.gmane.org>
References:  <f2figb$ke4$1@sea.gmane.org>
Reply-To:  "Carsten Wiedmann" <carsten_sttgt@gmx.de>
Mime-Version:  1.0
Content-Type:  multipart/mixed;
	boundary="----=_NextPart_000_0214_01C798EB.9BB81EE0"
Sender: news <news@sea.gmane.org>
Subject:  Re: [PHP-DEV]  comment to Bug  #41408
From: carsten_sttgt@gmx.de ("Carsten Wiedmann")

------=_NextPart_000_0214_01C798EB.9BB81EE0
Content-Type: text/plain;
	format=flowed;
	charset="iso-8859-15";
	reply-type=response
Content-Transfer-Encoding: 7bit

Hello list,

> For a correct solution you should allways use forwardslashs
> in the Apache config files, also on Windows:
> > PHPIniDir "C:/Program Files/PHP"

Just to clarify that. The Apache manual [1] is very clear about that:
| For example, in a default install, Apache resides at ...
| "c:/Program Files/Apache Group/Apache2" in the Windows filesystem.
| (Note that forward slashes should always be used as the path separator
| in Apache, even for Windows.)


After a review of the file "PHPInstallerScripts60.vbs" in the cvs, I found 
other problems. The cgi setup can't work:
- wrong target in the Action directive
- no Directory block for the ScriptAlias (this should also changed in the 
documentation)

I find problematic also, that you change the "mime.types" instead of using 
AddType. From the Apache manual [2]/[3] :
| It is recommended that new MIME types be added using the AddType
| directive rather than changing the TypesConfig file.
|
| You should not edit the mime.types file, because it may be replaced
| when you upgrade your server.


IMHO you should respect the Apache manual. And in addition not confusing the 
user. Apache also uses only forwardslashs in its installer and default 
config file. And with the current PHP installer you have single backslashs 
(module) or double backslashs (cgi).


I've appended a diff to this mail, which change/correct the behavior of the 
installer. It's a diff against revision 1.4 of the file 
"PHPInstallerScripts60.vbs" and replace the sub procedure configApache. With 
a change of the line 'strVersion = "php6"' to 'strVersion = "php5"', this 
should also work in the file "PHPInstallerScripts52.vbs". Maybe someone want 
look at this.


Regards,
Carsten

[1] http://httpd.apache.org/docs/2.2/en/sections.html#file-and-web
[2] http://httpd.apache.org/docs/2.2/en/mod/mod_mime.html#addtype
[3] http://httpd.apache.org/docs/2.2/en/mod/mod_mime.html#typesconfig

------=_NextPart_000_0214_01C798EB.9BB81EE0--