unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: Add ncmpcpp
@ 2014-06-28 14:52 Cyrill Schenkel
  2014-06-28 23:31 ` David Thompson
  0 siblings, 1 reply; 4+ messages in thread
From: Cyrill Schenkel @ 2014-06-28 14:52 UTC (permalink / raw)
  To: Guix-devel

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: 0001-gnu-Add-ncmpcpp.patch --]
[-- Type: text/x-diff, Size: 1701 bytes --]

From 0c01e9239cc9b19e34323fe5bffb0e3bb2c40fed Mon Sep 17 00:00:00 2001
From: Cyrill Schenkel <cyrill.schenkel@gmail.com>
Date: Sat, 28 Jun 2014 16:36:49 +0200
Subject: [PATCH] gnu: Add ncmpcpp.

* gnu/packages/mpd.scm (ncmpcpp): New variable.
---
 gnu/packages/mpd.scm | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/mpd.scm b/gnu/packages/mpd.scm
index 5841e8b..179a9a5 100644
--- a/gnu/packages/mpd.scm
+++ b/gnu/packages/mpd.scm
@@ -39,7 +39,8 @@
   #:use-module (gnu packages xiph)
   #:export (libmpdclient
             mpd
-            ncmpc))
+            ncmpc
+            ncmpcpp))
 
 (define libmpdclient
   (package
@@ -147,3 +148,25 @@ protocol.")
 terminal using ncurses.")
     (home-page "http://www.musicpd.org/clients/ncmpc/")
     (license license:gpl2)))
+
+(define ncmpcpp
+  (package
+    (name "ncmpcpp")
+    (version "0.5.10")
+    (source (origin
+              (method url-fetch)
+              (uri
+               (string-append "http://ncmpcpp.rybczak.net/stable/ncmpcpp-"
+                              version ".tar.bz2"))
+              (sha256
+               (base32
+                "1a54g6dary1rirrny9fd0hpxpyyffypni3mpbdpvmjnrl9v56vgz"))))
+    (build-system gnu-build-system)
+    (inputs `(("libmpdclient" ,libmpdclient)
+              ("ncurses" ,ncurses)))
+    (native-inputs `(("pkg-config" ,pkg-config)))
+    (synopsis "featureful ncurses based MPD client inspired by ncmpc")
+    (description "Ncmpcpp is very similar to ncmpc if it comes to user
+interface, but it provides a lot of new useful features. ")
+    (home-page "http://ncmpcpp.rybczak.net/")
+    (license license:gpl2)))
-- 
2.0.0

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

* Re: [PATCH] gnu: Add ncmpcpp
  2014-06-28 14:52 [PATCH] gnu: Add ncmpcpp Cyrill Schenkel
@ 2014-06-28 23:31 ` David Thompson
       [not found]   ` <87d2dsf7c3.fsf@gmail.com>
  0 siblings, 1 reply; 4+ messages in thread
From: David Thompson @ 2014-06-28 23:31 UTC (permalink / raw)
  To: Cyrill Schenkel, Guix-devel

Cyrill Schenkel <cyrill.schenkel@gmail.com> writes:

> From 0c01e9239cc9b19e34323fe5bffb0e3bb2c40fed Mon Sep 17 00:00:00 2001
> From: Cyrill Schenkel <cyrill.schenkel@gmail.com>
> Date: Sat, 28 Jun 2014 16:36:49 +0200
> Subject: [PATCH] gnu: Add ncmpcpp.
>
> * gnu/packages/mpd.scm (ncmpcpp): New variable.

Awesome!  This was on my "things to package" list.

> ---
>  gnu/packages/mpd.scm | 25 ++++++++++++++++++++++++-
>  1 file changed, 24 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/mpd.scm b/gnu/packages/mpd.scm
> index 5841e8b..179a9a5 100644
> --- a/gnu/packages/mpd.scm
> +++ b/gnu/packages/mpd.scm
> @@ -39,7 +39,8 @@
>    #:use-module (gnu packages xiph)
>    #:export (libmpdclient
>              mpd
> -            ncmpc))
> +            ncmpc
> +            ncmpcpp))
>  
>  (define libmpdclient
>    (package
> @@ -147,3 +148,25 @@ protocol.")
>  terminal using ncurses.")
>      (home-page "http://www.musicpd.org/clients/ncmpc/")
>      (license license:gpl2)))
> +
> +(define ncmpcpp
> +  (package
> +    (name "ncmpcpp")
> +    (version "0.5.10")
> +    (source (origin
> +              (method url-fetch)
> +              (uri
> +               (string-append "http://ncmpcpp.rybczak.net/stable/ncmpcpp-"
> +                              version ".tar.bz2"))
> +              (sha256
> +               (base32
> +                "1a54g6dary1rirrny9fd0hpxpyyffypni3mpbdpvmjnrl9v56vgz"))))
> +    (build-system gnu-build-system)
> +    (inputs `(("libmpdclient" ,libmpdclient)
> +              ("ncurses" ,ncurses)))
> +    (native-inputs `(("pkg-config" ,pkg-config)))
> +    (synopsis "featureful ncurses based MPD client inspired by ncmpc")

Capitalize "featureful".

> +    (description "Ncmpcpp is very similar to ncmpc if it comes to user
> +interface, but it provides a lot of new useful features. ")

I think a slightly modified version of the description found on the Arch
Wiki would be more useful:

  Ncmpcpp is an mpd client with a UI very similar to ncmpc, but it
  provides new useful features such as support for regular expressions
  for library searches, extended song format, items filtering, the
  ability to sort playlists, and a local filesystem browser.

https://wiki.archlinux.org/index.php/Ncmpcpp

> +    (home-page "http://ncmpcpp.rybczak.net/")
> +    (license license:gpl2)))
> -- 
> 2.0.0
>
>

Looks good, otherwise.  Thanks!

-- 
David Thompson
Web Developer - Free Software Foundation - http://fsf.org
GPG Key: 0FF1D807
Support the FSF: https://fsf.org/donate

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

* Re: [PATCH] gnu: Add ncmpcpp
       [not found]   ` <87d2dsf7c3.fsf@gmail.com>
@ 2014-06-29  8:31     ` Cyrill Schenkel
  2014-06-29 20:38       ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Cyrill Schenkel @ 2014-06-29  8:31 UTC (permalink / raw)
  To: Guix-devel

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

Hi David

> Capitalize "featureful".

Done.

> I think a slightly modified version of the description found on the Arch
> Wiki would be more useful:
>
>   Ncmpcpp is an mpd client with a UI very similar to ncmpc, but it
>   provides new useful features such as support for regular expressions
>   for library searches, extended song format, items filtering, the
>   ability to sort playlists, and a local filesystem browser.

I agree. Changed it in the attached patch.

Regards
Cyrill


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-ncmpcpp.patch --]
[-- Type: text/x-diff, Size: 1853 bytes --]

From 75afd07580a130a50a067511114c7e32417624f5 Mon Sep 17 00:00:00 2001
From: Cyrill Schenkel <cyrill.schenkel@gmail.com>
Date: Sun, 29 Jun 2014 10:23:28 +0200
Subject: [PATCH] gnu: Add ncmpcpp.

* gnu/packages/mpd.scm (ncmpcpp): New variable.
---
 gnu/packages/mpd.scm | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/mpd.scm b/gnu/packages/mpd.scm
index 5841e8b..5d603a1 100644
--- a/gnu/packages/mpd.scm
+++ b/gnu/packages/mpd.scm
@@ -39,7 +39,8 @@
   #:use-module (gnu packages xiph)
   #:export (libmpdclient
             mpd
-            ncmpc))
+            ncmpc
+            ncmpcpp))
 
 (define libmpdclient
   (package
@@ -147,3 +148,27 @@ protocol.")
 terminal using ncurses.")
     (home-page "http://www.musicpd.org/clients/ncmpc/")
     (license license:gpl2)))
+
+(define ncmpcpp
+  (package
+    (name "ncmpcpp")
+    (version "0.5.10")
+    (source (origin
+              (method url-fetch)
+              (uri
+               (string-append "http://ncmpcpp.rybczak.net/stable/ncmpcpp-"
+                              version ".tar.bz2"))
+              (sha256
+               (base32
+                "1a54g6dary1rirrny9fd0hpxpyyffypni3mpbdpvmjnrl9v56vgz"))))
+    (build-system gnu-build-system)
+    (inputs `(("libmpdclient" ,libmpdclient)
+              ("ncurses" ,ncurses)))
+    (native-inputs `(("pkg-config" ,pkg-config)))
+    (synopsis "Featureful ncurses based MPD client inspired by ncmpc")
+    (description "Ncmpcpp is an mpd client with a UI very similar to ncmpc,
+but it provides new useful features such as support for regular expressions
+for library searches, extended song format, items filtering, the ability to
+sort playlists, and a local filesystem browser.")
+    (home-page "http://ncmpcpp.rybczak.net/")
+    (license license:gpl2)))
-- 
2.0.0


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

* Re: [PATCH] gnu: Add ncmpcpp
  2014-06-29  8:31     ` Cyrill Schenkel
@ 2014-06-29 20:38       ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2014-06-29 20:38 UTC (permalink / raw)
  To: Cyrill Schenkel; +Cc: Guix-devel

Hello,

Cyrill Schenkel <cyrill.schenkel@gmail.com> skribis:

> I agree. Changed it in the attached patch.

Applied.  Thanks, and welcome aboard.  ;-)

> +    (license license:gpl2)))

I changed it to ‘license:gpl2+’, because all the source files carry the
“or later” clause.

Under the assumption that you’re the copyright holder, I’ve added this
line to mpd.scm:

;;; Copyright © 2014 Cyrill Schenkel <cyrill.schenkel@gmail.com>

Please let me know if this is incorrect.

Ludo’.

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

end of thread, other threads:[~2014-06-29 20:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-28 14:52 [PATCH] gnu: Add ncmpcpp Cyrill Schenkel
2014-06-28 23:31 ` David Thompson
     [not found]   ` <87d2dsf7c3.fsf@gmail.com>
2014-06-29  8:31     ` Cyrill Schenkel
2014-06-29 20:38       ` Ludovic Courtès

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