GNU bug report logs - #40931
[PATCH] gnu: add j4-dmenu-desktop

Previous Next

Full log

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

From: Marius Bakke <mbakke <at> fastmail.com>
To: Alex McGrath <amk <at> amk.ie>, 40931 <at> debbugs.gnu.org
Cc: Alex McGrath <amk <at> amk.ie>
Subject: Re: [bug#40931] [PATCH v2] gnu: add j4-dmenu-desktop
Date: Sat, 02 May 2020 16:18:17 +0200

[Message part 1 (text/plain, inline)]

Alex McGrath <amk <at> amk.ie> writes:

> * gnu/packages/xdisorg.scm (j4-dmenu-desktop): New variable.

Thanks!  Unfortunately the patch no longer applies, can you rebase it on
the current master branch?

[...]

> +(define-public j4-dmenu-desktop
> +  (package
> +    (name "j4-dmenu-desktop")
> +    (version "2.17")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/enkore/j4-dmenu-desktop.git")
> +                    (commit (string-append "r" version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "0v23fimkn83dcm5p53y2ymhklff3kwppxhf75sm8xmswrzkixpgc"))))
> +    (build-system cmake-build-system)
> +    (arguments
> +     `(#:configure-flags '("-DCMAKE_BUILD_TYPE=Release" "-DWITH_GIT_CACHE=off" "-DWITH_TESTS=off")

Note that cmake-build-system exposes #:build-type, so you don't have to
pass "-DCMAKE_BUILD_TYPE".  By default it uses RelWithDebugInfo which is
better because it adds debug information.  There are cases where we
want "Release" instead (size or performance concerns), but it probably
is fine for this package?

> +       #:tests? #f)) ;; tests fetch an external git repo

Is it possible to provide this repo as a native-input?  See e.g. 'ghc'.
Don't stress it if it's not possible, though.

> +    (synopsis "A fast desktop menu")

Just "Fast desktop menu" ('./pre-inst-env guix lint j4-dmenu-desktop'
should warn about this).

> +    (description "j4-dmenu-desktop is a replacement for i3-dmenu-desktop. It's purpose is to find desktop files and offer you a menu to start an application using dmenu.")

Use two spaces between sentences, and try to wrap lines at ~78
characters.  'guix lint' will warn about this, too.  :-)

Can you send an updated patch?  Thanks in advance!

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

Previous Next

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