Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 3.7.2
Bug #19428 PHPCS Report "hgblame" doesn't support windows paths
Submitted: 2012-05-22 01:04 UTC
From: thinice Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.3.4)
PHP Version: 5.3.13 OS: Windows 7
Roadmaps: (Not assigned)    
Subscription  


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 6 - 3 = ?

 
 [2012-05-22 01:04 UTC] thinice (Justin Rovang)
Description: ------------ L91 and L97 of CodeSniffer/Reports/Hgblame.php should use DIRECTORY_SEPARATOR

Comments

 [2012-05-22 01:10 UTC] thinice (Justin Rovang)
And L96
 [2012-05-22 01:12 UTC] thinice (Justin Rovang)
Added patch Windows.Paths.diff.
 [2012-05-22 01:26 UTC] thinice (Justin Rovang)
Pull request with patch applied: https://github.com/pear/PHP_CodeSniffer/pull/5
 [2012-06-05 06:09 UTC] squiz (Greg Sherwood)
-Status: Open +Status: Closed -Assigned To: +Assigned To: squiz
Fix committed to git repo. Thanks for the patch: https://github.com/squizlabs/PHP_CodeSniffer/commit/b178b4fe271a77da60899fa3fd b0f21d50ae8474