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

Bug #10647 no mktempdir but config temp_dir in Channels.php
Submitted: 2007-04-07 05:11 UTC Modified: 2007-04-08 20:52 UTC
From: tias Assigned: tias
Status: Closed Package: PEAR (version 1.5.1)
PHP Version: Irrelevant OS:
Roadmaps: 1.5.3    
Subscription  


 [2007-04-07 05:11 UTC] tias (Tias Guns)
Description: ------------ Hey, In Command/Channels.php, when a temporary directory is needed, a new one is created with: $tmpdir = System::mktemp(array('-d')); This is not needed as PEAR has its own tempdir, so that can be replaced with: $tmpdir = $this->config->get('temp_dir'); (I noticed this because System::mktemp() doesn't work on my system, and no error about it was given.)

Comments

 [2007-04-07 21:41 UTC] cellog (Greg Beaver)
actually, PEAR doesn't use temp_dir, but instead uses download_dir (strange design choice by Pierre back in the day). Perhaps this would be a good time to start using it :)
 [2007-04-08 20:52 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.
 [2007-04-08 20:52 UTC] cellog (Greg Beaver)
this patch needs to be merged to HEAD from PEAR_1_5