comment to Bug #41408

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