Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.10.18

Bug #1330 unused variable in php-src/pear/PEAR/Command/Install.php
Submitted: 2004-05-03 09:05 UTC
From: kameshj at fastmail dot fm Assigned: pajoye
Status: Closed Package: PEAR
PHP Version: 5.0.0RC2 (Release Candidate 2) OS: linux
Roadmaps: (Not assigned)    
Subscription  
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes. If this is not your bug, you can add a comment by following this link. If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
2004-05-16 15:29 UTC
Package:
Bug Type:
Summary:
From: kameshj at fastmail dot fm
New email:
PHP Version: Package Version: OS:

 

 [2004-05-03 09:05 UTC] kameshj at fastmail dot fm
Description: ------------ In php-src/pear/PEAR/Command/Install.php there is a variable bn which is no more used in the code Pasting the patch that removes the unwanted variable. Index: Install.php =================================================================== RCS file: /repository/php-src/pear/PEAR/Command/Install.php,v retrieving revision 1.52 diff -u -r1.52 Install.php --- Install.php 8 Jan 2004 17:33:13 -0000 1.52 +++ Install.php 27 Apr 2004 11:18:22 -0000 @@ -303,7 +303,6 @@ $downloaded = $this->downloader->getDownloadedPackages(); $this->installer->sortPkgDeps($downloaded); foreach ($downloaded as $pkg) { - $bn = basename($pkg['file']); $info = $this->installer->install($pkg['file'], $options, $this->config); if (is_array($info)) { if ($this->config->get('verbose') > 0) {

Comments

 [2004-05-03 09:44 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send
MANUAL CONFIRMATION IS NOT POSSIBLE.  Write a message to pear-dev@lists.php.net
to request the confirmation link.  All bugs/comments/patches associated with this

email address will be deleted within 48 hours if the account request is not confirmed!
 [2004-05-16 15:29 UTC] paj at pearfr dot org
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.