Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.10.13

Bug #2396 Error Parsing path causes pear.bat to be renamed
Submitted: 2004-09-25 10:01 UTC
From: wormus Assigned: cellog
Status: Closed Package: PEAR
PHP Version: Irrelevant OS: Windows
Roadmaps: (Not assigned)    

 [2004-09-25 10:01 UTC] wormus
Description: ------------ This looks like a duplicate of 1156 which was closed a while ago, but the change doesn't appear in go-pear.php which is distributed with PHP itself. So the issue still remains. There is another possible bug having to do with a trailing slash, which the original bug doesn't address. When a user has c:\php\ (or the path to their php dist) in their system path and run go-pear.bat the installation process will install pear but then rename pear.bat to pear.bat_old before leaving. The issue of the Case sensitivity seems to have been taken care of, but there is another issue and that is even if the case matches you may or may not have a trailing slash at the end of your PATH environment var, which will also not match. When it finds the current pear.bat in the path it assumes that it is a previously installed version and renames it. Leaving the user without a pear.bat Note: I have not tried this with the case sensitivity patch, so it could be just the case sensitivity issue as well. Reproduce code: --------------- Change your PATH environment variable to c:\php\ (note the trailing slash) and run the go-pear.bat Expected result: ---------------- I expect that it will install PEAR Actual result: -------------- It installs PEAR and then renames pear.bat to pear.bat_old, leaving me with no way to run PEAR.


 [2004-10-25 00:19 UTC] pear-dev at mocsnik dot hu
confirmed - worked the same way for me (w2k), even with the latest go-pear $Id: go-pear,v 1.69 2004/10/22 05:34:31 cellog Exp $
 [2004-10-25 04:09 UTC] cellog
I've just committed a new version of go-pear that contains several fixes including this one. It works on my system now, could you test it on yours?
 [2004-10-25 04:53 UTC] cellog
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on In case this was a website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.