Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.10.15

Bug #8384 PEAR_PackageFile doesn't include error messages from Archive_Tar
Submitted: 2006-08-08 10:40 UTC
From: yunosh Assigned: cellog
Status: Closed Package: PEAR (version 1.4.10)
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 32 + 28 = ?

 
 [2006-08-08 10:40 UTC] yunosh (Jan Schneider)
Description: ------------ If an error happens in Archive_Tar its methods are raising a PEAR_Error and return false. PEAR_PackageFile checks for the return value but doesn't get the original error message through PEAR's error handling. An example can be found in line 345 of PackageFile.php. Test script: --------------- Run pear install Foo with a write protected package.xml file in the temporary directory, i.e. /tmp/pear/cache Expected result: ---------------- could not extract the package.xml file from "/tmp/pear/cache/Archive_Tar-1.3.1.tgz": File /tmp/pear/cache/package.xml already exists and is write protected Download of "pear/archive_tar" succeeded, but extraction failed Error: cannot install "pear/Archive_Tar" Actual result: -------------- could not extract the package.xml file from "/tmp/pear/cache/Archive_Tar-1.3.1.tgz" Download of "pear/archive_tar" succeeded, but it is not a valid package archive Error: cannot download "pear/Archive_Tar"

Comments

 [2006-09-21 16:43 UTC] cellog (Greg Beaver)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.