unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH 0/2] Add cuetools and shntool
@ 2015-12-28  5:32 Leo Famulari
  2015-12-28  5:32 ` [PATCH 1/2] gnu: Add cuetools Leo Famulari
  2015-12-28  5:32 ` [PATCH 2/2] gnu: Add shntool Leo Famulari
  0 siblings, 2 replies; 7+ messages in thread
From: Leo Famulari @ 2015-12-28  5:32 UTC (permalink / raw)
  To: guix-devel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/2] gnu: Add cuetools.
  2015-12-28  5:32 [PATCH 0/2] Add cuetools and shntool Leo Famulari
@ 2015-12-28  5:32 ` Leo Famulari
  2015-12-29 23:26   ` Ludovic Courtès
  2015-12-28  5:32 ` [PATCH 2/2] gnu: Add shntool Leo Famulari
  1 sibling, 1 reply; 7+ messages in thread
From: Leo Famulari @ 2015-12-28  5:32 UTC (permalink / raw)
  To: guix-devel

* 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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/2] gnu: Add shntool.
  2015-12-28  5:32 [PATCH 0/2] Add cuetools and shntool Leo Famulari
  2015-12-28  5:32 ` [PATCH 1/2] gnu: Add cuetools Leo Famulari
@ 2015-12-28  5:32 ` Leo Famulari
  2015-12-29 23:28   ` Ludovic Courtès
  1 sibling, 1 reply; 7+ messages in thread
From: Leo Famulari @ 2015-12-28  5:32 UTC (permalink / raw)
  To: guix-devel

* 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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/2] gnu: Add cuetools.
  2015-12-28  5:32 ` [PATCH 1/2] gnu: Add cuetools Leo Famulari
@ 2015-12-29 23:26   ` Ludovic Courtès
  2015-12-30  0:55     ` Leo Famulari
  0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2015-12-29 23:26 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add shntool.
  2015-12-28  5:32 ` [PATCH 2/2] gnu: Add shntool Leo Famulari
@ 2015-12-29 23:28   ` Ludovic Courtès
  2015-12-30  0:56     ` Leo Famulari
  0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2015-12-29 23:28 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/2] gnu: Add cuetools.
  2015-12-29 23:26   ` Ludovic Courtès
@ 2015-12-30  0:55     ` Leo Famulari
  0 siblings, 0 replies; 7+ messages in thread
From: Leo Famulari @ 2015-12-30  0:55 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add shntool.
  2015-12-29 23:28   ` Ludovic Courtès
@ 2015-12-30  0:56     ` Leo Famulari
  0 siblings, 0 replies; 7+ messages in thread
From: Leo Famulari @ 2015-12-30  0:56 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-12-30  0:56 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-28  5:32 [PATCH 0/2] Add cuetools and shntool Leo Famulari
2015-12-28  5:32 ` [PATCH 1/2] gnu: Add cuetools Leo Famulari
2015-12-29 23:26   ` Ludovic Courtès
2015-12-30  0:55     ` Leo Famulari
2015-12-28  5:32 ` [PATCH 2/2] gnu: Add shntool Leo Famulari
2015-12-29 23:28   ` Ludovic Courtès
2015-12-30  0:56     ` Leo Famulari

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).