svn: /pear/pear-core/tags/PEAR-1.9.2/ package2.xml scripts/pearcmd.php

dufuz                                    Sun, 27 Feb 2011 13:56:15 +0000

Revision: http://svn.php.net/viewvc?view=revision&revision=308720

Log:
Fixed Bug #18238: Wrong return code from "pear help" [till]

Bug: http://pear.php.net/bugs/18238 (unknown) 
      
Changed paths:
    U   pear/pear-core/tags/PEAR-1.9.2/package2.xml
    U   pear/pear-core/tags/PEAR-1.9.2/scripts/pearcmd.php

Modified: pear/pear-core/tags/PEAR-1.9.2/package2.xml
===================================================================
--- pear/pear-core/tags/PEAR-1.9.2/package2.xml	2011-02-27 09:39:37 UTC (rev 308719)
+++ pear/pear-core/tags/PEAR-1.9.2/package2.xml	2011-02-27 13:56:15 UTC (rev 308720)
@@ -103,6 +103,7 @@
   * Fixed Bug #17997: Strange output if directories are not writeable [dufuz]
   * Fixed Bug #18001: PEAR/RunTest coverage fails [dufuz]
   * Fixed Bug #18056 [SECURITY]: Symlink attack in PEAR install [dufuz]
+  * Fixed Bug #18238: Wrong return code from "pear help" [till]

   This feature is implemented as a result of #18056
   * Implemented Request #16648: Use TMPDIR for builds instead of /var/tmp [dufuz]

Modified: pear/pear-core/tags/PEAR-1.9.2/scripts/pearcmd.php
===================================================================
--- pear/pear-core/tags/PEAR-1.9.2/scripts/pearcmd.php	2011-02-27 09:39:37 UTC (rev 308719)
+++ pear/pear-core/tags/PEAR-1.9.2/scripts/pearcmd.php	2011-02-27 13:56:15 UTC (rev 308720)
@@ -343,6 +343,10 @@
     }
     fputs($stdout, "$put\n");
     fclose($stdout);
+
+    if ($error === null) {
+        exit(0);
+    }
     exit(1);
 }