From eec920ba93ecb086366576e31b785962fbaf81c2 Mon Sep 17 00:00:00 2001 From: Ludovic Courtès Date: Mon, 17 Oct 2022 09:36:43 +0200 Subject: Revert "nix: Guard against removing temporary roots of living processes." This reverts commit bb0beaecdee1a2315a1269b1746d238c8ab0e699. This change broke a number of tests that assumed the previous behavior. --- nix/libstore/gc.cc | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'nix') diff --git a/nix/libstore/gc.cc b/nix/libstore/gc.cc index f101de92d6..16519116e4 100644 --- a/nix/libstore/gc.cc +++ b/nix/libstore/gc.cc @@ -2,7 +2,6 @@ #include "misc.hh" #include "local-store.hh" -#include #include #include #include @@ -227,10 +226,10 @@ static void readTempRoots(PathSet & tempRoots, FDs & fds) //FDPtr fd(new AutoCloseFD(openLockFile(path, false))); //if (*fd == -1) continue; - /* Try to acquire a write lock without blocking. This can only - succeed if the owning process has died, in which case we don't care - about its temporary roots, or if we are the owning process. */ - if (i.name != std::to_string(getpid()) && lockFile(*fd, ltWrite, false)) { + /* Try to acquire a write lock without blocking. This can + only succeed if the owning process has died. In that case + we don't care about its temporary roots. */ + if (lockFile(*fd, ltWrite, false)) { printMsg(lvlError, format("removing stale temporary roots file `%1%'") % path); unlink(path.c_str()); writeFull(*fd, "d"); -- cgit v1.2.3