GNU bug report logs - #64490
[PATCH] gnu: Add gnome-shell-extension-vitals.

Previous Next

Full log

Message #10 received at 64490-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Eidvilas Markevičius <markeviciuseidvilas <at> gmail.com>
Cc: Raghav Gururajan <rg <at> raghavgururajan.name>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>, 64490-done <at> debbugs.gnu.org
Subject: Re: [bug#64490] [PATCH] gnu: Add gnome-shell-extension-vitals.
Date: Sat, 08 Jul 2023 00:51:45 -0400

Hello,

Eidvilas Markevičius <markeviciuseidvilas <at> gmail.com> writes:

> * gnu/packages/gnome-xyz.scm (gnome-shell-extension-vitals): New variable.
> ---
>  gnu/packages/gnome-xyz.scm | 56 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 56 insertions(+)

Looks great!  Thanks for reaching out to the GNOME team.  I've installed
it (see commit c0833f3b0c), after making the changes detailed below.

> diff --git a/gnu/packages/gnome-xyz.scm b/gnu/packages/gnome-xyz.scm
> index 4ce102d8ed..b2652d422f 100644
> --- a/gnu/packages/gnome-xyz.scm
> +++ b/gnu/packages/gnome-xyz.scm
> @@ -19,6 +19,7 @@
>  ;;; Copyright © 2022 Eric Bavier <bavier <at> posteo.net>
>  ;;; Copyright © 2022 Sughosha <sughosha <at> proton.me>
>  ;;; Copyright © 2022 Denis 'GNUtoo' Carikli <GNUtoo <at> cyberdimension.org>
> +;;; Copyright © 2023 Eidvilas Markevičius <markeviciuseidvilas <at> gmail.com>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -1328,6 +1329,61 @@ (define-public gnome-shell-extension-transparent-window
>  of windows.")
>        (license license:expat))))
>  
> +(define-public gnome-shell-extension-vitals
> +  (package
> +    (name "gnome-shell-extension-vitals")
> +    (version "62.0.0")
> +    (source
> +      (origin
> +        (method git-fetch)
> +        (uri (git-reference
> +               (url "https://github.com/corecoding/Vitals")
> +               (commit (string-append "v" version))))
> +        (file-name (git-file-name name version))
> +        (sha256 (base32 "0wmw5yd38vyv13x6frbafp21bdhlyjd5ggimdf2696irfhnm828h"))
> +        (modules '((guix build utils)))
> +        (snippet
> +          '(begin
> +             (delete-file "schemas/gschemas.compiled")
> +             (for-each delete-file
> +               (find-files "locale" "\\.mo$"))))))
> +    (build-system copy-build-system)
> +    (native-inputs (list `(,glib "bin") gettext-minimal))
> +    (inputs (list libgtop))
> +    (arguments
> +     '(#:modules
> +       ((guix build copy-build-system)
> +        (guix build utils)
> +        (ice-9 string-fun))
> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-before 'install 'compile-schemas
> +           (lambda _
> +             (invoke "glib-compile-schemas" "--strict" "schemas")))
> +         (add-before 'install 'compile-locales
> +           (lambda _
> +             (for-each (lambda (file)
> +                         (let ((destfile (string-replace-substring file ".po" ".mo")))

This line looks too long (our coding style uses a max column of 80
chars).  I've run 'guix style' on the package, and it re-flowed the
arguments.

> +                           (invoke "msgfmt" "-c" file "-o" destfile)))
> +               (find-files "locale" "\\.po$")))))
> +       #:install-plan
> +       '(("." "share/gnome-shell/extensions/Vitals <at> CoreCoding.com"
> +          #:include-regexp
> +          ("\\.js(on)?$"
> +           "\\.css$"
> +           "\\.ui$"
> +           "\\.svg$"
> +           "\\.xml$"
> +           "\\.mo$"
> +           "\\.compiled$")))))
> +    (home-page "https://github.com/corecoding/Vitals")
> +    (synopsis "GNOME Shell extension for monitoring your computer's sensors' stats")
> +    (description "Vitals is a GNOME Shell extension that gives you a glimpse into your
> +computer's temperature, voltage, fan speed, memory usage and CPU load, right from the top
> +menu bar of the GNOME Shell.  It is a one stop shop for monitoring all the vital sensors
> +of your computer.")

I've taken the liberty to streamline the synopsis and description, using
a more descriptive tone and stripping the last sentence, which amounts
more to marketing than a factual description :-).

> +    (license license:gpl2)))

This is gpl2+, with the '+' denoting the "version 2 or later" clause,
preserved in the GPL license text (thus effective).

-- 
Thanks,
Maxim

Previous Next

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