GNU bug report logs - #63082
mpd defaul configuration does not work ('No database' error)

Previous Next

Full log

View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Bruno Victal <mirai <at> makinata.eu>
Cc: 63082 <at> debbugs.gnu.org
Subject: bug#63082: [PATCH 09/17] services: mpd: Let Shepherd effect the user/group change.
Date: Sat, 29 Apr 2023 12:52:08 -0400

Hi Bruno,

Bruno Victal <mirai <at> makinata.eu> writes:

[...]

>> +  ;; Note: The user and its group are not serialized, otherwise MPD would
>> +  ;; attempt to switch the user/group itself.  The task of switching the
>> +  ;; user/group is left to Shepherd instead.
>>    (user
>>     (user-account %mpd-user)
>> -   "The user to run mpd as."
>> -   (sanitizer mpd-user-sanitizer))
>> +   "The user to run @command{mpd} as."
>> +   (sanitizer mpd-user-sanitizer)
>> +   (serializer empty-serializer))
>
> Simply write empty-serializer after sanitizer instead.

Done, and for other commits too, though both works and the procedure
version appeared more consistent/readable to my eyes.

>>                       "--cachedir" #$cache-directory)
>>                     #:environment-variables (list #$log-level*)
>> -                   #:log-file #$(if (string? log-to) log-to #f))))
>> +                   #:log-file #$(if (string? log-to) log-to #f)
>
> Generic advice but how about this instead?
>
> #$@(if (string? log-to) `(#:log-file ,log-to) '())
>
> It's cleaner to not explicitly set the keyword argument values when they're not used.

They are used, but set to #f :-).  I have a slight preference for the
current version, which I find more readable.

-- 
Thanks,
Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.