GNU bug report logs -
#56840
[PATCH] FeatherPad package
Previous Next
Full log
Message #8 received at 56840 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
On 30-07-2022 14:31, Pavel Shlyak wrote:
> Hello!
>
> Here is my patch to add FeatherPad package to guix. I upstream it from PantherX channel, hence 3 copyright lines.
> I build it against Qt5 as Qt6 support in FeatherPad is experimental and is not recommended for daily usage yet.
>
> Thank you in advance for your feedback.
> Have a nice day!
>
> + (synopsis "Lightweight Qt5 plain-text editor for Linux")
> + (description "FeatherPad is a lightweight Qt5 plain-text editor
> for Linux")
If it's only for Linux, that needs to be mentioned in the
'supported-systems'; there is no need to duplicate this information in
the description and synopsis. Also, according to the README, this is
incorrect -- e.g., it mentions 'Haiku OS' support; maybe it supports the
Hurd too. There is also no need to mention implementation details such
as Qt5 -- if the user is interested in that, they can do "guix show
featherpad" to see the list of dependencies.
Additionally, descriptions are not the same thing as descriptions, yet
you are writing essentially the same thing in both, try mentioning what
FeatherPad can do and its limitations (especially if they are
limitations compared to other text editors or features not implemented
by them, to help the user deciding between them). Marketing talk like
"lightweight" (*) is to be avoided, see (guix)Synopses and Descriptions.
(*) It's super subjective. Is "lightweight" considering memory usage,
disk usage, CPU usage, ...? I've tested this statement (with disk usage)
for "nano" and the inputs of FeatherPad, and I find that nano takes 86.7
MiB in total and FeatherPad's dependencies take 1156.5 MiB in total --
13.3 times larger! So in a certain sense, it's not lightweight at all,
but heavyweight. If you really want to mention it's lightweight, then be
precise in your exact claim.
> + (native-inputs (list pkg-config hunspell qtsvg qtx11extras
qtbase-5))
Only pkg-config looks like a native-input to me. See (guix)package
Reference for the difference.
> + (arguments `(#:tests? #f))
Tests exist for a reason, don't simply disable them -- if there is a
reason, write down the reason, in a comment. Also, there appears to be
some preference for (list #:tests? #false) -- ` / , is a complicated
construct, and if used, you can get things like ,#~.
> + (license license:gpl3)))
Looking at a random source file (featherpad/fpwin.cpp), this appears to
be incorrect -- GPL-3.0 and GPL-3.0+ are different.
> https://github.com/tsujan/FeatherPad/blob/master/cmake/Modules/FindHUNSPELL.cmake
>
Greetings,
Maxime.
[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]
[OpenPGP_signature (application/pgp-signature, attachment)]
This bug report was last modified 2 years and 258 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.