| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.10.16 |
| Bug #18102 | pear install does not fail on error | ||
|---|---|---|---|
| Submitted: | 2010-12-08 19:27 UTC | ||
| From: | yecril71pl | Assigned: | ashnazg |
| Status: | Closed | Package: | PEAR (version 1.9.1) |
| PHP Version: | 5.3.3 | OS: | Linux |
| Roadmaps: | (Not assigned) | ||
[2010-12-08 19:27 UTC] yecril71pl
(Christopher Yeleighton)
Description:
------------
When pear fails to install a package, it should return an exit code to the calling process. Failure to do this breaks make, rpmbuild and similar scripts.
Test script:
---------------
/usr/bin/gzip -dc /usr/src/packages/SOURCES/PhpDocumentor-1.4.3.tgz | /bin/tar -xvvf -
pear install --nodeps package.xml || echo 'Failed.'
Expected result:
----------------
pear/PhpDocumentor can optionally use package "pear/XML_Beautifier" (version >= 1.1)
rolling back 1 file operations
ERROR: file ./docbuilder/images/rc-gui-install-24.png does not exist
Failed.
Actual result:
--------------
pear/PhpDocumentor can optionally use package "pear/XML_Beautifier" (version >= 1.1)
rolling back 1 file operations
ERROR: file ./docbuilder/images/rc-gui-install-24.png does not exist
Comments
[2017-04-13 22:56 UTC] aripringle
(Ari Pringle)
I know this bug is quite old, but I ran into it recently when the "zip" package stopped bundling libzip. Our automated docker image builds were still succeeding, even though the necessary (for our case) zip extension was not being installed.
[2017-04-13 23:27 UTC] aripringle
(Ari Pringle)
FWIW, I made a PR to attempt to address this:
https://github.com/pear/pear-core/pull/66
[2017-04-26 18:30 UTC] ashnazg
(Chuck Burgess)
-Status: Open
+Status: Assigned
-Assigned To:
+Assigned To: ashnazg
[2017-04-26 18:30 UTC] ashnazg
(Chuck Burgess)
-Status: Assigned
+Status: Closed
Fix should be included in upcoming v1.10.4 release.