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

Bug #8836 path case issues in package.xml
Submitted: 2006-10-03 09:54 UTC
From: hholzgra Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version CVS)
PHP Version: 5.2.0 RC4 OS: linux
Roadmaps: (Not assigned)    

 [2006-10-03 09:54 UTC] hholzgra (Hartmut Holzgraefe)
Description: ------------ "pear install package.xml" works on Mac OS/X (and probably on Windows) but fails on Linux due to its case sensitive file system, the following change to package.xml fixes this ('s' vs. 'S' in "Whitespace") Index: package.xml =================================================================== RCS file: /repository/pear/PHP_CodeSniffer/package.xml,v retrieving revision 1.7 diff -u -r1.7 package.xml --- package.xml 1 Oct 2006 05:45:52 -0000 1.7 +++ package.xml 3 Oct 2006 09:51:34 -0000 @@ -184,7 +184,7 @@ <file name=""/> <file name="MultipleStatementUnitTest.php"/> </dir> - <dir name="WhiteSpace"> + <dir name="Whitespace"> <file name=""/> <file name="ScopeClosingBraceUnitTest.php"/> <file name=""/> Test script: --------------- cd pear/PHP_CodeSniffer pear install -f package.xml Expected result: ---------------- success Actual result: -------------- ERROR: file ./CodeSniffer/Standards/PEAR/Tests/WhiteSpace/ does not exist


 [2006-10-03 10:07 UTC] squiz at php dot net (Greg Sherwood)
Thank you for taking the time to report a problem with the package. This problem may have been already fixed by a previous change that is in the CVS of the package. Please log into CVS with: cvs -d login and check out the CVS repository of this package and upgrade cvs -d co pear/PHP_CodeSniffer pear upgrade pear/PHP_CodeSniffer/package2.xml or pear upgrade pear/PHP_CodeSniffer/package.xml If you are able to reproduce the bug with the latest CVS, please change the status back to "Open". Again, thank you for your continued support of PEAR.
 [2006-10-03 11:09 UTC] squiz at php dot net (Greg Sherwood)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on by the end of next Sunday (CET). If this was a problem with the 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. Wrong status selected. Actually fixed in CVS by renaming the Tests/Whitespace directory to Tests/WhiteSpace to match Sniffs dir and package.xml file.