#25008 [Opn->Asn]: unhelpful Installer Error Message

From: 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

« previous php.pear.dev (#19497) next »