[PATCH 0/2] Add cuetools and shntool

[PATCH 0/2] Add cuetools and shntool

[Leo Famulari]

Cuetools and shntool are tools for working with CUE sheets and audio
CD's table of contents.

For example, I just used them to split a single FLAC file into it's
constituent tracks based on the .cue file. Then, I used them to copy the
metadata tags from the .cue file to the FLAC files.

Leo Famulari (2):
  gnu: Add cuetools.
  gnu: Add shntool.

 gnu/packages/audio.scm | 53 ++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

-- 
2.6.4

[PATCH 1/2] gnu: Add cuetools.

[Leo Famulari]

* gnu/packages/audio.scm (cuetools): New variable.
---
 gnu/packages/audio.scm | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 6a8347a..89dc468 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -1953,3 +1953,36 @@ access to ALSA PCM devices, taking care of the many functions required to
 open, initialise and use a hw: device in mmap mode, and providing floating
 point audio data.")
     (license license:gpl3+)))
+
+(define-public cuetools
+  (package
+    (name "cuetools")
+    (version "1.4.1")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "https://github.com/svend/cuetools/archive/"
+                                 version ".tar.gz"))
+             (file-name (string-append name "-" version ".tar.gz"))
+             (sha256
+              (base32
+               "01xi3rvdmil9nawsha04iagjylqr1l9v9vlzk99scs8c207l58i4"))))
+    (build-system gnu-build-system)
+    ;; The source tarball is not bootstrapped.
+    (arguments
+     `(#:phases
+        (modify-phases %standard-phases
+          (add-after 'unpack 'bootstrap
+                     (lambda _ (zero? (system* "autoreconf" "-vfi")))))))
+    ;; Bootstrapping tools
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("flex" ,flex)
+       ("bison" ,bison)))
+    (synopsis "Cue and toc file parsers and utilities")
+    (description "Cuetools is a set of programs that are useful for manipulating
+and using CUE sheet (cue) files and Table of Contents (toc) files.  CUE and TOC
+files are a way to represent the layout of a data or audio CD in a
+machine-readable ASCII format.")
+    (home-page "https://github.com/svend/cuetools")
+    (license license:gpl2)))
-- 
2.6.4

[PATCH 2/2] gnu: Add shntool.

[Leo Famulari]

* gnu/packages/audio.scm (shntool): New variable.
---
 gnu/packages/audio.scm | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 89dc468..cfece94 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -1986,3 +1986,23 @@ files are a way to represent the layout of a data or audio CD in a
 machine-readable ASCII format.")
     (home-page "https://github.com/svend/cuetools")
     (license license:gpl2)))
+
+(define-public shntool
+  (package
+    (name "shntool")
+    (version "3.0.10")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "http://etree.org/shnutils/shntool/dist/src/"
+                                 "shntool-" version ".tar.gz"))
+             (sha256
+              (base32
+               "00i1rbjaaws3drkhiczaign3lnbhr161b7rbnjr8z83w8yn2wc3l"))))
+    (build-system gnu-build-system)
+    (synopsis "Multi-purpose WAVE data utility")
+    (description "shntool is a multi-purpose WAVE data processing and reporting
+utility.  File formats are abstracted from its core, so it can process any file
+that contains WAVE data, compressed or not - provided there exists a format
+module to handle that particular file type.")
+    (home-page "http://etree.org/shnutils/shntool/")
+    (license license:gpl3+)))
-- 
2.6.4

Re: [PATCH 1/2] gnu: Add cuetools.

[Ludovic Courtès]

Leo Famulari <leo@famulari.name> skribis:

> * gnu/packages/audio.scm (cuetools): New variable.

[...]

> +        (modify-phases %standard-phases
> +          (add-after 'unpack 'bootstrap
> +                     (lambda _ (zero? (system* "autoreconf" "-vfi")))))))

Note that, as per a recent rule, the “(lambda” should be aligned with
the ‘d’ of ‘add-after’.

> +    (home-page "https://github.com/svend/cuetools")
> +    (license license:gpl2)))

AFAICS, source file headers do not mention a specific GPL version; thus,
it should be ‘gpl2+’.

Otherwise LGTM, thanks!

Ludo’.

Re: [PATCH 2/2] gnu: Add shntool.

[Ludovic Courtès]

Leo Famulari <leo@famulari.name> skribis:

> * gnu/packages/audio.scm (shntool): New variable.

[...]

> +    (synopsis "Multi-purpose WAVE data utility")

“WAVE audio data processing tool”?

> +    (description "shntool is a multi-purpose WAVE data processing and reporting
> +utility.  File formats are abstracted from its core, so it can process any file
> +that contains WAVE data, compressed or not - provided there exists a format
                                             ^^^
Literally “---” (three hyphens), with no surrounding space, which
translates to an em dash in Texinfo.

OK with these changes!

Ludo’.

Re: [PATCH 1/2] gnu: Add cuetools.

[Leo Famulari]

On Wed, Dec 30, 2015 at 12:26:19AM +0100, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
> 
> > * gnu/packages/audio.scm (cuetools): New variable.
> 
> [...]
> 
> > +        (modify-phases %standard-phases
> > +          (add-after 'unpack 'bootstrap
> > +                     (lambda _ (zero? (system* "autoreconf" "-vfi")))))))
> 
> Note that, as per a recent rule, the “(lambda” should be aligned with
> the ‘d’ of ‘add-after’.

Done.

> 
> > +    (home-page "https://github.com/svend/cuetools")
> > +    (license license:gpl2)))
> 
> AFAICS, source file headers do not mention a specific GPL version; thus,
> it should be ‘gpl2+’.

Done.

> 
> Otherwise LGTM, thanks!

Pushed as 6da16993eb.

> 
> Ludo’.

Re: [PATCH 2/2] gnu: Add shntool.

[Leo Famulari]

On Wed, Dec 30, 2015 at 12:28:51AM +0100, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
> 
> > * gnu/packages/audio.scm (shntool): New variable.
> 
> [...]
> 
> > +    (synopsis "Multi-purpose WAVE data utility")
> 
> “WAVE audio data processing tool”?

That is better.

> 
> > +    (description "shntool is a multi-purpose WAVE data processing and reporting
> > +utility.  File formats are abstracted from its core, so it can process any file
> > +that contains WAVE data, compressed or not - provided there exists a format
>                                              ^^^
> Literally “---” (three hyphens), with no surrounding space, which
> translates to an em dash in Texinfo.

Done.

> 
> OK with these changes!

Pushed as 6819a8ddd6.

> 
> Ludo’.