Re: phar and e_strict
| From: | Greg Beaver | Date: | Tue, 04 Oct 2005 04:27:31 +0000 |
| Subject: | Re: phar and e_strict | ||
| References: | 1 2 3 | Groups: | php.internals php.pear.dev |
| Request: | Send a blank email to pear-dev+get-40081@lists.php.net to get a copy of this message | ||
Curt Zirzow wrote:
> On Sat, 01 Oct 2005 08:21:57 -0700, Greg Beaver wrote:
>
>>I've been running the pear command using PHP 5.1rc1 ever since it came
>>out, and it works just fine. This is a problem specific to the
>>install-pear.phar that in fact did not happen just 2 weeks ago - perhaps
>>something has changed in PHP 5.1 itself?
>
>
> A little side track on the E_STRICT issue... I ran into two issues while
> installing php 5.1-branch.
>
> 1. zlib is required now with php, and by default php isn't installed
> with zlib nor does configure catch this, so, while make install is
> running and when pear is being installed make dies leaving a bad
> installation.
I could eliminate this requirement, at the expense of a larger .phar
file (3.3MB instead of 640k).
> I have have a patch for configure.in that is ready that will add a check
> for zlib in the configure process to avoid the make install bailing.
>
> 2. I'm not sure if the release of php5.1 is going to include the
> install-pear.phar file, but if I don't have that file within the pear/
> directory, make install also requires that the system has wget
> installed, causing a fail in make install as well.
>
> I can write a patch for configure.in to check for wget (or fetch on *bsd
> machines), if the .phar file isn't going to be included with a release.
Better (I discussed this with Andi, but haven't had time to do anything
fruitful yet) is to write a short PHP script that would download it.
Just now, it occurs to me that by changing the wget call in
Makefile.frag from:
wget http://pear.php.net/install-pear.phar -nd
-P $(srcdir);
to:
@$(top_builddir)/sapi/cli/php -dallow_url_fopen=1 -r "$a =
file_get_contents('http://pear.php.net/install-pear.phar');file_put_contents($_SERVER['argv'][1]
. '/install-pear.phar',$a);" $(srcdir)
this might be our cross-platform ticket. Could one of you kind folks
try this, and commit if it works?
Thanks,
Greg