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

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 64490 in the body.
You can then email your comments to 64490 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

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 #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

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

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

Am Donnerstag, dem 06.07.2023 um 12:10 +0300 schrieb Eidvilas
Markevičius:
> * 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))
Inputs should go below arguments.  I'm also curious as to how vitals
ingests libgtop.  Being Javascript, I'd assume you'd need a wrapper.
> +    (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)))
I think you should just replace the end of the file name, which is a
little more complicated, but saves you the #:modules blurb.
> +               (find-files "locale" "\\.po$")))))
> +       #:install-plan
> +       '(("." "share/gnome-shell/extensions/Vitals <at> CoreCoding.com"
> +          #:include-regexp
> +          ("\\.js(on)?$"
> +           "\\.css$"
> +           "\\.ui$"
> +           "\\.svg$"
> +           "\\.xml$"
> +           "\\.mo$"
> +           "\\.compiled$")))))
Preferably, install plans go before phases.
> +    (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)))
No +?

Cheers

Previous Next

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