From: Christopher Baines <mail@cbaines.net>
To: Riku Viitanen <riku.viitanen0@gmail.com>
Cc: 38625-done@debbugs.gnu.org
Subject: bug#38625: Fwd: [bug#38625] [PATCH] music.scm: Add glyr
Date: Thu, 19 Nov 2020 09:27:06 +0000 [thread overview]
Message-ID: <87d009lms5.fsf@cbaines.net> (raw)
In-Reply-To: <CA+2hkpXEWmafo+aYjX2TvdW0iqkqRV8zXRU8Yn2GCPa3PrBZng@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 3842 bytes --]
Riku Viitanen <riku.viitanen0@gmail.com> writes:
> Hi Ricardo, and thanks for the feedback!
>
> I think all the mentioned issues should now be fixed.
> I checked some of the source files, and it turns out they say permit any
> later version of the license, so I changed the license to lgpl3+.
> Only the README file mentions "just" LGPLv3 without saying anything about
> later versions, so I thought it meant the same as "only".
Hi Riku,
Apologies for the delay in following up on this.
I've gone ahead and pushed the package definition to master as
aee183e1a57aa7606c79d9375e8e3f13e7726e41. I made some tweaks which I'll
describe below.
Also, I couldn't figure out how to apply the patch you included. Because
of things like email clients wrapping lines, the best approach for
sending patches is "git send-email", and if that isn't an option,
attaching the patch files to an email is the next best option.
> Here's the new patch in git format:
>
> From 97d4d5d7d16eb87896d81e070510d919118aa622 Mon Sep 17 00:00:00 2001
> From: Riku Viitanen <riku.viitanen0@gmail.com>
> Date: Wed, 18 Dec 2019 18:30:14 +0200
> Subject: [PATCH] gnu: Add glyr
>
> * gnu/packages/music.scm (glyr): New variable
> ---
> gnu/packages/music.scm | 35 +++++++++++++++++++++++++++++++++++
> 1 file changed, 35 insertions(+)
>
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index f80fb067e8..0adfa96fbe 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -25,6 +25,7 @@
> ;;; Copyright © 2019 raingloom <raingloom@protonmail.com>
> ;;; Copyright © 2019 David Wilson <david@daviwil.com>
> ;;; Copyright © 2019 Alexandros Theodotou <alex@zrythm.org>
> +;;; Copyright © 2019 Riku Viitanen <riku.viitanen0@gmail.com>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -5106,3 +5107,37 @@ MIDI drums and comes as two separate drumkits: Black
> Pearl and Red Zeppelin.")
> (description "Helm is a cross-platform polyphonic synthesizer
> available standalone
> and as an LV2 plugin.")
> (license license:gpl3+)))
> +
> +(define-public glyr
> + (package
> + (name "glyr")
> + (version "1.0.10")
> + (source (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/sahib/glyr.git")
> + (commit version)))
> + (file-name (git-file-name name version))
> + (sha256
> + (base32
> + "1miwbqzkhg0v3zysrwh60pj9sv6ci4lzq2vq2hhc6pc6hdyh8xyr"))))
> + (build-system cmake-build-system)
> + (arguments
> + '(#:tests? #f))
So I had a go at running the tests. Unfortunately some of them depend on
the network, so I was only able to run some of them. There's also Python
tests, but I think that requires packaging some additional Python
libraries, so I've just put a comment in about that.
> + (inputs
> + `(("glib" ,glib)
> + ("curl" ,curl)
> + ("sqlite" ,sqlite)))
> + (native-inputs
> + `(("pkg-config" ,pkg-config)))
> + (home-page "https://github.com/sahib/glyr")
> + (synopsis "Search engine for music related metadata")
> + (description
> + "Glyr comes both in a command-line interface tool (@command{glyrc})
> and
> +as a C library (libglyr), both with an easy to use interface. The sort of
> +metadata glyr is searching (and downloading) is usually the data you see in
> +your musicplayer. And indeed, originally it was written to serve as
> internally
> +library for a musicplayer, but has been extended to work as a standalone
> +program which is able to download cover art, lyrics, photos, biographies,
> +reviews and more.")
> + (license license:lgpl3+)))
Thanks again for the patch,
Chris
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]
prev parent reply other threads:[~2020-11-19 9:28 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-15 16:10 [bug#38625] [PATCH] music.scm: Add glyr Riku Viitanen
2019-12-15 22:31 ` Ricardo Wurmus
[not found] ` <CA+2hkpV44p4Lw2dh46j2_yYMAn0tv0Y5Q4hzQ_mrZWhD1H3ODQ@mail.gmail.com>
2019-12-18 16:57 ` [bug#38625] Fwd: " Riku Viitanen
2020-11-19 9:27 ` Christopher Baines [this message]
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87d009lms5.fsf@cbaines.net \
--to=mail@cbaines.net \
--cc=38625-done@debbugs.gnu.org \
--cc=riku.viitanen0@gmail.com \
/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 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).