#25209 [Opn->Asn]: pear build fails unexpectedly
| From: | arnaud@php.net | Date: | Sun, 24 Aug 2003 11:39:53 +0000 |
| Subject: | #25209 [Opn->Asn]: pear build fails unexpectedly | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-20450@lists.php.net to get a copy of this message | ||
ID: 25209
Updated by: arnaud@php.net
Reported By: eru@php.net
-Status: Open
+Status: Assigned
Bug Type: PEAR related
Operating System: Linux
PHP Version: Irrelevant
-Assigned To:
+Assigned To: cellog
Previous Comments:
------------------------------------------------------------------------
[2003-08-22 09:25:51] eru@php.net
Description:
------------
pear build/install will fail with almost every PECL out there, because
of two misplaced !s.
Diff:
--- Builder.php.orig Fri Aug 22 10:57:54 2003
+++ Builder.php Fri Aug 22 16:21:47 2003
@@ -201,8 +201,7 @@
$err = $this->_runCommand("phpize", array(&$this,
'phpizeCallback'));
if (PEAR::isError($err)) {
return $err;
- }
- if (!$err) {
+ } elseif ($err) {
return $this->raiseError("`phpize' failed");
}
@@ -255,8 +254,7 @@
if (PEAR::isError($err)) {
chdir($old_cwd);
return $err;
- }
- if (!$err) {
+ } elseif ($err) {
chdir($old_cwd);
return $this->raiseError("`$cmd' failed");
}
So either we have a PHP_Error, or some error-code is in $err, or
everything is fine and the show can go on...
With this patch it works again...
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=25209&edit=1