summary refs log tree commit diff
diff options
context:
space:
mode:
authorMathieu Othacehe <m.othacehe@gmail.com>2020-05-05 14:26:35 +0200
committerMathieu Othacehe <m.othacehe@gmail.com>2020-05-05 16:08:33 +0200
commit892bbea750e5733979ee0423cbdfcea222b07925 (patch)
treed462349651a38b5fb157ba217703ecc67d9e9d48
parent5990e95b607b888edd862a2ccb5ca69dcd17d801 (diff)
downloadguix-892bbea750e5733979ee0423cbdfcea222b07925.tar.gz
build: install: Do not set store GID.
There's no need to set the store GID as is will be done by the guix-daemon,
with the following snippet:

  if (chown(chrootStoreDir.c_str(), 0, buildUser.getGID()) == -1)
        throw SysError(format("cannot change ownership of ‘%1%’") % chrootStoreDir);

* gnu/build/install.scm (directives): Do not set store GID.
-rw-r--r--gnu/build/install.scm4
1 files changed, 1 insertions, 3 deletions
diff --git a/gnu/build/install.scm b/gnu/build/install.scm
index 9753792216..59a118e905 100644
--- a/gnu/build/install.scm
+++ b/gnu/build/install.scm
@@ -101,9 +101,7 @@ the context of the caller.  If the directive matches those defaults then,
 (define (directives store)
   "Return a list of directives to populate the root file system that will host
 STORE."
-  `(;; Note: the store's GID is fixed precisely so we can set it here rather
-    ;; than at activation time.
-    (directory ,store 0 30000 #o1775)
+  `((directory ,store 0 0 #o1775)
 
     (directory "/etc")
     (directory "/var/log")                          ; for shepherd