| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.10.16 |
| Bug #10647 | no mktempdir but config temp_dir in Channels.php | ||
|---|---|---|---|
| Submitted: | 2007-04-07 10:11 UTC | ||
| From: | tias | Assigned: | tias |
| Status: | Closed | Package: | PEAR (version 1.5.1) |
| PHP Version: | Irrelevant | OS: | |
| Roadmaps: | 1.5.3 | ||
[2007-04-07 10: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-08 02: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-09 01: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.