-
Notifications
You must be signed in to change notification settings - Fork 2
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
Adding file_temp_path to settings #144
Conversation
targets/the-build.xml
Outdated
@@ -138,6 +138,7 @@ | |||
<property name="drupal.site.database.database" value="${build.site}" /> | |||
<property name="drupal.site.settings.file_public_path" value="sites/${drupal.site.dir}/files" /> | |||
<property name="drupal.site.settings.file_private_path" value="../artifacts/private/${drupal.site.dir}" /> | |||
<property name="drupal.site.settings.file_temp_path" value="{$_ENV['TEMP']}" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that this {$_ENV['TEMP']}
environment variable is specific to Acquia (docs).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@becw should we add a specific tmp
for every vendor (acquia / pantheon)?
Is there a better way to do it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Each vendor does their own handling of this, so I think that we can customize this behavior in the settings.*.php
templates, rather than providing it as a setting (because then everybody would have to figure out how it should be set for whatever provider they're using). The temp directory is generally an absolute path, rather than a relative path like the public and private files directories, so it tends to be a bit more host-specific.
For example, Pantheon already handles the temp directory as part of its scaffolding: https://pantheon.io/docs/tmp
For Platform.sh, the temp directory is specified in the .platform.app.yaml
file (/tmp
), so we can use that.
Even the settings.build.php
should be fine to hard-code it, since projects can edit that template as necessary (it's copied to each project at .the-build/drupal/settings.build.php
).
This is the right direction! We'll also need to cover the other config files. |
…ific settings file we download from pantheon-systems/drops-8: https://github.com/pantheon-systems/drops-8/blob/default/sites/default/settings.pantheon.php
…, so update our Drupal settings to use that.
… set this in different ways (e.g. Acquia uses an environment variable, which we couldn't put in this property).
Per comment in https://www.drupal.org/project/drupal/releases/8.8.0, under "Other important update information".
Original d.o issue: https://www.drupal.org/project/drupal/issues/3039026