Description: ------------ In PEAR_PackageFileManager_File#getFileList(), if 'packagedirectory' terminates with '/'(directory separator), result of file 'path' will be wrong. File : /your/package/directory/foo/bar.php 'packagedirectory' : /your/package/directory/ (it'll equal dirname(__FILE__) in many cases.) This will be: /your/package/directory/oo/bar.php I solve this problem. I switch the character index (= cut point of substr()) according to whether 'packagedirectory' terminate with '/' or not. See details my patch which was posted with this report.

I'm msakamoto. Sorry, I missed patch file attachement. See B12023_p01 patch.

Oh, no... sorry. My patch isn't "cvs diff -u". My patch is about local files from PEAR stable release. (PEAR/PackageFileManager/File.php)

Whilst I see what you're saying, I can't immediately reproduce the problem on Linux - adding a trailing / to packagedirectory doesn't seem to cause any harm. I wonder if it's a Windows-specific thing somehow?

timj, thank you for your comment. Yes, I tried latest version(PEAR_PackageFileManager-1.6.2) on my linux box (PHP4.4.7), but I couldn't reproduce this problem, too. By more trial, I found that the reason is difference of realpath() behaviour between PHP4 and PHP5 when realpath() called with trailing DIRECTORY_SEPARATOR(DS) on Windows. (I tried PHP4.4.7 and PHP5.2.4 on WinXP SP2) Imagine there's "C:\temp" direcotry. If you calls realpath() without trailing DS: > php -r "echo realpath('c:\\temp');" This will result "C:\\temp" on both PHP4 and PHP5. But if you calls realpath() with trailing DS: > php -r "echo realpath('c:\\temp\\');" on PHP4: "c:\\temp" BUT on PHP5: "c:\\temp\\" !!!!!! I couldn't believe this result ... BUT, it is the FACT.

I can confirm the reported behavior on Windows. A simpler patch is attached.

quipo (Lorenzo Alberton), Thank you for your reproduction confirmation and simpler patch. I confirmed that your patch works correctly on PHP4(WinXPSP2, Linux box) and PHP5(WinXPSP2). Thank you!

I can confirm this bug on windows vista

I can confirm it on windows, and would like to know when a new release might be released? Here is a testcase which will show the problem. Use both the directories and the bug_cutsoffirstletterofrootdirectories.phpt. http://svn.intraface.dk/intrafacepublic/3Party/PEAR/PEAR_PackageFileManager2/tests/

Like Laurent pointed out in PEAR dev,this only affects PHP 5.2.4 it seems on Windows, http://bugs.php.net/bug.php?id=43248 but the patch Lorenzo pro vided is still valid

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.