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

[Efraim Flashner <efraim@flashner.co.il>]

[-- Attachment #1.1: Type: text/plain, Size: 3153 bytes --]

On Fri, 21 Aug 2015 10:08:56 +0200
Andreas Enge <andreas@enge.fr> wrote:

> On Thu, Aug 20, 2015 at 05:50:10PM +0800, 宋文武 wrote:
> > eyeD3 is both a tool and python library, I wonder whether
> > 'python-eyed3' fits better.
> 
> Usually when a package provides binaries we do not use the "python" prefix.
> It would also depend on whether there is a "python-" and a "python2-"
> variant. If not, this could be seen as an indication that the binary
> characteristic is stronger than the library property.
> 
> Andreas
> 

output of apt-{cache,file} show eyeD3:

efraim@debian-netbook:~$ apt-cache search eyed3
eyed3 - Display and manipulate id3-tags on the command-line
python-eyed3 - Python module for id3-tags manipulation

efraim@debian-netbook:~$ apt-file show eyed3
eyed3: /usr/bin/eyeD3
eyed3: /usr/share/doc/eyed3/changelog.Debian.gz
eyed3: /usr/share/doc/eyed3/changelog.gz
eyed3: /usr/share/doc/eyed3/copyright
eyed3: /usr/share/man/man1/eyeD3.1.gz
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3-0.6.18.egg-info
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3/__init__.py
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3/binfuncs.py
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3/frames.py
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3/mp3.py
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3/tag.py
python-eyed3: /usr/lib/python2.6/dist-packages/eyeD3/utils.py
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3-0.6.18.egg-info
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3/__init__.py
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3/binfuncs.py
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3/frames.py
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3/mp3.py
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3/tag.py
python-eyed3: /usr/lib/python2.7/dist-packages/eyeD3/utils.py
python-eyed3: /usr/share/doc/python-eyed3/NEWS.gz
python-eyed3: /usr/share/doc/python-eyed3/README.gz
python-eyed3: /usr/share/doc/python-eyed3/README.html
python-eyed3: /usr/share/doc/python-eyed3/TODO
python-eyed3: /usr/share/doc/python-eyed3/changelog.Debian.gz
python-eyed3: /usr/share/doc/python-eyed3/changelog.gz
python-eyed3: /usr/share/doc/python-eyed3/copyright
python-eyed3: /usr/share/pyshared/eyeD3-0.6.18.egg-info
python-eyed3: /usr/share/pyshared/eyeD3/__init__.py
python-eyed3: /usr/share/pyshared/eyeD3/binfuncs.py
python-eyed3: /usr/share/pyshared/eyeD3/frames.py
python-eyed3: /usr/share/pyshared/eyeD3/mp3.py
python-eyed3: /usr/share/pyshared/eyeD3/tag.py
python-eyed3: /usr/share/pyshared/eyeD3/utils.py

The only dependencies of eyed3 are python-eyed3 and python, python-eyed3's
only dependency is python2.7, and both are from the same source package. I
think having the package called eyed3 and not python-eyed3 is the better
choice.

I've attached a patch with 'license:gpl2' changed to 'license:gpl2+'.

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: gnu-Add-eyed3.patch --]
[-- Type: text/x-patch, Size: 2099 bytes --]

From bc64fc363e144ac26f6a6b34565a1827c2ce31cf Mon Sep 17 00:00:00 2001
From: Efraim Flashner <efraim@flashner.co.il>
Date: Mon, 17 Aug 2015 22:17:32 +0300
Subject: [PATCH] gnu: Add eyed3.

* gnu/packages/mp3.scm (eyed3): New variable.
---
 gnu/packages/mp3.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/mp3.scm b/gnu/packages/mp3.scm
index ddd90fb..8f73a11 100644
--- a/gnu/packages/mp3.scm
+++ b/gnu/packages/mp3.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013 Andreas Enge <andreas@enge.fr>
 ;;; Copyright © 2014, 2015 Ludovic Courtès <ludo@gnu.org>
 ;;; Copyright © 2015 Mark H Weaver <mhw@netris.org>
+;;; Copyright © 2015 Efraim Flashner <efraim@flashner.co.il>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -39,6 +40,7 @@
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system python)
   #:use-module (guix build-system cmake))
 
 (define-public libmad
@@ -450,3 +452,26 @@ format.")
      "Mpc123 is a command-line player for files in the Musepack audio
 compression format (.mpc files).")
     (license license:gpl2+)))
+
+(define-public eyed3
+  (package
+    (name "eyed3")
+    (version "0.7.8")
+    (source (origin
+	      (method url-fetch)
+	      (uri (string-append "http://eyed3.nicfit.net/releases/eyeD3-"
+				  version ".tar.gz"))
+	      (sha256
+	       (base32
+		"1nv7nhfn1d0qm7rgkzksbccgqisng8klf97np0nwaqwd5dbmdf86"))))
+    (build-system python-build-system)
+    (arguments
+     `(#:tests? #f)) ;fail with: UserWarning: Duplicate name: 'paver/shell.py'
+    (home-page "http://eyed3.nicfit.net/")
+    (synopsis "Display and manipulate id3-tags on mp3 files")
+    (description
+     "Eyed3 is a command-line editor to add/edit/remove ID3-tags on mp3 files.
+It supports version 1.0, 1.1, 2.3 and 2.4 of the ID3 standard. Additionally it
+displays several information about the file such as length and bitrate from an
+MP3 file.")
+    (license license:gpl2+)))
-- 
2.5.0


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]