Subject: bug#62298: closed (Re: [PATCH v2 6/8] services: mpd: Set PulseAudio related variables as default value for environment-variables field.)

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 24 Mar 2023 18:11:02 +0000

Your bug report

#62298: [PATCH 0/8] Extensible define-configuration & mpd/mympd service fixes

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
62298: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62298
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [PATCH v2 6/8] services: mpd: Set PulseAudio related variables as default value for environment-variables field.
From: Maxim Cournoyer <[email-masked]>
To: Bruno Victal <[email-masked]>
Cc: [email-masked], [email-masked], [email-masked]
Date: Fri, 24 Mar 2023 14:10:46 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi,

Bruno Victal <[email-masked]> writes:

> These variables are necessary for PulseAudio to work properly out-of-the-box
> for 'non-interactive' users.

> * doc/guix.texi (Audio Services): Update environment-variables field description for
> mpd-configuration data type.
> * gnu/services/audio.scm (mpd-configuration)[environment-variables]: Set
> PULSE_CLIENTCONFIG and PULSE_CONFIG environment variables to the system-wide
> PulseAudio configuration.
> ---
>  doc/guix.texi          | 2 +-
>  gnu/services/audio.scm | 3 ++-
>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/doc/guix.texi b/doc/guix.texi
> index 2b62605b51..af9f7d78c0 100644
> --- a/doc/guix.texi
> +++ b/doc/guix.texi
> @@ -33501,7 +33501,7 @@ Audio Services
>  This is a list of symbols naming Shepherd services that this service
>  will depend on.
>  
> -@item @code{environment-variables} (default: @code{()}) (type: list-of-strings)
> +@item @code{environment-variables} (default: @code{("PULSE_CLIENTCONFIG=/etc/pulse/client.conf" "PULSE_CONFIG=/etc/pulse/daemon.conf")}) (type: list-of-strings)
>  A list of strings specifying environment variables.
>
>  @item @code{log-file} (default: @code{"/var/log/mpd/log"}) (type: maybe-string)
> diff --git a/gnu/services/audio.scm b/gnu/services/audio.scm
> index 56ea2f8638..198157a83b 100644
> --- a/gnu/services/audio.scm
> +++ b/gnu/services/audio.scm
> @@ -361,7 +361,8 @@ (define-configuration mpd-configuration
>     empty-serializer)
>  
>    (environment-variables
> -   (list-of-strings '())
> +   (list-of-strings '("PULSE_CLIENTCONFIG=/etc/pulse/client.conf"
> +                      "PULSE_CONFIG=/etc/pulse/daemon.conf"))
>     "A list of strings specifying environment variables."
>     empty-serializer)

Installed, thank you!

-- 
Thanks,
Maxim

Subject: [PATCH 0/8] Extensible define-configuration & mpd/mympd service fixes
From: Bruno Victal <[email-masked]>
To: [email-masked]
Cc: Bruno Victal <[email-masked]>
Date: Mon, 20 Mar 2023 16:45:47 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

Highlights:

* Make define-configuration extensible.
  define-configuration can now have extra fields where custom-serializer
  was located.

  * New literals: sanitizer, serializer.
    Support user-specified sanitizers.    

* Fixes <https://issues.guix.gnu.org/61570>.
  * Switch to user-account/group for mympd-service-type as well.

* Make mpd-service-type with pulseaudio usable out-of-the-box.

* Fix a mympd-service-type when logging with syslog.

Bruno Victal (8):
  services: configuration: Add user-defined sanitizer support.
  services: replace bare serializers with (serializer ...)
  services: audio: remove redundant list-of-string? predicate.
  services: mympd: Require 'syslog service when configured to log to
    syslog.
  services: mpd: Fix unintentional API breakage for mixer-type field.
  services: mpd: Set PulseAudio related variables as default value for
    environment-variables field.
  services: mpd: Use user-account (resp. user-group) for user (resp.
    group) fields.
  services: mympd: Use user-account (resp. user-group) for user (resp.
    group) fields.

 doc/guix.texi                    |  46 +++++--
 gnu/home/services/shells.scm     |  12 +-
 gnu/services/audio.scm           | 224 ++++++++++++++++++++++---------
 gnu/services/configuration.scm   |  97 ++++++++++---
 gnu/services/security.scm        |   6 +-
 tests/services/configuration.scm | 156 ++++++++++++++++++++-
 6 files changed, 431 insertions(+), 110 deletions(-)

-- 
2.39.1