Hi Bruno,

Bruno Victal <[email protected]> writes:

> Hi Maxim,
>
> On 2023-04-29 18:21, Maxim Cournoyer wrote:
>> @@ -185,20 +174,13 @@ (define-maybe boolean (prefix mpd-))
>>  (define %mpd-user
>>    (user-account
>>     (name "mpd")
>> -   ;; XXX: This is a place-holder to be lazily substituted in (…-accounts)
>> -   ;; with the value from the 'group' field of <mpd-configuration>.
>> -   (group %lazy-group)
>> +   (group "audio")
>
> Perhaps the group should be set to "mpd", with "audio" added to 
> supplementary-groups instead?

I agree it'd be better, but I'd prefer to defer this change until we get
around to supporting <user-group> objects for the group field of
<user-account>, at which point we won't need to add fragile ad-hoc
complications here.

Does this sound reasonable?

-- 
Thanks,
Maxim



Reply via email to