GNU bug report logs -
#74801
[PATCH] gnu: home: services: Add home-mpv-service-type.
Previous Next
Reported by: Tomas Volf <~@wolfsden.cz>
Date: Wed, 11 Dec 2024 21:54:02 UTC
Severity: normal
Tags: patch
Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
On 2025-05-09 10:49, Ludovic Courtès wrote:
> Hello,
>
> Tomas Volf <~@wolfsden.cz> writes:
>
>> Ludovic Courtès <ludo <at> gnu.org> writes:
>
> [...]
>
>>> Could you include the code you used to generate the options?
>>
>> I definitely *could* do that. While the code is maybe a bit ugly, it is
>> not intended to be kept secret. Couple of questions though.
>>
>> 1. Where should I put it? Somewhere in etc directory? Or next to the
>> service, so e.g. gnu/home/services/mpv-refresh.sh?
>>
>> 2. It has external dependencies, some of which are packaged in Guix
>> (emacs, emacs-paredit) and some of them are not (guile-wolfsden). Is
>> that a problem? Do I need to rewrite it to use just packaged
>> libraries and programs?
>
> Well, in that case, maybe just include the URL in the script in the
> comment (“Generated by …”).
>
> Had it been pure Scheme (possibly based on (guix read-print) rather than
> Emacs), we could have included it straight in the code, as is done for
> Dovecot IIRC. Someone updating it could use the relevant procedures at
> the REPL to re-generate the option definitions.
I would also suggest to keep auto-generated schemas and docs in separate
files, so they can be easily re-generated without copy-pasting or other
manual interventions.
--
Best regards,
Andrew Tropin
[signature.asc (application/pgp-signature, inline)]
This bug report was last modified 24 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.