- The config object is stored by the factory method

- Make install/unistall/package work again (didn't work for me)

Author: tvvcox

pear/PEAR/Command.php

@@ -88,14 +88,14 @@ class PEAR_Command
      *
      * @access public
      */
-    function factory($command)
+    function factory($command, &$config)
     {
         if (empty($GLOBALS['_PEAR_Command_commandlist'])) {
             PEAR_Command::registerCommands();
         }
         if (isset($GLOBALS['_PEAR_Command_commandlist'][$command])) {
             $class = $GLOBALS['_PEAR_Command_commandlist'][$command];
-            $obj = &new $class(PEAR_Command::getUIObject());
+            $obj = &new $class(PEAR_Command::getUIObject(), $config);
             return $obj;
         }
         return PEAR::raiseError("unknown command `$command'");

pear/PEAR/Command/Common.php

@@ -42,10 +42,10 @@ class PEAR_Command_Common extends PEAR
      *
      * @access public
      */
-    function PEAR_Command_Common(&$ui)
+    function PEAR_Command_Common(&$ui, &$config)
     {
         parent::PEAR();
-        $this->config = &PEAR_Config::singleton();
+        $this->config =& $config;
         $this->ui = $ui;
     }
 

pear/PEAR/Command/Install.php

@@ -29,17 +29,6 @@
 class PEAR_Command_Install extends PEAR_Command_Common
 {
     // {{{ properties
-
-    /** Stack of executing commands, to make run() re-entrant
-     * @var array
-     */
-    var $command_stack; // XXX UNUSED to make run() re-entrant
-
-    /** Currently executing command.
-     * @var string
-     */
-    var $command; // XXX UNUSED
-
     // }}}
 
     // {{{ constructor
@@ -49,9 +38,9 @@ class PEAR_Command_Install extends PEAR_Command_Common
      *
      * @access public
      */
-    function PEAR_Command_Install($ui)
+    function PEAR_Command_Install(&$ui, &$config)
     {
-        parent::PEAR_Command_Common($ui);
+        parent::PEAR_Command_Common($ui, $config);
     }
 
     // }}}
@@ -71,19 +60,21 @@ function getCommands()
     // }}}
     // {{{ run()
 
-    function run($command, $options, $params)
+    function run($command, $params)
     {
-        $installer =& new PEAR_Installer($options['php_dir'],
-                                         $options['ext_dir'],
-                                         $options['doc_dir']);
-        $installer->debug = @$options['verbose'];
+        $installer =& new PEAR_Installer($this->config->get('php_dir'),
+                                         $this->config->get('ext_dir'),
+                                         $this->config->get('doc_dir'));
+        $installer->debug = $this->config->get('verbose');
         $failmsg = '';
+        $options = array();
         switch ($command) {
             case 'install':
             case 'upgrade': {
                 if ($command == 'upgrade') {
                     $options['upgrade'] = true;
                 }
+                // The ['force'] and ['nodeps'] options are still missing
                 if ($installer->install($params[0], $options, $this->config)) {
                     $this->ui->displayLine("install ok");
                 } else {
@@ -92,13 +83,13 @@ function run($command, $options, $params)
                 break;
             }
             case 'uninstall': {
-                if ($installer->uninstall($params[0], $uninstall_options)) {
+                if ($installer->uninstall($params[0], $options)) {
                     $this->ui->displayLine("uninstall ok");
                 } else {
                     $failmsg = "uninstall failed";
                 }
                 break;
-            }                
+            }
             default: {
                 return false;
             }

pear/PEAR/Command/Package.php

@@ -12,9 +12,9 @@ class PEAR_Command_Package extends PEAR_Command_Common
      *
      * @access public
      */
-    function PEAR_Command_Package(&$ui)
+    function PEAR_Command_Package(&$ui, &$config)
     {
-        parent::PEAR_Command_Common($ui);
+        parent::PEAR_Command_Common($ui, $config);
     }
 
     // }}}
@@ -48,7 +48,7 @@ function getCommands()
      *
      * @access public
      */
-    function run($command, $options, $params)
+    function run($command, $params)
     {
         $failmsg = '';
         switch ($command) {