Bug #70917 [Ver->Csd]: install-pear-nozlib.phar missing (regression to RC6)
| From: | ab@php.net | Date: | Sun, 15 Nov 2015 18:04:02 +0000 |
| Subject: | Bug #70917 [Ver->Csd]: install-pear-nozlib.phar missing (regression to RC6) | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-197271@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=70917&edit=1
ID: 70917
Updated by: ab@php.net
Reported by: bug@php.net
Summary: install-pear-nozlib.phar missing (regression to RC6)
-Status: Verified
+Status: Closed
Type: Bug
Package: *General Issues
Operating System: Linux
PHP Version: 7.0.0RC7
Assigned To: ab
Block user comment: N
Private report: N
New Comment:
Automatic comment on behalf of ab
Revision: http://git.php.net/?p=php-src.git;a=commit;h=7819fef345f2ec80c4fea804b350e229af589990
Log: Fixed bug #70917 install-pear-nozlib.phar missing
Previous Comments:
------------------------------------------------------------------------
[2015-11-15 16:32:54] ab@php.net
Unfortunately I've no output anymore. But it seems to be nothing else than the failed host
resolution, fe also bug #70832 around the tagging date. So most likely my bad not seeing the
download error (and the absence of that phar haven't also affect the end tarball size,
unfortunately).
Ferenc, do you think we could add this or similar to the script?
diff --git a/makedist b/makedist
index 5b28dd8..48c6312 100755
--- a/makedist
+++ b/makedist
@@ -119,6 +119,11 @@ sed -i 's,^#ifndef YYTOKENTYPE,#include "zend.h"\n#ifndef
YYTOKENTYPE,g' $MY_OLD
$ECHO_N "makedist: Attempting to download PEAR's phar archive"
if test ! -x wget; then
wget https://pear.php.net/install-pear-nozlib.phar
-nd -P pear/
+ if [ "x$?" != "x0" ]
+ then
+ $ECHO_N "Pear download failed";
+ exit 7
+ fi
else
$ECHO_N "Missing wget binary needed for pear download";
exit 7
Thanks.
------------------------------------------------------------------------
[2015-11-15 16:03:38] tyrael@php.net
it seems that the RC7 packages are missing the downloaded install-pear-nozlib.phar.
I can't reproduce the dns issue provided in the report but I guess that was a self-inflicted to
point out the lack of install-pear-nozlib.phar in the tarball.
Anatol, do you happen to have the makedist output for RC7? Would be nice knowing why did the pear
download fail or fix it if there is some persistent problem so the next version can have
install-pear-nozlib.phar in the tarball.
------------------------------------------------------------------------
[2015-11-15 13:35:05] laruence@php.net
hmm, I read it wrongly, seems a network issue?
------------------------------------------------------------------------
[2015-11-15 13:34:14] laruence@php.net
@tyrael could you please have a look ?
------------------------------------------------------------------------
[2015-11-14 21:22:47] bug@php.net
Description:
------------
Tarball is not self-contained:
$ make install
:
Installing man pages: /tmp/local/php/man/man1/
page: phpize.1
page: php-config.1
Installing PEAR environment: /tmp/local/lib/php/
--2015-11-14 22:12:19-- https://pear.php.net/install-pear-nozlib.phar
Resolving pear.php.net (pear.php.net)... failed: Name or service not known.
wget: unable to resolve host address `pear.php.net'
make: *** [install-pear] Error 4
Regression: #41143 (2007)
Expected result:
----------------
Add install-pear-nozlib.phar to the tarball.
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=70917&edit=1