Subject: [bug#74801] [PATCH] gnu: home: services: Add home-mpv-service-type.

From: Ludovic Courtès <[email-masked]>
Date: Mon, 12 May 2025 09:10:58 +0200

Hi,

Tomas Volf <[email-masked]> writes:

> This commit adds a new service type to generate configuration file for the mpv
> media player.
>
> * gnu/home/services/mpv.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Register it.
> * doc/guix.texi (mpv Media Player): Document it.
>
> Change-Id: I2deb44799a28047cb5d67da97dc6007a9df873af

[...]

> v3:
> - Drop majority of commit message.  (I am not sold on this.)
> - Use ? suffix for boolean options.
> - Include link to the updater script.

[...]

> +          #:global (make-mpv-profile-configuration
> +                    #:fullscreen #t

Are we missing a question mark?

> + #:profiles `((fullscreen . ,(make-mpv-profile-configuration
> +                              #:fullscreen #t))))

Here also.

Otherwise LGTM.

Thanks,
Ludo’.