Skip to content

Added a tip about the TMPDIR env variable #7150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from

Conversation

javiereguiluz
Copy link
Member

This will help solve issues like symfony/symfony#20423

@javiereguiluz javiereguiluz changed the base branch from master to 2.7 November 21, 2016 10:58
@nicolas-grekas
Copy link
Member

We should mention that this doesn't apply to Windows.

@nicolas-grekas
Copy link
Member

From msdn for Windows, this var is called TMP, TEMP or USERPROFILE (in this order).

@javiereguiluz
Copy link
Member Author

According to the PHP 7 source code, the variables checked on Windows are TMP and TEMP (in this order). I can't see USERPROFILE See http://lxr.php.net/xref/PHP-7.0/main/php_open_temporary_file.c#199

@nicolas-grekas
Copy link
Member

I'd better trust MSDN than a comment... especially since the code is calling GetTempPath.


Another way to configure the directory where the locks are created is to
define a special environment variable, because PHP will use that value to
override the default temporary directory. In Unix-based systems, define the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On Unix-based systems [...]?

Another way to configure the directory where the locks are created is to
define a special environment variable, because PHP will use that value to
override the default temporary directory. In Unix-based systems, define the
``TMPDIR`` variable. In Windows systems, define any of these variables:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here (On Windows Systems [...])?

@xabbuh
Copy link
Member

xabbuh commented Dec 9, 2016

👍

Status: Reviewed

@xabbuh
Copy link
Member

xabbuh commented Dec 12, 2016

Thank you Javier.

xabbuh added a commit that referenced this pull request Dec 12, 2016
This PR was squashed before being merged into the 2.7 branch (closes #7150).

Discussion
----------

Added a tip about the TMPDIR env variable

This will help solve issues like symfony/symfony#20423

Commits
-------

565dd93 Added a tip about the TMPDIR env variable
@xabbuh xabbuh closed this Dec 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants