[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#40756] gnu: Add ktsuss.

From: Nicolas Goaziou
Subject: [bug#40756] gnu: Add ktsuss.
Date: Thu, 23 Apr 2020 10:17:14 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux)


Raghav Gururajan <address@hidden> writes:

> Subject: [PATCH 4/4] gnu: Add ktsuss.

Thank you! 

It looks mostly good. I have some comments, tho.

> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'patch

Could you provide a more insightful name for the new phase? :)

> +           (lambda _
> +             (substitute* ""
> +               (("sudopath=`which sudo 2>/dev/null`")
> +                (string-append "sudopath="
> +                               (string-append (assoc-ref %build-inputs 
> "sudo")
> +                                              "/bin/sudo"))))

Please use (lambda* (#:key inputs #:allow-other-keys) ...) and `inputs'
instead of the magical %build-inputs.

> +    (synopsis "Graphical front-end for Su")

I would write:

  Graphical front end for @command{su}

> +    (description "Ktsuss stands for 'keep the su simple, stupid', and as the
> +name says, is a graphical version (frontend) of su written in C and GTK+ 2.  
> The
> +idea of the project is to remain simple and bug free.")

What about :

     "Ktsuss stands for ``Keep the @command{su} simple, stupid''.
It is a graphical version of @command{su} written in C and GTK+ 2,
with simplicity in mind.")

> +    (license
> +     (list
> +      (license:non-copyleft "file://COPYING")
> +      (license:non-copyleft "file://LICENSE")))))

It looks like the license is `bsd-3'. Do you have any reason to think
otherwise ?

Could you send an updated patch?


Nicolas Goaziou

reply via email to

[Prev in Thread] Current Thread [Next in Thread]