From: Ricardo Wurmus <rekado@elephly.net>
To: Al McElrath <hello@yrns.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add pianobar.
Date: Thu, 11 Feb 2016 09:14:03 +0100 [thread overview]
Message-ID: <871t8jvf3o.fsf@elephly.net> (raw)
In-Reply-To: <87lh6rn0zf.fsf@atonesir.com>
Al McElrath <hello@yrns.org> writes:
> Another new package. Feedback appreciated.
Thank you for the patch!
> This is free software, but it
> only works with a non-free music service, Pandora. Is that acceptable?
The concept of “non-free” doesn’t directly apply to services. See
“Untangling the SaaSS Issue from the Proprietary Software Issue” here:
https://www.gnu.org/philosophy/who-does-that-server-really-serve.en.html
and also this:
https://www.gnu.org/philosophy/network-services-arent-free-or-nonfree.html
I don’t know anything about Pandora (I do know my mythology, though), so
I’m not in a position to anticipate and answer questions about it. In
general I think that if “pianobar” itself is free software (which you
say it is) and you do not need to use non-free software to set it up in
the first place then it is probably acceptable.
Now, on to a review of the patch!
> From 4f477999635b4a16f19ef0833727abcf899dbb69 Mon Sep 17 00:00:00 2001
> From: Al McElrath <hello@yrns.org>
> Date: Fri, 11 Dec 2015 21:22:33 -0800
> Subject: [PATCH] gnu: Add pianobar.
> * gnu/packages/music.scm (pianobar): New variable.
> ---
> gnu/packages/music.scm | 36 ++++++++++++++++++++++++++++++++++++
> 1 file changed, 36 insertions(+)
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index a60ba4b..c5a6dc5 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -2,6 +2,7 @@
> ;;; Copyright © 2014 Eric Bavier <bavier@member.fsf.org>
> ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
> ;;; Copyright © 2015 Paul van der Walt <paul@denknerd.org>
> +;;; Copyright © 2016 Al McElrath <hello@yrns.org>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -39,6 +40,7 @@
> #:use-module (gnu packages code)
> #:use-module (gnu packages check)
> #:use-module (gnu packages compression)
> + #:use-module (gnu packages curl)
> #:use-module (gnu packages docbook)
> #:use-module (gnu packages doxygen)
> #:use-module (gnu packages flex)
> @@ -46,6 +48,7 @@
> #:use-module (gnu packages fonts)
> #:use-module (gnu packages fontutils)
> #:use-module (gnu packages gcc)
> + #:use-module (gnu packages gnupg)
> #:use-module (gnu packages gettext)
> #:use-module (gnu packages ghostscript)
> #:use-module (gnu packages gl)
> @@ -1078,3 +1081,36 @@ computer's keyboard.")
> JACK for audio and ALSA sequencer for MIDI as multimedia infrastructures and
> follows a traditional multi-track tape recorder control paradigm.")
> (license license:gpl2+)))
> +
> +(define-public pianobar
> + (package
> + (name "pianobar")
> + (version "2015.11.22")
> + (source (origin
> + (method url-fetch)
> + (uri (string-append "https://github.com/PromyLOPh/"
> + name "/archive/" version ".tar.gz"))
> + (file-name (string-append name "-" version ".tar.gz"))
> + (sha256
> + (base32
> + "022df19bhxqvkhy0qy21xahba5s1fm17b13y0p9p9dnf2yl44wfv"))))
> + (build-system gnu-build-system)
> + (arguments
> + `(#:tests? #f ; no tests
> + #:make-flags (list "CC=gcc" "CFLAGS=-std=c99"
> + (string-append "PREFIX=" %output))
> + #:phases (modify-phases %standard-phases
> + (delete 'configure))))
> + (inputs
> + `(("ao" ,ao)
> + ("curl" ,curl)
> + ("libgcrypt" ,libgcrypt)
> + ("json-c" ,json-c)
> + ("ffmpeg" ,ffmpeg)))
> + (native-inputs
> + `(("pkg-config" ,pkg-config)))
> + (home-page "http://6xq.net/projects/pianobar/")
> + (synopsis "Console-based pandora.com player")
> + (description "pianobar is a free/open-source, console-based client for the
> +personalized online radio Pandora.")
Everything in Guix qualifies as free software, so we do not need to
explicitly mention that it is free software. Maybe the description is a
little bit short — could you possibly incorporate parts of the feature
list?
> + (license license:x11)))
I think it’s the Expat license, not X11. I only compared these
texts:
https://github.com/PromyLOPh/pianobar/blob/master/COPYING
http://directory.fsf.org/wiki/License:Expat
http://directory.fsf.org/wiki/License:X11
Thanks again and welcome!
~~ Ricardo
next prev parent reply other threads:[~2016-02-11 8:14 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-11 7:46 [PATCH] gnu: Add pianobar Al McElrath
2016-02-11 8:14 ` Ricardo Wurmus [this message]
2016-02-11 21:36 ` Al McElrath
2016-02-12 4:51 ` Leo Famulari
2016-02-12 6:00 ` Al McElrath
2016-02-12 23:19 ` Leo Famulari
2016-02-13 22:37 ` Al McElrath
2016-02-13 22:49 ` Leo Famulari
2016-02-13 22:56 ` Leo Famulari
2016-02-13 23:52 ` Leo Famulari
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=871t8jvf3o.fsf@elephly.net \
--to=rekado@elephly.net \
--cc=guix-devel@gnu.org \
--cc=hello@yrns.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.