[prev in list] [next in list] [prev in thread] [next in thread] 

List:       pear-dev
Subject:    [PEAR-DEV] #25326 [Asn]: When Upgrading PEAR modules, Mail::Mail needed Net_SMTP.
From:       cox () php ! net
Date:       2003-08-31 14:20:46
[Download RAW message or body]

 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

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[prev in list] [next in list] [prev in thread] [next in thread]