unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH 1/2] gnu: Add eyed3.
@ 2015-08-17 19:57 Efraim Flashner
  2015-08-20  9:50 ` 宋文武
  0 siblings, 1 reply; 11+ messages in thread
From: Efraim Flashner @ 2015-08-17 19:57 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 1905 bytes --]

* 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..629c585 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 --]

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-17 19:57 [PATCH 1/2] gnu: Add eyed3 Efraim Flashner
@ 2015-08-20  9:50 ` 宋文武
  2015-08-20 15:19   ` Efraim Flashner
                     ` (2 more replies)
  0 siblings, 3 replies; 11+ messages in thread
From: 宋文武 @ 2015-08-20  9:50 UTC (permalink / raw)
  To: Efraim Flashner, guix-devel

Efraim Flashner <efraim@flashner.co.il> writes:

> * gnu/packages/mp3.scm (eyed3): New variable.
> ---
>  gnu/packages/mp3.scm | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
> [...]
> +    (license license:gpl2)))
The sources have "any later version", so it's 'gpl2+'.
> -- 
> 2.5.0
eyeD3 is both a tool and python library, I wonder whether
'python-eyed3' fits better.

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-20  9:50 ` 宋文武
@ 2015-08-20 15:19   ` Efraim Flashner
  2015-08-24 22:48     ` Mark H Weaver
  2015-08-20 17:55   ` Efraim Flashner
  2015-08-21  8:08   ` Andreas Enge
  2 siblings, 1 reply; 11+ messages in thread
From: Efraim Flashner @ 2015-08-20 15:19 UTC (permalink / raw)
  To: 宋文武, guix-devel

[-- Attachment #1: Type: text/plain, Size: 1271 bytes --]

On Thu, 20 Aug 2015 17:50:10 +0800
宋文武 <iyzsong@gmail.com> wrote:

> Efraim Flashner <efraim@flashner.co.il> writes:
> 
> > * gnu/packages/mp3.scm (eyed3): New variable.
> > ---
> >  gnu/packages/mp3.scm | 25 +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> > [...]
> > +    (license license:gpl2)))
> The sources have "any later version", so it's 'gpl2+'.

The website says: "eyeD3 is written and maintained by Travis Shirk and is
licensed under version 2 of the GPL." And that is backed up by the COPYING
file in the mercurial repo: https://bitbucket.org/nicfit/eyed3/src. Upon
checking the actual source files, it does indeed say "(at your option) any
later version", so it looks like I was mislead by not finding that wording in
the COPYING file.

> eyeD3 is both a tool and python library, I wonder whether
> 'python-eyed3' fits better.

'python-eyed3' is probably a better name. I wonder perhaps as a python
library it should be in python.scm instead of mp3.scm. Anyone have a
suggestion on that front?

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

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

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-20  9:50 ` 宋文武
  2015-08-20 15:19   ` Efraim Flashner
@ 2015-08-20 17:55   ` Efraim Flashner
  2015-08-21  8:08   ` Andreas Enge
  2 siblings, 0 replies; 11+ messages in thread
From: Efraim Flashner @ 2015-08-20 17:55 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 1271 bytes --]

On Thu, 20 Aug 2015 17:50:10 +0800
宋文武 <iyzsong@gmail.com> wrote:

> Efraim Flashner <efraim@flashner.co.il> writes:
> 
> > * gnu/packages/mp3.scm (eyed3): New variable.
> > ---
> >  gnu/packages/mp3.scm | 25 +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> > [...]
> > +    (license license:gpl2)))
> The sources have "any later version", so it's 'gpl2+'.

The website says: "eyeD3 is written and maintained by Travis Shirk and is
licensed under version 2 of the GPL." And that is backed up by the COPYING
file in the mercurial repo: https://bitbucket.org/nicfit/eyed3/src. Upon
checking the actual source files, it does indeed say "(at your option) any
later version", so it looks like I was mislead by not finding that wording in
the COPYING file.

> eyeD3 is both a tool and python library, I wonder whether
> 'python-eyed3' fits better.

'python-eyed3' is probably a better name. I wonder perhaps as a python
library it should be in python.scm instead of mp3.scm. Anyone have a
suggestion on that front?

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

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

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-20  9:50 ` 宋文武
  2015-08-20 15:19   ` Efraim Flashner
  2015-08-20 17:55   ` Efraim Flashner
@ 2015-08-21  8:08   ` Andreas Enge
  2015-08-24 18:54     ` Efraim Flashner
  2 siblings, 1 reply; 11+ messages in thread
From: Andreas Enge @ 2015-08-21  8:08 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

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

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-21  8:08   ` Andreas Enge
@ 2015-08-24 18:54     ` Efraim Flashner
  0 siblings, 0 replies; 11+ messages in thread
From: Efraim Flashner @ 2015-08-24 18:54 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel


[-- 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 --]

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-20 15:19   ` Efraim Flashner
@ 2015-08-24 22:48     ` Mark H Weaver
  2015-08-25  7:38       ` Efraim Flashner
  0 siblings, 1 reply; 11+ messages in thread
From: Mark H Weaver @ 2015-08-24 22:48 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: guix-devel

Efraim Flashner <efraim@flashner.co.il> writes:

> On Thu, 20 Aug 2015 17:50:10 +0800
> 宋文武 <iyzsong@gmail.com> wrote:
>
>> Efraim Flashner <efraim@flashner.co.il> writes:
>> 
>> > * gnu/packages/mp3.scm (eyed3): New variable.
>> > ---
>> >  gnu/packages/mp3.scm | 25 +++++++++++++++++++++++++
>> >  1 file changed, 25 insertions(+)
>> > [...]
>> > +    (license license:gpl2)))
>> The sources have "any later version", so it's 'gpl2+'.
>
> The website says: "eyeD3 is written and maintained by Travis Shirk and is
> licensed under version 2 of the GPL." And that is backed up by the COPYING
> file in the mercurial repo: https://bitbucket.org/nicfit/eyed3/src. Upon
> checking the actual source files, it does indeed say "(at your option) any
> later version", so it looks like I was mislead by not finding that wording in
> the COPYING file.

I suppose it's not widely known, but what really matters is the copying
permission notices in the files themselves.  So indeed, this should be
gpl2+.

>> eyeD3 is both a tool and python library, I wonder whether
>> 'python-eyed3' fits better.
>
> 'python-eyed3' is probably a better name. I wonder perhaps as a python
> library it should be in python.scm instead of mp3.scm. Anyone have a
> suggestion on that front?

If it includes a library, then I think it should be named
'python-eyed3'.  This is our usual convention, and it has the advantage
that we can add 'python2-eyed3' later, in case someone wants it.

Also, your mail client seems to have applied word-wrapping to the patch,
so some of the "+" signs have been wrapped into the description.  This
is what I see:

--8<---------------cut here---------------start------------->8---
+    (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.")
--8<---------------cut here---------------end--------------->8---

This makes it difficult for us to apply it, since we'd have to fix it up
by hand.  Can you persuade your mail client to avoid word-wrapping?  You
could make it an attachment if needed.

     Thanks!
       Mark

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-24 22:48     ` Mark H Weaver
@ 2015-08-25  7:38       ` Efraim Flashner
  2015-08-25  8:11         ` Efraim Flashner
  0 siblings, 1 reply; 11+ messages in thread
From: Efraim Flashner @ 2015-08-25  7:38 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel


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

On Mon, 24 Aug 2015 18:48:04 -0400
Mark H Weaver <mhw@netris.org> wrote:

> Efraim Flashner <efraim@flashner.co.il> writes:
> 
> > On Thu, 20 Aug 2015 17:50:10 +0800
> > 宋文武 <iyzsong@gmail.com> wrote:
> >
> >> Efraim Flashner <efraim@flashner.co.il> writes:
> >> 
> >> eyeD3 is both a tool and python library, I wonder whether
> >> 'python-eyed3' fits better.
> >
> > 'python-eyed3' is probably a better name. I wonder perhaps as a python
> > library it should be in python.scm instead of mp3.scm. Anyone have a
> > suggestion on that front?
> 
> If it includes a library, then I think it should be named
> 'python-eyed3'.  This is our usual convention, and it has the advantage
> that we can add 'python2-eyed3' later, in case someone wants it.

Changed the package to python-eyed3 and moved it to python.scm.

> 
> Also, your mail client seems to have applied word-wrapping to the patch,
> so some of the "+" signs have been wrapped into the description.  This
> is what I see:
> 
> --8<---------------cut here---------------start------------->8---
> +    (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.")
> --8<---------------cut here---------------end--------------->8---
> 
> This makes it difficult for us to apply it, since we'd have to fix it up
> by hand.  Can you persuade your mail client to avoid word-wrapping?  You
> could make it an attachment if needed.
> 
>      Thanks!
>        Mark

I've turned off line wrapping so there shouldn't be any more wrapping in my email messages.

-- 
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: 0001-gnu-Add-python-eyed3.patch --]
[-- Type: text/x-patch, Size: 1745 bytes --]

From b6830016d2bcd3cd68d7eec61f2627677141b13f Mon Sep 17 00:00:00 2001
From: Efraim Flashner <efraim@flashner.co.il>
Date: Tue, 25 Aug 2015 10:16:43 +0300
Subject: [PATCH] gnu: Add python-eyed3.

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

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 940efec..6ef4303 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -4629,3 +4629,25 @@ term.js Javascript terminal emulator library.")
                  ,python2-backport-ssl-match-hostname)
                 ,@(alist-delete "python-tornado"
                                 (package-propagated-inputs terminado)))))))
+
+(define-public python-eyed3
+  (package
+    (name "python-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 gpl2+)))
-- 
2.5.0


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

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-25  7:38       ` Efraim Flashner
@ 2015-08-25  8:11         ` Efraim Flashner
  2015-08-25  8:19           ` Efraim Flashner
  0 siblings, 1 reply; 11+ messages in thread
From: Efraim Flashner @ 2015-08-25  8:11 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel


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

forgot the copyright line

On Tue, 25 Aug 2015 10:38:31 +0300
Efraim Flashner <efraim@flashner.co.il> wrote:

> On Mon, 24 Aug 2015 18:48:04 -0400
> Mark H Weaver <mhw@netris.org> wrote:
> 
> > Efraim Flashner <efraim@flashner.co.il> writes:
> > 
> > > On Thu, 20 Aug 2015 17:50:10 +0800
> > > 宋文武 <iyzsong@gmail.com> wrote:
> > >
> > >> Efraim Flashner <efraim@flashner.co.il> writes:
> > >> 
> > >> eyeD3 is both a tool and python library, I wonder whether
> > >> 'python-eyed3' fits better.
> > >
> > > 'python-eyed3' is probably a better name. I wonder perhaps as a python
> > > library it should be in python.scm instead of mp3.scm. Anyone have a
> > > suggestion on that front?
> > 
> > If it includes a library, then I think it should be named
> > 'python-eyed3'.  This is our usual convention, and it has the advantage
> > that we can add 'python2-eyed3' later, in case someone wants it.
> 
> Changed the package to python-eyed3 and moved it to python.scm.
> 
> > 
> > Also, your mail client seems to have applied word-wrapping to the patch,
> > so some of the "+" signs have been wrapped into the description.  This
> > is what I see:
> > 
> > --8<---------------cut here---------------start------------->8---
> > +    (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.")
> > --8<---------------cut here---------------end--------------->8---
> > 
> > This makes it difficult for us to apply it, since we'd have to fix it up
> > by hand.  Can you persuade your mail client to avoid word-wrapping?  You
> > could make it an attachment if needed.
> > 
> >      Thanks!
> >        Mark
> 
> I've turned off line wrapping so there shouldn't be any more wrapping in my email messages.
> 

-- 
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: 0001-gnu-Add-python-eyed3.patch --]
[-- Type: text/x-patch, Size: 1745 bytes --]

From 9be04652a8e0cefb037d348c6c7e112a7056cc91 Mon Sep 17 00:00:00 2001
From: Efraim Flashner <efraim@flashner.co.il>
Date: Tue, 25 Aug 2015 10:16:43 +0300
Subject: [PATCH] gnu: Add python-eyed3.

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

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 940efec..6ef4303 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -4629,3 +4629,25 @@ term.js Javascript terminal emulator library.")
                  ,python2-backport-ssl-match-hostname)
                 ,@(alist-delete "python-tornado"
                                 (package-propagated-inputs terminado)))))))
+
+(define-public python-eyed3
+  (package
+    (name "python-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 gpl2+)))
-- 
2.5.0


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

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-25  8:11         ` Efraim Flashner
@ 2015-08-25  8:19           ` Efraim Flashner
  2015-09-17 12:12             ` Efraim Flashner
  0 siblings, 1 reply; 11+ messages in thread
From: Efraim Flashner @ 2015-08-25  8:19 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel


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

now I feel like I'm spamming the email list. Comitted the copyright line and made the patch...again.

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

From ff94a5cf13b82d998324fb3d0a4a55dfb992cd86 Mon Sep 17 00:00:00 2001
From: Efraim Flashner <efraim@flashner.co.il>
Date: Tue, 25 Aug 2015 10:16:43 +0300
Subject: [PATCH] gnu: Add python-eyed3.

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

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 940efec..7f4d3c5 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -10,6 +10,7 @@
 ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2015 Christopher Allan Webber <cwebber@dustycloud.org>
 ;;; Copyright © 2015 Eric Dvorsak <eric@dvorsak.fr>
+;;; Copyright © 2015 Efraim Flashner <efraim@flashner.co.il>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -4629,3 +4630,25 @@ term.js Javascript terminal emulator library.")
                  ,python2-backport-ssl-match-hostname)
                 ,@(alist-delete "python-tornado"
                                 (package-propagated-inputs terminado)))))))
+
+(define-public python-eyed3
+  (package
+    (name "python-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 gpl2+)))
-- 
2.5.0


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

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

* Re: [PATCH 1/2] gnu: Add eyed3.
  2015-08-25  8:19           ` Efraim Flashner
@ 2015-09-17 12:12             ` Efraim Flashner
  0 siblings, 0 replies; 11+ messages in thread
From: Efraim Flashner @ 2015-09-17 12:12 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 100 bytes --]

a little bump. this patch seems to have been forgotten, and I think this was
the resolved version

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

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

end of thread, other threads:[~2015-09-17 12:13 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-17 19:57 [PATCH 1/2] gnu: Add eyed3 Efraim Flashner
2015-08-20  9:50 ` 宋文武
2015-08-20 15:19   ` Efraim Flashner
2015-08-24 22:48     ` Mark H Weaver
2015-08-25  7:38       ` Efraim Flashner
2015-08-25  8:11         ` Efraim Flashner
2015-08-25  8:19           ` Efraim Flashner
2015-09-17 12:12             ` Efraim Flashner
2015-08-20 17:55   ` Efraim Flashner
2015-08-21  8:08   ` Andreas Enge
2015-08-24 18:54     ` Efraim Flashner

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