Merge pull request #43 from creative-commoners/pulls/1.0/thats-not-my…

…-home-either FIX Don't blindly assume that HOME is writable
bringyourownideas · Nov 7, 2018 · 9e05e30 · 9e05e30
2 parents 0618b7a + a2a3ad5
commit 9e05e30
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Extensions/ComposerLoaderExtension.php b/src/Extensions/ComposerLoaderExtension.php
@@ -100,8 +100,12 @@ protected function getInstalledConstraint(BaseRepository $repository, $packageNa
     public function onAfterBuild()
     {
         // Mock COMPOSER_HOME if it's not defined already. Composer requires one of the two to be set.
-        if (!getenv('HOME') && !getenv('COMPOSER_HOME')) {
-            putenv('COMPOSER_HOME=/tmp');
+        if (!getenv('COMPOSER_HOME')) {
+            $home = getenv('HOME');
+            if (!$home || !is_dir($home) || !is_writable($home)) {
+                // Set our own directory
+                putenv('COMPOSER_HOME=' . sys_get_temp_dir());
+            }
         }
 
         $originalDir = getcwd();