[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
01/02: daemon: Handle EXDEV when moving to trash directory.
From: |
guix-commits |
Subject: |
01/02: daemon: Handle EXDEV when moving to trash directory. |
Date: |
Sat, 6 Jun 2020 21:22:52 -0400 (EDT) |
marusich pushed a commit to branch master
in repository guix.
commit d445c30ea6f9693454ca96bae9089ba8738a6f78
Author: Chris Marusich <cmmarusich@gmail.com>
AuthorDate: Thu Jun 4 23:26:19 2020 -0700
daemon: Handle EXDEV when moving to trash directory.
Fixes <https://bugs.gnu.org/41607>.
Reported by Stephen Scheck <singularsyntax@gmail.com>.
* nix/libstore/gc.cc (LocalStore::deletePathRecursive): When we try to
move a dead directory into the trashDir using rename(2) but it returns
an EXDEV error, just delete the directory instead. This can happen in a
Docker container when the directory is not on the "top layer".
---
nix/libstore/gc.cc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/nix/libstore/gc.cc b/nix/libstore/gc.cc
index 8bc4e01..e1d0765 100644
--- a/nix/libstore/gc.cc
+++ b/nix/libstore/gc.cc
@@ -455,7 +455,10 @@ void LocalStore::deletePathRecursive(GCState & state,
const Path & path)
throw SysError(format("unable to rename `%1%' to `%2%'") %
path % tmp);
state.bytesInvalidated += size;
} catch (SysError & e) {
- if (e.errNo == ENOSPC) {
+ /* In a Docker container, rename(2) returns EXDEV when the source
+ and destination are not both on the "top layer". See:
+ https://bugs.gnu.org/41607 */
+ if (e.errNo == ENOSPC || e.errNo == EXDEV) {
printMsg(lvlInfo, format("note: can't create move `%1%': %2%")
% path % e.msg());
deleteGarbage(state, path);
}