# comment to Bug #41408

- Thread: https://externals.io/message/29483
- Original archive: https://news-web.php.net/php.internals/29483
- Started by Carsten Wiedmann on 2007-05-16T18:28:25+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #29483 — Carsten Wiedmann — 2007-05-16T18:28:25+00:00

URL: https://externals.io/message/29483#29483

Sorry for writing here, but can't comment a closed bug.

> PHPIniDir "C:\Program Files\PHP\"

Quoting (all) filesystem paths is always a good idea. For a correct solution
you should allways use forwardslashs in the Apache config files, also on
Windows:
| PHPIniDir "C:/Program Files/PHP"

Regards,
Carsten

---

## #29507 — Carsten Wiedmann — 2007-05-17T23:26:50+00:00

URL: https://externals.io/message/29483#29507 · In reply to: #29483

Hello list,

> For a correct solution you should allways use forwardslashs
> in the Apache config files, also on Windows:
> > […]

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