Subject: bug#76289: closed (Re: [bug#76289] [PATCH v3 2/2] services: Add speakersafetyd service.)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 15 Feb 2025 14:36:02 +0000

Your bug report

#76289: [PATCH 0/2] Add speakersafetyd system service.

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].

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

Subject: Re: [bug#76289] [PATCH v3 2/2] services: Add speakersafetyd service.
From: Maxim Cournoyer <[email-masked]>
To: Roman Scherer <[email-masked]>
Cc: Ludovic Courtès <[email-masked]>, [email-masked]
Date: Sat, 15 Feb 2025 23:35:24 +0900
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi,

Roman Scherer <[email-masked]> writes:

> * gnu/services/sound.scm (speakersafetyd-service-type) New variable.
> * doc/guix.texi: Document the speakersafetyd service.
>
> Change-Id: Ib8fa19b056a2036019ae7c199d81e1139664e951

I've made the following small adjustments:

--8<---------------cut here---------------start------------->8---
modified   doc/guix.texi
@@ -26593,6 +26593,7 @@ Sound Services
 See below for details about @code{speakersafetyd-configuration}.
 @end defvar
 
+@c %start of fragment
 @deftp {Data Type} speakersafetyd-configuration
 Available @code{speakersafetyd-configuration} fields are:
 
@@ -26608,14 +26609,14 @@ Sound Services
 contains the configuration files of the speaker models.
 
 @item @code{maximum-gain-reduction} (default: @code{7}) (type: integer)
-Maximum gain reduction before panicing, useful for debugging.
+Maximum gain reduction before panicking, useful for debugging.
 
 @item @code{speakersafetyd} (default: @code{speakersafetyd}) (type: file-like)
 The Speaker Safety Daemon package to use.
 
 @end table
-
 @end deftp
+@c %end of fragment
 
 @node File Search Services
 @subsection File Search Services
modified   gnu/services/sound.scm
@@ -3,6 +3,7 @@
 ;;; Copyright © 2020 Liliana Marie Prikler <[email-masked]>
 ;;; Copyright © 2020 Marius Bakke <[email-masked]>
 ;;; Copyright © 2022 Maxim Cournoyer <[email-masked]>
+;;; Copyright © 2025 Roman Scherer <[email-masked]>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -274,7 +275,7 @@ (define ladspa-service-type
 
 
 ;;;
-;;; Speaker Safety Daemon
+;;; Speaker Safety Daemon.
 ;;;
 
 (define-configuration/no-serialization speakersafetyd-configuration
@@ -289,7 +290,7 @@ (define-configuration/no-serialization speakersafetyd-configuration
 the configuration files of the speaker models.")
   (maximum-gain-reduction
    (integer 7)
-   "Maximum gain reduction before panicing, useful for debugging.")
+   "Maximum gain reduction before panicking, useful for debugging.")
   (speakersafetyd
    (file-like speakersafetyd)
    "The Speaker Safety Daemon package to use."))

--8<---------------cut here---------------end--------------->8---

And expound the changelog a bit to:

--8<---------------cut here---------------start------------->8---
services: Add speakersafetyd service.

* gnu/services/sound.scm (speakersafetyd-shepherd-service)
(speakersafetyd-configuration)
(speakersafetyd-service-type): New variables.
* doc/guix.texi (Sound Services): Document it.
--8<---------------cut here---------------end--------------->8---

And pushed as commit af643735a5.

Thank you for your contribution!  Closing by replying to
[email-masked] in CC.

-- 
Thanks,
Maxim

Subject: [PATCH 0/2] Add speakersafetyd system service.
From: Roman Scherer <[email-masked]>
To: [email-masked]
Cc: Roman Scherer <[email-masked]>, [email-masked]
Date: Fri, 14 Feb 2025 14:55:26 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello Guix,

this patch series adds a system service for speakersafetyd [1], a userspace
daemon written in Rust that implements an analogue of the Texas Instruments
Smart Amp speaker protection model. It is developed and used by the Asahi
Linux project to protect the speakers on Apple Silicon devices.

The patch series contains 2 patches:

- The first one adjusts the SHAREDIR in the speakersafetyd package I submitted
  earlier this year. The shared dir was wrong forcing users to specify it with
  the --config-path option. Now just running speakersafetyd without any
  options does the right thing, instead of complaining it could not find the
  config.

- The second patch is the system service and its documentation.

Could you please review the patch series?

Thanks, Roman.

[1] https://github.com/AsahiLinux/speakersafetyd/

Roman Scherer (2):
  gnu: speakersafetyd: Use correct shared directory.
  services: Add speakersafetyd service.

 doc/guix.texi              | 41 +++++++++++++++++++++++++
 gnu/packages/rust-apps.scm |  2 +-
 gnu/services/sound.scm     | 61 +++++++++++++++++++++++++++++++++++++-
 3 files changed, 102 insertions(+), 2 deletions(-)

base-commit: 4b5f0408e66392ab745dc0f7830732217d88f17d
--
2.48.1