From 6e7212bad7af1aeeda5b29094fd19b0c8a6c8327 Mon Sep 17 00:00:00 2001
From: Emily <hello@emily.moe>
Date: Thu, 31 Jul 2025 18:43:22 +0100
Subject: [PATCH] libstore: simplify fallback build directory logic
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This does change the behaviour when the global temporary directory
does not exist, but other uses of the global temporary directory are
already broken in that circumstance, and it should be fixed centrally
if the use case is considered desirable. The logic was not present
before the recent churn around build directories – it was added now
that Lix is taking ownership of the build directory in the store –
so this should not be a meaningful regression.

Change-Id: I6a6a6964e345ea6803226c5ad759e836de7cb0ed
---
 lix/libstore/build/local-derivation-goal.cc | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/lix/libstore/build/local-derivation-goal.cc b/lix/libstore/build/local-derivation-goal.cc
index 531affb49..864bd6432 100644
--- a/lix/libstore/build/local-derivation-goal.cc
+++ b/lix/libstore/build/local-derivation-goal.cc
@@ -467,8 +467,6 @@ try {
             throw;
         }
 
-        auto globalTmp = defaultTempDir();
-        createDirs(globalTmp);
 #if __APPLE__
         /* macOS filesystem namespacing does not exist, to avoid breaking builds, we need to weaken
          * the mode bits on the top-level directory. This avoids issues like
@@ -478,7 +476,7 @@ try {
         constexpr int toplevelDirMode = 0700;
 #endif
         auto nixBuildsTmp = createTempDir(
-            globalTmp, fmt("nix-builds-%s", geteuid()), false, false, toplevelDirMode
+            "", fmt("nix-builds-%s", geteuid()), false, false, toplevelDirMode
         );
         printTaggedWarning(
             "Failed to use the system-wide build directory '%s', falling back to a temporary "
@@ -486,10 +484,10 @@ try {
             settings.buildDir.get(),
             nixBuildsTmp
         );
-        worker.buildDirOverride = nixBuildsTmp;
         tmpDirRoot = createTempDir(
             nixBuildsTmp, "nix-build-" + std::string(drvPath.name()), false, false, 0700
         );
+        worker.buildDirOverride = nixBuildsTmp;
     }
     /* The TOCTOU between the previous mkdir call and this open call is unavoidable due to
      * POSIX semantics.*/