[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#41573] [PATCH Shepherd] shepherd: service: Add #:supplementary-grou
From: |
Oleg Pykhalov |
Subject: |
[bug#41573] [PATCH Shepherd] shepherd: service: Add #:supplementary-groups. |
Date: |
Fri, 19 Jun 2020 04:28:57 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Hello,
Ludovic Courtès <ludo@gnu.org> writes:
> Oleg Pykhalov <go.wigust@gmail.com> skribis:
>
>> From 5718eb5f4130530b48df896d7f7e4a126e08428a Mon Sep 17 00:00:00 2001
>> From: Oleg Pykhalov <go.wigust@gmail.com>
>> Date: Sun, 24 May 2020 20:30:27 +0300
>> Subject: [PATCH] service: Add #:supplementary-groups.
>>
>> * modules/shepherd/service.scm (format-supplementary-groups): New procedure.
>> (exec-command, fork+exec-command, make-forkexec-constructor): Add
>> '#:supplementary-groups'.
>> * doc/shepherd.texi (Service De- and Constructors): Document this.
>
> [...]
>
>> +(define (format-supplementary-groups supplementary-groups)
>> + (if (vector? supplementary-groups)
>> + supplementary-groups
>> + (list->vector (map (lambda (group) (group:gid (getgr group)))
>> + supplementary-groups))))
>
> Perhaps we should remove the ‘vector?’ case, no? I find it clearer when
> the interface accepts just one single data type.
OK.
> Apart from that, it LGTM!
>
> Note that for compatibility reasons we’ll have to wait before using it
> in Guix System.
No problem.
I updated the patch and tested it again with make check and
reconfiguring my system.
0001-service-Add-supplementary-groups.patch
Description: [PATCH] service: Add #:supplementary-groups.
signature.asc
Description: PGP signature