Subject: Re: bug#63082: [PATCH 04/17] services: mpd: Obsolete the 'group' field.

From: Maxim Cournoyer <[email-masked]>
Date: Sat, 29 Apr 2023 13:12:46 -0400

Hi Bruno,

Bruno Victal <[email-masked]> writes:

[...]

>>>> -The default @code{%mympd-group} is a system group with name ``mympd''.
>>>>  @item @code{work-directory} (default: @code{"/var/lib/mympd"}) (type: string)
>>>>  Where myMPD will store its data.
>>>>
>>>> @@ -33872,7 +33869,7 @@ Audio Services
>>>>  Override URI to myMPD.  See
>>>>  @uref{https://github.com/jcorporation/myMPD/issues/950}.
>>>>
>>>> -@item @code{script-acl} (default: @code{(mympd-ip-acl (allow
>>>> '("127.0.0.1")))}) (type: maybe-mympd-ip-acl)
>>>> +@item @code{script-acl} (type: maybe-mympd-ip-acl)
>>>>  ACL to access the myMPD script backend.
>>>
>>> Unrelated change?
>> > From the generated doc, which was probably lagging behind the source.
>
> It's unfortunate but iirc this had to be manually edited in since generate-documentation
> will skip “serializing” values it doesn't know how to represent.

OK.  I'm not sure it's worth manually fiddling with the generated code
instead of keeping it as a reminder to fix it at the source :-).  It's
already a bit of a pain to copy-paste the snippets already.

-- 
Thanks,
Maxim