#25008 [Opn->Asn]: unhelpful Installer Error Message
| From: | cox@php.net | Date: | Mon, 11 Aug 2003 01:31:57 +0000 |
| Subject: | #25008 [Opn->Asn]: unhelpful Installer Error Message | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-19497@lists.php.net to get a copy of this message | ||
ID: 25008
Updated by: cox@php.net
Reported By: cellog@php.net
-Status: Open
+Status: Assigned
Bug Type: PEAR related
Operating System: na
PHP Version: 4.3.3RC3
-Assigned To:
+Assigned To: cox
Previous Comments:
------------------------------------------------------------------------
[2003-08-10 19:33:19] cellog@php.net
Description:
------------
if you try to install a package from a package.xml, and a file is
listed in the package.xml that doesn't exist, the error message is not
helpful. This patch changes the error message to let you know what
happened
Index: pear/PEAR/Installer.php
===================================================================
RCS file: /repository/php-src/pear/PEAR/Installer.php,v
retrieving revision 1.93
diff -u -r1.93 Installer.php
--- pear/PEAR/Installer.php 4 Aug 2003 13:31:35 -0000 1.93
+++ pear/PEAR/Installer.php 11 Aug 2003 00:29:45 -0000
@@ -222,6 +222,10 @@
$this->log(3, "+ mkdir $dest_dir");
}
if (empty($atts['replacements'])) {
+ if (!file_exists($orig_file)) {
+ return $this->raiseError("file does not exist",
+ PEAR_INSTALLER_FAILED);
+ }
if (!@copy($orig_file, $dest_file)) {
return $this->raiseError("failed to write
$dest_file",
PEAR_INSTALLER_FAILED);
@@ -231,6 +235,10 @@
$md5sum = md5_file($dest_file);
}
} else {
+ if (!file_exists($orig_file)) {
+ return $this->raiseError("file does not exist",
+ PEAR_INSTALLER_FAILED);
+ }
$fp = fopen($orig_file, "r");
$contents = fread($fp, filesize($orig_file));
fclose($fp);
@@ -695,6 +703,9 @@
if (PEAR::isError($res)) {
if (empty($options['ignore-errors'])) {
$this->rollbackFileTransaction();
+ if ($res->getMessage() == "file does not
exist") {
+ $this->raiseError("file $file in
package.xml does not exist");
+ }
return $this->raiseError($res);
} else {
$this->log(0, "Warning: " .
$res->getMessage());
Expected result:
----------------
file XXX in package.xml does not exist
Actual result:
--------------
failed to write /temppath/.tmpXXX
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=25008&edit=1