Re: testing tarball of PEAR 1.7.0RC1 (not 1.7.0) uploaded

Jorrit Schippers - nCode wrote: >> Hi all, >> >> I'm not comfortable with the volume of serious problems we've been >> having in the testing due to "minor" bugfixes in core >> elements like the >> System class. As such, I've downgraded to RC1, and uploaded a testing >> tarball. >> > > While working on the System class (I submitted the find() bug) I found quite > some Windows/space path incompatibilities, especially because most methods > accept command line like arguments. It uses very basic space splitting (as > mentioned in the 1.7.0 thread), wouldn't it be more sensible to use > something like GetOpt here? Besides that, the method to get the TMP If you pass an array to System methods, it works with spaces in paths. In other words, do the getopt prior to calling System. > directory on windows should also use realpath, as at least in my case the > TMP environment variables are in the 8.3 format (with ~1 etc). > > The System class is mean to provide the functionality of unix tools in a > platform independent way, especially on Windows, where these tools are not > available. I think it is strange that precisely on Windows the class does > not work at all. Clearly, Windows has not been used to test or develop this > class. Au contraire - please note that most PEAR releases and tests were done on windows by me. You've found some functionality that no one has been using for the past 2 or 3 years, it's not a case of no testing at all. > Has someone found time to have a look at my find() patch? I'm really > interested in your opinion at this. Not yet Greg