all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Mark H Weaver <mhw@netris.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 1/2] gnu: Add eyed3.
Date: Tue, 25 Aug 2015 11:11:39 +0300	[thread overview]
Message-ID: <20150825111139.20c4b878@debian-netbook> (raw)
In-Reply-To: <20150825103831.474c6cc7@debian-netbook>


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

  reply	other threads:[~2015-08-25  8:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
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

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150825111139.20c4b878@debian-netbook \
    --to=efraim@flashner.co.il \
    --cc=guix-devel@gnu.org \
    --cc=mhw@netris.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.