On Fri, 21 Aug 2015 10:08:56 +0200
Andreas Enge <[email protected]> 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   <[email protected]>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
From bc64fc363e144ac26f6a6b34565a1827c2ce31cf Mon Sep 17 00:00:00 2001
From: Efraim Flashner <[email protected]>
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 <[email protected]>
 ;;; Copyright © 2014, 2015 Ludovic Courtès <[email protected]>
 ;;; Copyright © 2015 Mark H Weaver <[email protected]>
+;;; Copyright © 2015 Efraim Flashner <[email protected]>
 ;;;
 ;;; 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: pgpsEkyYS5YnQ.pgp
Description: OpenPGP digital signature

Reply via email to