#25326 [Asn]: When Upgrading PEAR modules, Mail::Mail needed Net_SMTP.

ID: 25326 Updated by: cox@php.net Reported By: ler at lerctr dot org Status: Assigned Bug Type: PEAR related Operating System: UnixWare 7.1.3UP2 PHP Version: 4.3.3 -Assigned To: pajoye +Assigned To: cox New Comment: I guess that Greg fixed that already, so we only need to pass all the packages to the $installer object as an array. I'll take this task if no other one appears. Previous Comments: ------------------------------------------------------------------------ [2003-08-31 08:14:42] ler at lerctr dot org Actually, the correct version of Net_SMTP is in the tarball, but the install/upgrade part is stupid in that it just does the upgrades in Alphabetical order, which means that Net_SMTP is NOT installed when Mail::Mail is installed. a 2nd make install installs Mail::Mail, because Net_SMTP is installed the first time, but Mail::Mail is not because of the dependency. Does that help any? ------------------------------------------------------------------------ [2003-08-31 06:29:01] cox@php.net Greg we can not access internet at php compile time, in any case. The solution is to ship all the packages needed by the packages bundled in the php distribution. In this case just need to add the correct release tarball of Net_SMTP to php-src/pear/packages. Pierre? ;) ------------------------------------------------------------------------ [2003-08-31 01:39:17] ler at lerctr dot org Given my stuff is already updated, I can't test real easily. I wish I could. LER ------------------------------------------------------------------------ [2003-08-31 01:23:07] cellog@php.net This patch, when used with the CVS version of PEAR (1.3b1) should fix the problem, but as simple as it looks, some serious testing is required. The patch will download and install all required dependencies, or upgrade them if they are already installed. There are some possible side effects from this that result from the increased dependence on downloading. I can't think of anything too serious offhand, but I don't have much experience installing PEAR using the compile --with-pear method. In any case, try out this patch, with the CVS version of PEAR (checkout php-src/pear, and pear upgrade package-PEAR.xml). Let us know if it fixes the problem. Regards, Greg Index: pear/install-pear.php =================================================================== RCS file: /repository/php-src/pear/install-pear.php,v retrieving revision 1.15 diff -u -r1.15 install-pear.php --- pear/install-pear.php 14 Aug 2003 20:14:33 -0000 1.15 +++ pear/install-pear.php 31 Aug 2003 06:19:05 -0000 @@ -86,6 +86,7 @@ $old_ver = $reg->packageInfo($package, 'version'); if (version_compare($new_ver, $old_ver, 'gt')) { $options['upgrade'] = true; + $options['onlyreqdeps'] = true; $err = $installer->install($instfile, $options); if (PEAR::isError($err)) { $ui->outputData(sprintf("[PEAR] %s: %s", $package, $err->getMessage())); ------------------------------------------------------------------------ [2003-08-31 00:42:18] cellog@php.net Perhaps the install script could utilize the new --onlyreqdeps option added in CVS, so that installed packages will always have their required dependencies installed/upgraded? That should fix this issue nicely. Greg ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/25326 -- Edit this bug report at http://bugs.php.net/?id=25326&edit=1