Home > Archive > PHP PEAR Questions and Answers > November 2007 > RE: [PEAR-QA] testing tarball of PEAR 1.7.0RC1 (not 1.7.0) uploaded
You are viewing an archived Text-only version of the thread.
To view this thread in it's original format and/or if you want to reply to
this thread please [click here]
| Author |
RE: [PEAR-QA] testing tarball of PEAR 1.7.0RC1 (not 1.7.0) uploaded
|
|
| Jorrit Schippers - nCode 2007-11-30, 8:03 am |
| > 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
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.
Has someone found time to have a look at my find() patch? I'm really
interested in your opinion at this.
Jorrit
| |
| Greg Beaver 2007-11-30, 7:02 pm |
| Jorrit Schippers - nCode wrote:
>
> 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
|
|
|
|
|