From patchwork Sun Oct 13 22:11:59 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: David Wilson X-Patchwork-Id: 15716 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 9205A1759D; Mon, 14 Oct 2019 06:38:16 +0100 (BST) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,HTML_MESSAGE, T_DKIM_INVALID,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 266B117520 for ; Mon, 14 Oct 2019 06:38:16 +0100 (BST) Received: from localhost ([::1]:44658 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iJt3f-0005Tu-Hw for patchwork@mira.cbaines.net; Mon, 14 Oct 2019 01:38:15 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60993) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iJt3V-0005TX-KV for guix-patches@gnu.org; Mon, 14 Oct 2019 01:38:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iJt3U-0007MK-1d for guix-patches@gnu.org; Mon, 14 Oct 2019 01:38:05 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:56954) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iJt3T-0007M5-KL for guix-patches@gnu.org; Mon, 14 Oct 2019 01:38:03 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1iJt3T-0001xb-HU for guix-patches@gnu.org; Mon, 14 Oct 2019 01:38:03 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#37701] [PATCH] gnu: Add playerctl. Resent-From: David Wilson Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Mon, 14 Oct 2019 05:38:03 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 37701 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Ludovic =?utf-8?q?Court=C3=A8s?= Cc: 37701-done@debbugs.gnu.org Received: via spool by 37701-done@debbugs.gnu.org id=D37701.15710314527476 (code D ref 37701); Mon, 14 Oct 2019 05:38:03 +0000 Received: (at 37701-done) by debbugs.gnu.org; 14 Oct 2019 05:37:32 +0000 Received: from localhost ([127.0.0.1]:37538 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iJt2x-0001wU-B1 for submit@debbugs.gnu.org; Mon, 14 Oct 2019 01:37:32 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:33505) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iJm6N-0008F2-IJ for 37701-done@debbugs.gnu.org; Sun, 13 Oct 2019 18:12:37 -0400 Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.nyi.internal (Postfix) with ESMTP id 7743021B2F; Sun, 13 Oct 2019 18:12:30 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Sun, 13 Oct 2019 18:12:30 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=daviwil.com; h= date:from:to:cc:message-id:in-reply-to:references:subject :mime-version:content-type; s=fm3; bh=yoV808lCZLzLnl/XcyLdseOEyI C1Z6uokquICcY9l5Q=; b=Q8USXftg6p6/dUh5U1LHjSBTm1BzTzEPP3O5x37GLh EM5pfwfegIJgJbmY9eXqB3qt7/eIGCtE2rMY0IUW1fb3mRE8Keo6KkaWjBrRgqLh BW+iUETPX07t7WeOSug4g4w1+4t1UjaXYgJZ6hS6sCMMOQ1LUe0F2ESXYc/frxuP hlRm64pVo1VlvFqid9mpnLWCRnNwT9L3aB6owmpzLAHY7Y/Zkq+cef023M017nwa yhv47u/wNWYaV5r/dEkOhHwXjOLFJony0BXiMT0ZOiDEWzenAfpFm+R41uhrj3gb qWsmU7mzet+YKmV1nJ0wdbY/bWVyarmuaQxiwDFueNzA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=yoV808 lCZLzLnl/XcyLdseOEyIC1Z6uokquICcY9l5Q=; b=aU1nfltrc3QC0trmt/Jp2W OeCTqTGxtEwAOAIN4N1VbeVDaRqwzzYhLQlkPnaWp6PQHGDriypqXUO9gYG0/3cc K2GN6aym7anfKjtXDQi08h2X+Z5hx4AxX9936oHWCPPadr+LTcf3S632/enJWC6X fG6ezXBW/qEnDzLN81V+YdMPYRvYmpdkrM8p5unSUQm+3OQJtJVxNXz+onXKTDMb GliMR6GUKWbS48421kJhM5FHdLGcTZq6Djw6eVX6uO0hLD7dPwxls2C0KtvnNHzm R0Ad6x1Oyjum3c3sNFPCerEChY2Lq3H18WtvQdiXV17aTJP1e8UJQyFMbhvLXhGQ == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedufedrjedtgddtkecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvkfgjfhfugggtofesmhdtregsredtjeenucfhrhhomhepffgrvhhiugcu hghilhhsohhnuceouggrvhhiugesuggrvhhifihilhdrtghomheqnecukfhppeehvddrud dvhedruddvkedrfeegnecurfgrrhgrmhepmhgrihhlfhhrohhmpegurghvihgusegurghv ihifihhlrdgtohhmnecuvehluhhsthgvrhfuihiivgeptd X-ME-Proxy: Received: from mail.outlook.com (unknown [52.125.128.34]) by mail.messagingengine.com (Postfix) with ESMTPA id E4C0ED60057; Sun, 13 Oct 2019 18:12:29 -0400 (EDT) Date: Sun, 13 Oct 2019 22:11:59 +0000 (UTC) From: David Wilson Message-ID: <146CCE75D21B3446.d994f6e3-509c-4a25-8cab-f9a20bc806d6@mail.outlook.com> In-Reply-To: <875zks8fz6.fsf@gnu.org> References: <5e3cd1fc-bb34-48cc-9f28-24bcec0333c9@www.fastmail.com> <875zks8fz6.fsf@gnu.org> MIME-Version: 1.0 X-Mailer: Outlook for iOS and Android X-Mailman-Approved-At: Mon, 14 Oct 2019 01:37:30 -0400 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches Thanks so much Ludo! I wasn't sure about whether the license included future versions, will keep an eye on the file headers in the future.  David On Sun, Oct 13, 2019 at 2:59 PM -0700, "Ludovic Courtès" wrote: Hi David, "David Wilson" skribis: > This patch adds the 'playerctl' command line utility for controlling media players that support MPRIS. As this is my first attempt at packaging software for Guix, please let me know if anything can be improved. I've run through most of the steps in the packaging checklist of the manual so I feel pretty confident that it's in good shape. > > Also, this is my first contribution to a GNU project (!!) so please advise if I need to follow any "new contributor" protocol. Looks like you followed the protocol perfectly well! :-) > From 536c7d321ca310762940205358ce26a7141c8e9c Mon Sep 17 00:00:00 2001 > From: David Wilson > Date: Thu, 10 Oct 2019 22:06:43 -0700 > Subject: [PATCH] gnu: Add playerctl. > > * gnu/packages/music.scm (playerctl): New variable. Applied with the minor changes below: the ‘file-name’ issue was reported by ‘guix lint’, and the license is LGPLv3-or-later because source file headers carry the “or any later version” wording. Thank you! Ludo’. diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index dcda71a43d..a051caca2b 100644 --- a/gnu/packages/music.scm +++ b/gnu/packages/music.scm @@ -4633,6 +4633,7 @@ easier to perform. Features include: (uri (git-reference (url "https://github.com/altdesktop/playerctl.git") (commit (string-append "v" version)))) + (file-name (git-file-name name version)) (sha256 (base32 "1f3njnpd52djx3dmhh9a8p5a67f0jmr1gbk98icflr2q91149gjz")))) @@ -4643,7 +4644,7 @@ easier to perform. Features include: (native-inputs `(("glib:bin" ,glib "bin") ("pkg-config" ,pkg-config))) - (synopsis "Controls MPRIS-supporting media player applications") + (synopsis "Control MPRIS-supporting media player applications") (description "Playerctl is a command-line utility and library for controlling media players that implement the MPRIS D-Bus Interface Specification. Playerctl @@ -4651,4 +4652,4 @@ makes it easy to bind player actions, such as play and pause, to media keys. You can also get metadata about the playing track such as the artist and title for integration into status line generators or other command-line tools.") (home-page "https://github.com/altdesktop/playerctl") - (license license:lgpl3))) + (license license:lgpl3+)))