GNU bug report logs - #49783
[PATCH] Add vimpc mpd client package definition

Previous Next

Package: guix-patches;

Reported by: guil_fsf <at> tutanota.com

Date: Fri, 30 Jul 2021 14:23:01 UTC

Severity: normal

Tags: patch

To reply to this bug, email your comments to 49783 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#49783; Package guix-patches. (Fri, 30 Jul 2021 14:23:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to guil_fsf <at> tutanota.com:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Fri, 30 Jul 2021 14:23:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: guil_fsf <at> tutanota.com
To: guix-patches <at> gnu.org
Subject: [PATCH] Add vimpc mpd client package definition
Date: Fri, 30 Jul 2021 15:30:52 +0200 (CEST)
[Message part 1 (text/plain, inline)]
---
gnu/packages/mpd.scm | 50 ++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 50 insertions(+)

diff --git a/gnu/packages/mpd.scm b/gnu/packages/mpd.scm
index b97abcafc3..2c1f189421 100644
--- a/gnu/packages/mpd.scm
+++ b/gnu/packages/mpd.scm
@@ -513,3 +513,53 @@ for your favorite MPD client but an addition to get a better
album-experience.")
     (home-page "https://gitlab.com/coderkun/mcg")
     (license license:gpl3+)))
+
+(define-module (vimpc)
+  #:use-module (guix licenses)
+  #:use-module (guix packages)
+  #:use-module (guix build-system gnu)
+  #:use-module (guix git-download)
+  #:use-module (gnu packages mpd)
+  #:use-module (gnu packages pcre)
+  #:use-module (gnu packages mp3)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages curl)
+  #:use-module (gnu packages pkg-config)
+  #:use-module (gnu packages ncurses))
+
+(define-public vimpc
+  (package
+   (name "vimpc")
+   (version "0.09.2")
+
+   (source (origin
+            (method git-fetch)
+            (uri (git-reference
+                  (url "https://github.com/boysetsfrog/vimpc.git")
+                  (commit (string-append "v" version))))
+            (sha256 (base32 "0lswzkap2nm7v5h7ppb6a64cb35rajysd09nb204rxgrkij4m6nx"))
+            (file-name (git-file-name name version))))
+   (build-system gnu-build-system)
+   (native-inputs `(("autoconf" ,autoconf)
+		    ("automake" ,automake)))
+   (inputs `(("libmpdclient" ,libmpdclient)
+	     ("pcre" ,pcre)
+	     ("taglib" ,taglib)
+	     ("pkg-config" ,pkg-config)
+	     ("curl" ,curl)
+	     ("ncurses" ,ncurses)
+	     ))
+   (arguments
+       `(#:tests? #false
+
+         #:phases
+         (modify-phases %standard-phases
+           (add-after 'unpack 'fix-hardcoded-paths
+             (lambda _
+                  (system* "./autogen")
+	       )))))
+
+   (synopsis "Vi-like Mpd TUI client written in C++")
+   (description "Vimpc provides an alternative to other mpd clients (such as ncmpc and ncmpcpp) that tries to provide an interface similar to that of the vim text editor.")
+   (home-page "https://github.com/boysetsfrog/vimpc")
+   (license gpl3+)))
--
2.32.0 
[Message part 2 (text/html, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#49783; Package guix-patches. (Fri, 30 Jul 2021 18:56:02 GMT) Full text and rfc822 format available.

Message #8 received at 49783 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: guil_fsf <at> tutanota.com, 49783 <at> debbugs.gnu.org
Subject: Re: [bug#49783] [PATCH] Add vimpc mpd client package definition
Date: Fri, 30 Jul 2021 20:55:14 +0200
[Message part 1 (text/plain, inline)]
Hi,

guil_fsf--- via Guix-patches via schreef op vr 30-07-2021 om 15:30 [+0200]:
> ---
> gnu/packages/mpd.scm | 50 ++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 50 insertions(+)

A commit message is missing.

> diff --git a/gnu/packages/mpd.scm b/gnu/packages/mpd.scm
> index b97abcafc3..2c1f189421 100644
> --- a/gnu/packages/mpd.scm
> +++ b/gnu/packages/mpd.scm
> @@ -513,3 +513,53 @@ for your favorite MPD client but an addition to get a better
> album-experience.")
>      (home-page "https://gitlab.com/coderkun/mcg")
>      (license license:gpl3+)))

> +
> +(define-module (vimpc)
> +  #:use-module (guix licenses)
> +  #:use-module (guix packages)
> +  #:use-module (guix build-system gnu)
> +  #:use-module (guix git-download)
> +  #:use-module (gnu packages mpd)
> +  #:use-module (gnu packages pcre)
> +  #:use-module (gnu packages mp3)
> +  #:use-module (gnu packages autotools)
> +  #:use-module (gnu packages curl)
> +  #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages ncurses))

This define-module block can be removed,
and the #:use-module can be moved to the top of the file.

> +(define-public vimpc
> +  (package
> +   (name "vimpc")
> +   (version "0.09.2")
> +
> +   (source (origin
> +            (method git-fetch)
> +            (uri (git-reference
> +                  (url "https://github.com/boysetsfrog/vimpc.git")

I think there is a linter checking that ".git" is not included in GitHub
URLs.  Try "./pre-inst-env guix lint vimpc".

> +                  (commit (string-append "v" version))))
> +            (sha256 (base32 "0lswzkap2nm7v5h7ppb6a64cb35rajysd09nb204rxgrkij4m6nx"))
> +            (file-name (git-file-name name version))))
> +   (build-system gnu-build-system)
> +   (native-inputs `(("autoconf" ,autoconf)
> +     ("automake" ,automake)))
> +   (inputs `(("libmpdclient" ,libmpdclient)
> +      ("pcre" ,pcre)
> +      ("taglib" ,taglib)
> +      ("pkg-config" ,pkg-config)

"pkg-config" should most likely be in 'native-inputs', not 'inputs'.
The difference between 'inputs' and 'native-inputs' matters when
cross-compiling.

I'd suggest running
"./pre-inst-env guix build vimpc --target=x86_64-linux-gnu"
for testing, but meson-build-system (used by libmpdclient) doesn't
support cross-compilation yet (at least, on master it doesn't.  On the
core-updates branch, meson-build-system supports cross-compilation),
so that wouldn't work yet anyway.

> +      ("curl" ,curl)
> +      ("ncurses" ,ncurses)
> +      ))
> +   (arguments
> +       `(#:tests? #false

When disabling tests, always write a comment explaining why.
If the tests fail, that may indicate a real problem (that's
the purpose of tests).

> +
> +         #:phases
> +         (modify-phases %standard-phases
> +           (add-after 'unpack 'fix-hardcoded-paths
> +             (lambda _
> +                  (system* "./autogen")
> +        )))))

The formatting went wrong here (whitespace and lonely parentheses).
I'd recommend using "git format-patch HEAD^..HEAD" and sending the patch
as an attachment, or using "git send-email".

> +   (synopsis "Vi-like Mpd TUI client written in C++")
> +   (description "Vimpc provides an alternative to other mpd clients (such as ncmpc and ncmpcpp) that tries to provide an interface similar to that of the vim text editor.")

This description doesn't explain anything to people who don't know what
'Mpd', 'Vi', 'nmpc' and 'ncmpcpp' are.  I only know of the first two
(a music/video player and a customisable text editor), but only because
I've read something about it some time ago. 

Browsing through the source code, I notice files named "lyricsloader.cpp",
"lyricsfetcher.cpp", "song.cpp".  Maybe you could note that "vimpc"
is a music player?

Greetings,
Maxime.
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 2 years and 241 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.