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
 [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: Mon Dec 09 13:01:26 2024 UTC