Closed (fixed)
Project:
Drush
Component:
Base system (internal API)
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
8 Nov 2010 at 22:19 UTC
Updated:
3 Dec 2010 at 22:10 UTC
I introduced a mad usage of DIRECTORY_SEPARATOR in a patch and it seems it was spread to other patches. I've read about it as the fact was that '/' seemed to work.
Here's a reference: http://alanhogan.com/tips/php/directory-separator-not-necessary
In summary we only need DIRECTORY_SEPARATOR to explode() paths, trim(), etc. Not needed to construct paths.
I'll do a patch after other issues with big changes are fixed, to not collide.
Comments
Comment #1
jonhattanJust commited some changes. There're some usage of explode('/'...) in environment.inc and sitealias.inc that I did not change as there are no bug reports.
http://drupal.org/cvs?commit=453470