-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
Override temporrary directory #28
Comments
Note: this is where phpstan looks for the tmpdir: https://github.com/phpstan/phpstan/blob/master/src/Command/CommandHelper.php#L104 |
You can set the
However, this requires modifying the drupal-check source, which not super flexbile. Given that PHPSTan wants a single project file config (see CommandHelper), it would be difficult to expose a parameter to the drupal-check CLI that lets someone pass their own config file. The user would need to know what configs to put in their neon file. |
What was the use case for this? Did people not have a writeable tmp directory? |
I believe it had to do with containers and not having /tmp available. One suggestion was to use the temp directory of the Drupal install being inspected. |
I have a need to change the temp path as well from the default of I'd like to be able to pass a temporary directory specific to my user account (either in my home directory or just |
@adamfranco you should use |
I've decided that this problem will not be fixed in drupal-check. Use PHPStan directly. |
Problem
After running phpstan, it tries to write it’s cache to the temporary directory, which is by default
sys_get_temp_dir() . '/phpstan'
. This may be an issue on filesystems that only have specific directories that are writable.Proposed Resolution
Allow overriding the phpstan tmp directory.
The text was updated successfully, but these errors were encountered: