# Patch: Specify temp directory

- Thread: https://externals.io/message/65021
- Original archive: https://news-web.php.net/php.internals/65021
- Started by ALeX Kazik on 2013-01-18T14:01:58+00:00
- Messages: 6
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #65021 — ALeX Kazik — 2013-01-18T14:01:58+00:00

URL: https://externals.io/message/65021#65021

Hi,

some time ago I created a small patch to make it possible to specify
the temp dir by the php.ini.

It can be found here:
https://bugs.php.net/bug.php?id=60524
(my latest patch (against 5.4.3) also works for 5.4.11 and 5.5.0a3)

Now I do wonder if anything will happen or if that's it?

I would really appreciate if the patch would be included and hopefully
also some other people.

Regards,
ALeX.

---

## #65022 — Lars Strojny — 2013-01-18T14:25:22+00:00

URL: https://externals.io/message/65021#65022 · In reply to: #65021

Hi Alex,

could you open a pull request on GitHub for it. Makes managing things much easier. Other than an outstanding review, I see no reason not to include this functionality.

cu,
Lars

Am 18.01.2013 um 15:01 schrieb ALeX Kazik <alex@kazik.de>:

> Hi,
> 
> some time ago I created a small patch to make it possible to specify
> the temp dir by the php.ini.
> 
> It can be found here:
> https://bugs.php.net/bug.php?id=60524
> (my latest patch (against 5.4.3) also works for 5.4.11 and 5.5.0a3)
> 
> Now I do wonder if anything will happen or if that's it?
> 
> I would really appreciate if the patch would be included and hopefully
> also some other people.
> 
> Regards,
> ALeX.

---

## #65222 — Stas Malyshev — 2013-01-28T02:21:12+00:00

URL: https://externals.io/message/65021#65222 · In reply to: #65021

Hi!

> some time ago I created a small patch to make it possible to specify
> the temp dir by the php.ini.

This looks fine, can't go into 5.4 though because it changes globals
structure size and offsets, which is a BC break.
Should work for 5.5 though since we're not past feature-complete cutoff
yet.
-- 
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227

---

## #65223 — Hannes Magnusson — 2013-01-28T02:55:06+00:00

URL: https://externals.io/message/65021#65223 · In reply to: #65021

>
> Hi,
>
> some time ago I created a small patch to make it possible to specify
> the temp dir by the php.ini.
>
> It can be found here:
> https://bugs.php.net/bug.php?id=60524
> (my latest patch (against 5.4.3) also works for 5.4.11 and 5.5.0a3)
>
> Now I do wonder if anything will happen or if that's it?
>
> I would really appreciate if the patch would be included and hopefully
> also some other people.



Wouldn't it make more sense to have the ini consistent with the
function name, sys_temp_dir?

-Hannes

---

## #65224 — Stas Malyshev — 2013-01-28T07:49:31+00:00

URL: https://externals.io/message/65021#65224 · In reply to: #65223

Hi!

> Wouldn't it make more sense to have the ini consistent with the
> function name, sys_temp_dir?

Yes, I think it would. Alex, could you change it?
-- 
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227

---

## #65227 — ALeX Kazik — 2013-01-28T09:29:01+00:00

URL: https://externals.io/message/65021#65227 · In reply to: #65224

Hi,

> > […]
>
> Yes, I think it would. Alex, could you change it?

You're right. It's changed.