GNU bug report logs - #38553
[PATCH 00/12] Add two KDE packages and 10 KDE utils

Previous Next

Full log

View this message in rfc822 format

From: Ludovic Courtès <ludo <at> gnu.org>
To: Hartmut Goebel <h.goebel <at> crazy-compilers.com>
Cc: 38553 <at> debbugs.gnu.org
Subject: [bug#38553] [PATCH 01/12] gnu: Add grantleetheme.
Date: Thu, 19 Dec 2019 23:20:35 +0100

Hi!

Hartmut Goebel <h.goebel <at> crazy-compilers.com> skribis:

> * gnu/packages/kde.scm (grantleetheme): New variable.

[...]

> +    (properties `((tags . ("Desktop" "KDE" "Themes"))))

Please remove this line since it’s not handled by tools and not agreed
upon.

> +       (modify-phases (@ (guix build qt-build-system) %standard-phases)
                          ^
As a rule of thumb, we should avoid using ‘@’ and ‘@@’.  So here, I’d
suggest writing

  #:modules ((guix build qt-build-system) …)

to make sure that module is in scope.

> +    (home-page "https://cgit.kde.org/grantleetheme.git")
> +    (synopsis "Library providing grantlee theme support")
> +    (description "This library provides grantlee theme support.")

Please capitalize “Grantlee” if it’s a proper noun, and ideally expound
a bit.

> +    (license (list license:gpl2+ license:lgpl2.0+ license:fdl1.2+))))

Could you add a comment explaining whether this is triple-license or
something else?

Thanks,
Ludo’.

Previous Next

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