[PATCH 0/2] Add redsea, minor changes to liquid-dsp

Done

Details

2 participants

Evan Straw
Ludovic Courtès

Owner: unassigned

Submitted by: Evan Straw

Severity: normal

Evan Straw wrote on 16 Jan 2020 08:18

Recipients:(address . guix-patches@gnu.org)

Message-ID:87wo9r98ds.fsf@virtualplaza.i-did-not-set--mail-host-address--so-tickle-me

From c7b8d253b5959f7e8b61bd1730249df26f071a29 Mon Sep 17 00:00:00 2001

Hi all,

This is a new patch series for the same packages submitted in my earlier bug

(38842). Some things changed since the last patch series and so I decided to

submit a different, modified patch series that includes only the changes that

are relevant now to avoid confusion when applying the patches.

Evan Straw (2):

gnu: liquid-dsp: Remove unused module imports.

gnu: Add redsea.

gnu/packages/ham-radio.scm | 51 +++++++++++++++++++++++++++++++++++++-

gnu/packages/sdr.scm | 3 +--

2 files changed, 51 insertions(+), 3 deletions(-)

2.20.1

Evan Straw wrote on 16 Jan 2020 08:37

[PATCH 1/2] gnu: liquid-dsp: Remove unused module imports.

Recipients:(address . 39150@debbugs.gnu.org)

Message-ID:87lfq76edm.fsf@gmail.com

Made some slight modifications to this package; specifically,

(guix utils) was included although it was not used.

-- Evan

From dc527071eb098130a87f6c9259c0faad5fe88efb Mon Sep 17 00:00:00 2001

* gnu/packages/sdr.scm (liquid-dsp): Remove unused module imports.

---

gnu/packages/sdr.scm | 3 +--

1 file changed, 1 insertion(+), 2 deletions(-)

Toggle diff (25 lines)diff --git a/gnu/packages/sdr.scm b/gnu/packages/sdr.scm
index f6c9050503..563a9b3887 100644
--- a/gnu/packages/sdr.scm
+++ b/gnu/packages/sdr.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2019 Christopher Howard <christopher@librehacker.com>
+;;; Copyright © 2019, 2020 Evan Straw <evan.straw99@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,11 +20,9 @@
 (define-module (gnu packages sdr)
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
-  #:use-module (guix utils)
   #:use-module (guix download)
   #:use-module (guix git-download)
   #:use-module (guix build-system gnu)
-  #:use-module (gnu packages)
   #:use-module (gnu packages algebra)
   #:use-module (gnu packages autotools))
 
-- 
2.20.1

Evan Straw wrote on 16 Jan 2020 08:38

[PATCH 2/2] gnu: Add redsea.

Recipients:(address . 39150@debbugs.gnu.org)

Message-ID:87h80v6ebr.fsf@gmail.com

Finally, here is the patch for Redsea, which now uses the liquid-dsp

definition in sdr.scm.

-- Evan

From c7b8d253b5959f7e8b61bd1730249df26f071a29 Mon Sep 17 00:00:00 2001

* gnu/packages/ham-radio.scm (redsea): New variable.

---

gnu/packages/ham-radio.scm | 51 +++++++++++++++++++++++++++++++++++++-

1 file changed, 50 insertions(+), 1 deletion(-)

Toggle diff (81 lines)diff --git a/gnu/packages/ham-radio.scm b/gnu/packages/ham-radio.scm
index 6b3f68cd7d..376191af54 100644
--- a/gnu/packages/ham-radio.scm
+++ b/gnu/packages/ham-radio.scm
@@ -1,6 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2017, 2018, 2019 Arun Isaac <arunisaac@systemreboot.net>
-;;; Copyright © 2019 Evan Straw <evan.straw99@gmail.com>
+;;; Copyright © 2019, 2020 Evan Straw <evan.straw99@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -22,6 +22,8 @@
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages base)
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages image)
   #:use-module (gnu packages libusb)
@@ -29,6 +31,7 @@
   #:use-module (gnu packages pulseaudio)
   #:use-module (gnu packages python)
   #:use-module (gnu packages python-xyz)
+  #:use-module (gnu packages sdr)
   #:use-module (gnu packages xml)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
@@ -127,3 +130,49 @@ satellites, the POES NOAA weather satellite series.  These transmissions are
 on a frequency of 137 MHz.  They can be received using an inexpensive antenna
 and a dedicated receiver.")
     (license license:gpl2+)))
+
+(define-public redsea
+  (package
+    (name "redsea")
+    (version "0.18")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/windytan/redsea")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1y96g0ra2krjb2kypm8s5gdfia45yci4f36klsvyzg8d53v5cwhn"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         ;; The configure.ac file does not explicitly link against libiconv
+         ;; except on Mac OS, causing the build to fail. This phase comments
+         ;; out the original AC_SUBST macro (located inside a conditional) and
+         ;; adds an explicit use of it underneath, so that libiconv is always
+         ;; linked against.
+         (add-after 'unpack 'patch-libiconv
+           (lambda _
+             (substitute* "configure.ac"
+               (("^ +AC_SUBST")
+                "# AC_SUBST")
+               (("esac")
+                "esac\nAC_SUBST([ICONV], [\"-liconv\"])"))
+             #t)))))
+    (inputs
+     `(("libiconv" ,libiconv)
+       ("libsndfile" ,libsndfile)
+       ("liquid-dsp" ,liquid-dsp)))
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)))
+    (home-page "https://github.com/windytan/redsea")
+    (synopsis "Lightweight RDS to JSON decoder")
+    (description "redsea is a lightweight command-line @dfn{FM Radio Data
+System} (FM-RDS) decoder.  Redsea can be used with any RTL-SDR USB radio stick
+with the rtl_fm tool, or any other @dfn{software-defined radio} (SDR) via
+csdr, for example.  It can also decode raw ASCII bitstream, the hex format
+used by RDS Spy, and audio files containing @dfn{multiplex} signals (MPX).")
+    (license license:expat)))
-- 
2.20.1

Ludovic Courtès wrote on 5 Feb 2020 00:07

Re: [bug#39150] [PATCH 0/2] Add redsea, minor changes to liquid-dsp

Recipients:(name . Evan Straw)(address . evan.straw99@gmail.com)(address . 39150-done@debbugs.gnu.org)

Message-ID:87v9omt0ij.fsf@gnu.org

Hi Evan,

Evan Straw <evan.straw99@gmail.com> skribis:

Toggle quote (9 lines)

> This is a new patch series for the same packages submitted in my earlier bug

> (38842). Some things changed since the last patch series and so I decided to

> submit a different, modified patch series that includes only the changes that

> are relevant now to avoid confusion when applying the patches.

> Evan Straw (2):

> gnu: liquid-dsp: Remove unused module imports.

> gnu: Add redsea.

Applied both patches, thank you!

Ludo’.

Closed

Your comment

This issue is archived.

To comment on this conversation send an email to 39150@debbugs.gnu.org

is:open	open issues
is:done	closed issues
submitter:<who>	search issue submitter
author:<who>	search by message author
date:yesterday..now	search by issue date
mdate:3m..2d	search by message date