build: install: Do not set store GID.
authorMathieu Othacehe <m.othacehe@gmail.com>
Tue, 5 May 2020 12:26:35 +0000 (14:26 +0200)
committerMathieu Othacehe <m.othacehe@gmail.com>
Tue, 5 May 2020 14:08:33 +0000 (16:08 +0200)
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.

gnu/build/install.scm

index 9753792..59a118e 100644 (file)
@@ -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