[bug#56840] [PATCH] FeatherPad package

[Pavel Shlyak <p.shlyak@pantherx.org>]

[-- Attachment #1: Type: text/plain, Size: 265 bytes --]

Thank you for your feedback! I have tried to fix the issues you mentioned and I also added translations. I have edited the package description in accordance with your recommendations and I kindly asked app author/maintainer to appear in this thread to comment it.

[-- Attachment #2: 0001-gnu-featherpad-new-package.patch --]
[-- Type: application/octet-stream, Size: 2356 bytes --]

From 6f2038d57b543e9fb1df22dcc75e778f3742660c Mon Sep 17 00:00:00 2001
From: Pavel Shlyak <p.shlyak@pantherx.org>
Date: Sat, 30 Jul 2022 13:57:30 +0300
Subject: [PATCH] gnu: featherpad: new package.

    * gnu/packages/text-editors.scm (featherpad): new package.
---
 gnu/packages/text-editors.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/text-editors.scm b/gnu/packages/text-editors.scm
index 0c8531cc4d..50ab492c0b 100644
--- a/gnu/packages/text-editors.scm
+++ b/gnu/packages/text-editors.scm
@@ -19,6 +19,9 @@
 ;;; Copyright © 2021 Calum Irwin <calumirwin1@gmail.com>
 ;;; Copyright © 2022 Luis Henrique Gomes Higino <luishenriquegh2701@gmail.com>
 ;;; Copyright © 2022 Foo Chuan Wei <chuanwei.foo@hotmail.com>
+;;; Copyright © 2018 Fakhri Sajadi <f.sajadi@pantherx.org>
+;;; Copyright © 2021 Reza Alizadeh Majd <r.majd@pantherx.org>
+;;; Copyright © 2022 Pavel Shlyak <p.shlyak@pantherx.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -849,6 +852,32 @@ (define-public editorconfig-core-c
 editors.")
     (license license:bsd-2)))
 
+(define-public featherpad
+  (package
+    (name "featherpad")
+    (version "1.3.0")
+    (home-page "https://github.com/tsujan/FeatherPad")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url home-page)
+                    (commit (string-append "V" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1knxf05lfvpfkj4lxi71kidg2f7nfkgsidb45m86h9mnbqmxzjcg"))))
+    (build-system cmake-build-system)
+    (arguments (list #:tests? #f))           ;no upstream tests
+    (native-inputs (list pkg-config qttools))           ; for lrelease
+    (inputs (list hunspell qtsvg qtx11extras qtbase-5))
+    (synopsis "GUI Plain-text editor")
+    (description "FeatherPad is a GUI plain-text editor.
+It is independent of any desktop environment and has
+syntax highlighting, session management, side-pane mode, auto-saving,
+spell checking, drag and drop support, instant highlighting
+and automatic detection of text encoding..")
+    (license license:gpl3+)))
+
 (define-public texmacs
   (package
     (name "texmacs")
-- 
2.32.1 (Apple Git-133)


[-- Attachment #3: Type: text/plain, Size: 2808 bytes --]



> 30 июля 2022 г., в 16:00, Maxime Devos <maximedevos@telenet.be> написал(а):
> 
> 
> 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>