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  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 49 + 8 = ?

 
 [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.