GNU bug report logs - #66823
[PATCH gnome-team 0/4] Add GNOME Calls

Previous Next

Package: guix-patches;

Reported by: Vivien Kraus <vivien <at> planete-kraus.eu>

Date: Sun, 29 Oct 2023 16:47:02 UTC

Severity: normal

Tags: patch

Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Vivien Kraus <vivien <at> planete-kraus.eu>
Cc: rg <at> raghavgururajan.name, liliana.prikler <at> gmail.com, 66823 <at> debbugs.gnu.org
Subject: [bug#66823] [PATCH gnome-team 3/4] gnu: Add libcallaudio.
Date: Tue, 31 Oct 2023 10:02:55 -0400
Hi,

Vivien Kraus <vivien <at> planete-kraus.eu> writes:

> * gnu/packages/telephony.scm (libcallaudio): New variable.
>
> Change-Id: I41cfe14136858fe0b2a302a9ed8d624d44ad2a95
> ---
>  gnu/packages/telephony.scm | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
>
> diff --git a/gnu/packages/telephony.scm b/gnu/packages/telephony.scm
> index d99b10903e..ca004afc7f 100644
> --- a/gnu/packages/telephony.scm
> +++ b/gnu/packages/telephony.scm
> @@ -94,6 +94,7 @@ (define-module (gnu packages telephony)
>    #:use-module (guix git-download)
>    #:use-module (guix build-system cmake)
>    #:use-module (guix build-system gnu)
> +  #:use-module (guix build-system meson)
>    #:use-module (guix build-system qt))
>  
>  (define-public phonesim
> @@ -1160,3 +1161,29 @@ (define-public sofia-sip
>  uses such as VoIP, IM, and many other real-time and person-to-person
>  communication services.")
>      (license license:lgpl2.1)))
> +
> +(define-public libcallaudio
> +  (package
> +    (name "libcallaudio")
> +    (version "0.1.9")
> +    (source (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://gitlab.com/mobian1/callaudiod/")
> +             (commit version)))
> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32
> +         "0qnllb28101c2ss1k3iwr04gljfyjqmbla5csj6vkq1y63aagr9s"))))
> +    (build-system meson-build-system)
> +    (inputs
> +     (list alsa-lib glib pulseaudio))

Nitpick: for less than 5 inputs, if they fit on the line, I prefer (and
'guix style' too, I think):

--8<---------------cut here---------------start------------->8---
(inputs (list one two ...))
--8<---------------cut here---------------end--------------->8---

on a single line.

> +    (native-inputs
> +     (list
> +      `(,glib "bin")          ;for gdbus-codegen
> +      pkg-config))
> +    (home-page "https://gitlab.com/mobian1/callaudiod")
> +    (synopsis "Library for audio routing during voice calls")
> +    (description "This package provides callaudiod, a daemon to route audio during phone calls,
> +and a library.")

I'd use @command{callaudiod}.

Otherwise LGTM.

-- 
Thanks,
Maxim




This bug report was last modified 1 year and 259 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.