php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #41789 Incorrect Modification to httpd.conf
Submitted: 2007-06-24 12:21 UTC Modified: 2008-08-22 18:14 UTC
From: philjcrawford at gmail dot com Assigned: jmertic (profile)
Status: Closed Package: Windows Installer
PHP Version: 5.2.3 OS: Windows XP SP2
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: philjcrawford at gmail dot com
New email:
PHP Version: OS:

 

 [2007-06-24 12:21 UTC] philjcrawford at gmail dot com
Description:
------------
When installing PHP 5.2.3 for Apache 2.2, the following is appended to the httpd.conf file:
#BEGIN PHP INSTALLER EDITS - REMOVE ONLY ON UNINSTALL
PHPIniDir "C:\\Program Files\\PHP\\"
LoadModule php5_module "C:\\Program Files\\PHP\\php5apache2_2.dll"
#END PHP INSTALLER EDITS - REMOVE ONLY ON UNINSTALL

I had just managed to enable PHP 5.2.1 using the following, which the 5.2.3 installer commented out:
#BEGIN PHP INSTALLER EDITS - REMOVE ONLY ON UNINSTALL
#PHPIniDir "C:/Program Files/PHP/"
#LoadModule php5_module "C:/Program Files/PHP/php5apache2_2.dll"
#END PHP INSTALLER EDITS - REMOVE ONLY ON UNINSTALL

Note the "/" instead of "\\" in the path.
I don't need help but thought this should be fixed for subsequent releases.


Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2007-06-24 16:31 UTC] sniper@php.net
Assigned to the installer guru.
 [2007-06-25 01:18 UTC] jmertic@php.net
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.

Actually, either way works fine ( I used double backslashes on all my PHP/Apache/Windows installs ), but I did change it to the more conventional forward slashes in CVS.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Dec 26 20:01:29 2024 UTC