[dev] [cvs] commit: horde test.php
Chuck Hagenbuch
chuck at horde.org
Fri Jul 27 15:05:35 UTC 2007
Quoting Jan Schneider <jan at horde.org>:
> No, we only have our own version of File/CSV.php, we still need
> File.php from the File package.
As far as I can tell, the only thing that we use File.php for is for
the FILE_MODE_* constants. If that's it, let's just have our own
constants in CSV.php and remove the dependency. Having a forked
File_CSV but requiring people to install the File package leads to
problems with the order of includes sometimes (and if it's happened to
me, I'm sure it's tripped up others. :)
-chuck
More information about the dev
mailing list