Subject: bug#64490: closed (Re: [bug#64490] [PATCH] gnu: Add gnome-shell-extension-vitals.)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 08 Jul 2023 04:52:02 +0000

Your bug report

#64490: [PATCH] gnu: Add gnome-shell-extension-vitals.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
64490: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=64490
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [bug#64490] [PATCH] gnu: Add gnome-shell-extension-vitals.
From: Maxim Cournoyer <[email-masked]>
To: Eidvilas Markevičius <[email-masked]>
Cc: Raghav Gururajan <[email-masked]>, Liliana Marie Prikler <[email-masked]>, [email-masked]
Date: Sat, 08 Jul 2023 00:51:45 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello,

Eidvilas Markevičius <[email-masked]> 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 <[email-masked]>
>  ;;; Copyright © 2022 Sughosha <[email-masked]>
>  ;;; Copyright © 2022 Denis 'GNUtoo' Carikli <[email-masked]>
> +;;; Copyright © 2023 Eidvilas Markevičius <[email-masked]>
>  ;;;
>  ;;; 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/[email-masked]"
> +          #: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

Subject: [PATCH] gnu: Add gnome-shell-extension-vitals.
From: Eidvilas Markevičius <[email-masked]>
To: [email-masked]
Cc: Eidvilas Markevičius <[email-masked]>
Date: Thu, 6 Jul 2023 12:10:01 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

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

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 <[email-masked]>
 ;;; Copyright © 2022 Sughosha <[email-masked]>
 ;;; Copyright © 2022 Denis 'GNUtoo' Carikli <[email-masked]>
+;;; Copyright © 2023 Eidvilas Markevičius <[email-masked]>
 ;;;
 ;;; 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")))
+                           (invoke "msgfmt" "-c" file "-o" destfile)))
+               (find-files "locale" "\\.po$")))))
+       #:install-plan
+       '(("." "share/gnome-shell/extensions/[email-masked]"
+          #: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.")
+    (license license:gpl2)))
+
 (define-public arc-theme
   (package
     (name "arc-theme")

base-commit: 67c276a870b9d6be69c2a9e867683e58928c38ef
-- 
2.40.1