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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#64490: closed ([PATCH] gnu: Add gnome-shell-extension-vitals.)
Date: Sat, 08 Jul 2023 04:52:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sat, 08 Jul 2023 00:51:45 -0400
with message-id <87lefrxaxq.fsf <at> gmail.com>
and subject line Re: [bug#64490] [PATCH] gnu: Add gnome-shell-extension-vitals.
has caused the debbugs.gnu.org bug report #64490,
regarding [PATCH] gnu: Add gnome-shell-extension-vitals.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
64490: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=64490
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Eidvilas Markevičius <markeviciuseidvilas <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Eidvilas Markevičius <markeviciuseidvilas <at> gmail.com>
Subject: [PATCH] gnu: Add gnome-shell-extension-vitals.
Date: Thu,  6 Jul 2023 12:10:01 +0300

* 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 <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")))
+                           (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.")
+    (license license:gpl2)))
+
 (define-public arc-theme
   (package
     (name "arc-theme")

base-commit: 67c276a870b9d6be69c2a9e867683e58928c38ef
-- 
2.40.1

[Message part 3 (message/rfc822, inline)]

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.