GNU logs - #53376, boring messages

Message sent to guix-patches@HIDDEN:

Subject: [bug#53376] [PATCH] gnu: Add fan2go
Resent-From: Elais Player <elais@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: guix-patches@HIDDEN
Resent-Date: Wed, 19 Jan 2022 23:19:02 +0000
Resent-Message-ID: <handler.53376.B.164263430525410 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: 53376 <at> debbugs.gnu.org
From: Elais Player <elais@HIDDEN>
Date: Wed, 19 Jan 2022 15:16:11 -0700
User-agent: mu4e 1.6.10; emacs 28.0.50
Message-ID: <877davuziz.fsf@HIDDEN>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
Received-SPF: pass client-ip=66.111.4.26; envelope-from=elais@HIDDEN;
 helo=out2-smtp.messagingengine.com
X-Spam_score_int: -27
X-Spam_score: -2.8
X-Spam_bar: --
X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
 DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001,
 RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001,
 SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no
X-Spam_action: no action
X-Spam-Score: -0.4 (/)
X-Mailman-Approved-At: Wed, 19 Jan 2022 18:18:21 -0500
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -2.6 (--)

--=-=-=
Content-Type: text/plain
Content-Disposition: inline

I have a couple of patches adding the fan2go package and its dependencies.

Elais Player (1):
  gnu: Add fan2go

gnu/packages/golang.scm   | 304 ++++++++++++++++++++++++++++++++++++++
gnu/packages/hardware.scm |  48 ++++++
2 files changed, 352 insertions(+)


base-commit: a27e47f9d1e22dc32bb250cfeef88cfacb930e23
--
2.34.0

--=-=-=
Content-Type: text/x-patch; charset=utf-8
Content-Disposition: attachment; filename=0001-gnu-Add-fan2go.patch
Content-Transfer-Encoding: quoted-printable

From c3117f446312f456087719f746b2793a1982f6a8 Mon Sep 17 00:00:00 2001
From: Elais Player <elais@HIDDEN>
Date: Wed, 19 Jan 2022 13:04:47 -0700
Subject: [PATCH] gnu: Add fan2go
To: guix-patches@HIDDEN

* gnu/packages/golang.scm (go-github-com-atomicgo-cursor): New variable

* gnu/packages/golang.scm (go-github-com-gookit-color): New variable

* /gnu/packages/golang.scm (go-github-com-pterm-pterm): New variable

* /gnu/packages/golang.scm (go-github-com-marvinjwendt-testza): New variable

* /gnu/packages/golang.scm (go-github-com-xo-terminfo): New variable.

* /gnu/packages/golang.scm (go-github-com-klauspost-cpuid-v2): New variable.

* /gnu/packages/golang.scm (go-github-com-asecurityteam-rolling): New varia=
ble.

* /gnu/packages/golang.scm (go-github-com-guptarohit-asciigraph): New varia=
ble.

* /gnu/packages/golang.scm (go-github-com-oklog-run): New variable.

* /gnu/packages/golang.scm (go-github-com-tomlazar-table): New variable.

* /gnu/packages/golang.scm (go-github-com-looplab-tarjan): New variable.

* /gnu/packages/golang.scm (go-github-com-md14454-gosensors): New variable.

* /gnu/packages/hardware.scm (fan2go): New variable.
---
 gnu/packages/golang.scm   | 304 ++++++++++++++++++++++++++++++++++++++
 gnu/packages/hardware.scm |  48 ++++++
 2 files changed, 352 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index e808d8ebf6..9b20cfb413 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -33,6 +33,7 @@
 ;;; Copyright =C2=A9 2021 Chadwain Holness <chadwainholness@HIDDEN>
 ;;; Copyright =C2=A9 2021 Philip McGrath <philip@HIDDEN>
 ;;; Copyright =C2=A9 2021 Lu Hui <luhux76@HIDDEN>
+;;; Copyright =C2=A9 2022 Elais Player <elais@HIDDEN>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -67,6 +68,7 @@ (define-module (gnu packages golang)
   #:use-module (gnu packages gcc)
   #:use-module (gnu packages glib)
   #:use-module (gnu packages gnupg)
+  #:use-module (gnu packages linux)
   #:use-module (gnu packages lua)
   #:use-module (gnu packages mp3)
   #:use-module (gnu packages password-utils)
@@ -9312,6 +9314,308 @@ (define-public go-github-com-alecthomas-units
       (home-page "https://github.com/alecthomas/units")
       (license license:expat))))
=20
+(define-public go-github-com-atomicgo-cursor
+  (package
+    (name "go-github-com-atomicgo-cursor")
+    (version "0.0.1")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/atomicgo/cursor")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "1mjgx9gsgda980pb5jrqwq64r7wrh7083849jjww40l30vjaqvh7"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/atomicgo/cursor"))
+    (home-page "https://github.com/atomicgo/cursor")
+    (synopsis "Description")
+    (description "Package cursor contains cross-platform methods to move t=
he
+terminal cursor in different directions.  This package can be used to crea=
te
+interactive CLI tools and games, live charts, algorithm visualizations and
+other updatable output of any kind.")
+    (license license:expat)))
+
+(define-public go-github-com-gookit-color
+  (package
+    (name "go-github-com-gookit-color")
+    (version "1.5.0")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/gookit/color")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "1cyhcnk5sknn2kxz8ffy6iy2ncdpwbgvdywj920fd7745jas3m1n"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/gookit/color"
+                 #:tests? #f))
+    (inputs
+     `(("go-github-com-stretchr-testify" ,go-github-com-stretchr-testify)))
+    (propagated-inputs
+      `(("go-golang-org-x-sys" ,go-golang-org-x-sys)
+        ("go-github-com-xo-terminfo" ,go-github-com-xo-terminfo)))
+    (home-page "https://github.com/gookit/color")
+    (synopsis "Command line color library")
+    (description "Package color is Command line color library.  Support ri=
ch
+color rendering output, universal API method, compatible with Windows syst=
em
+")
+    (license license:expat)))
+
+(define-public go-github-com-pterm-pterm
+  (package
+    (name "go-github-com-pterm-pterm")
+    (version "0.12.34")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/pterm/pterm")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "1spc1v4xwm3vcrfyc3wjpswjx5klg2qq9ipgjj8qncn8czszwx93"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/pterm/pterm"
+                 #:tests? #f))
+    (propagated-inputs
+      `(("go-golang-org-x-term" ,go-golang-org-x-term)
+        ("go-golang-org-x-sys" ,go-golang-org-x-sys)
+        ("go-github-com-mattn-go-runewidth" ,go-github-com-mattn-go-runewi=
dth)
+        ("go-github-com-gookit-color" ,go-github-com-gookit-color)
+        ("go-github-com-atomicgo-cursor" ,go-github-com-atomicgo-cursor)))
+    (home-page "https://github.com/pterm/pterm")
+    (synopsis "Console output beautifier")
+    (description "Pterm is a modern go module to beautify console output. =
 It
+can be used without configuration, but if desired, everything can be
+customized down to the smallest detail.")
+    (license license:expat)))
+
+(define-public go-github-com-marvinjwendt-testza
+  (package
+    (name "go-github-com-marvinjwendt-testza")
+    (version "0.2.14")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/MarvinJWendt/testza")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "0fw7458yan99jfyvn46bhyc18x0dz3dz2kd6sffz9cj38rh5kym3"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/MarvinJWendt/testza"
+                 #:tests? #f))
+    (inputs
+     `(("go-github-com-pterm-pterm" ,go-github-com-pterm-pterm)))
+    (propagated-inputs
+      `(("go-github-com-pmezard-go-difflib" ,go-github-com-pmezard-go-diff=
lib)
+        ("go-github-com-klauspost-cpuid-v2" ,go-github-com-klauspost-cpuid=
-v2)
+        ("go-github-com-davecgh-go-spew" ,go-github-com-davecgh-go-spew)))
+    (home-page "https://github.com/MarvinJWendt/testza")
+    (synopsis "Testing framework for go")
+    (description "Package testza is a full-featured testing framework for =
Go.
+It integrates with the default test runner, so you can use it with the
+standard `go test` tool.  Testza contains easy to use methods, like
+assertions, output capturing, mocking, and much more.")
+    (license license:expat)))
+
+(define-public go-github-com-xo-terminfo
+  (package
+    (name "go-github-com-xo-terminfo")
+    (version "0.0.0-20210125001918-ca9a967f8778")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/xo/terminfo")
+               (commit (go-version->git-ref version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "05gdcvcbwcrcwxznhvs1q1xh4irz2d10v2mz179pydjh30kjc0j5"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/xo/terminfo"))
+    (home-page "https://github.com/xo/terminfo")
+    (synopsis "Terminfo file reader in pure go")
+    (description "Package terminfo provides a pure-Go implementation of
+reading information from the terminfo database.")
+    (license license:expat)))
+
+(define-public go-github-com-klauspost-cpuid-v2
+  (package
+    (name "go-github-com-klauspost-cpuid-v2")
+    (version "2.0.9")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/klauspost/cpuid")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "12bx0kip3yv2416f4ilafk5674m69mn873gnadhyv473cy4jy499"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/klauspost/cpuid/v2"))
+    (home-page "https://github.com/klauspost/cpuid")
+    (synopsis "CPU feature identification for Go")
+    (description "Cpuid provides information about the CPU running the cur=
rent
+program.  CPU features are detected on startup, and kept for fast access
+through the life of the application.  Currently x86 / x64 (AMD64/i386) and=
 ARM
+(ARM64) is supported, and no external C (cgo) code is used, which should m=
ake
+the library very easy to use.")
+    (license license:expat)))
+
+(define-public go-github-com-asecurityteam-rolling
+  (package
+    (name "go-github-com-asecurityteam-rolling")
+    (version "2.2.1")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/asecurityteam/rolling")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "18m1fpq8svkys8fi08chrphr506nb6jjhlbn9h1rh5d395bi3rfz"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/asecurityteam/rolling"))
+    (home-page "https://github.com/asecurityteam/rolling")
+    (synopsis "Rolling window implementation in go")
+    (description "A rolling window implemenation in go that continuously
+overwrites the oldest value with the latest to preserve the specified value
+count.  This type of window is useful for collecting data that have a known
+interval on which they are capture or for tracking data where time is not a
+factor.")
+    (license license:asl2.0)))
+
+(define-public go-github-com-guptarohit-asciigraph
+  (package
+    (name "go-github-com-guptarohit-asciigraph")
+    (version "0.5.2")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/guptarohit/asciigraph")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "1hkxaqs30hbybbxapr8vng09r6wr8afpkqkn0g6aj1vl3fvhjn49"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/guptarohit/asciigraph"))
+    (home-page "https://github.com/guptarohit/asciigraph")
+    (synopsis "Package to make a lightweight ASCII line graph")
+    (description "Go package to make a lightweight ASCII line graph for CLI
+apps.")
+    (license license:bsd-3)))
+
+(define-public go-github-com-oklog-run
+  (package
+    (name "go-github-com-oklog-run")
+    (version "1.1.0")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/oklog/run")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "0r55p3kgdkgw55i33lqvvvl60mjp92mhd1170m980sw98z9150jk"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/oklog/run"))
+    (home-page "https://github.com/oklog/run")
+    (synopsis "Universal mechanism to manage goroutine lifecycles")
+    (description "Package run implements an actor-runner with deterministic
+teardown.  It is somewhat similar to package errgroup, except it does not
+require actor goroutines to understand context semantics.  This makes it
+suitable for use in more circumstances; for example, goroutines which are
+handling connections from net.Listeners, or scanning input from a closable
+io.Reader.")
+    (license license:asl2.0)))
+
+(define-public go-github-com-tomlazar-table
+  (package
+    (name "go-github-com-tomlazar-table")
+    (version "0.1.0")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/tomlazar/table")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "017ps5n44xvdl2b656v89qi0gib95fz60vqvnpq7kjzjn9sr3awy"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/tomlazar/table"))
+    (inputs
+     `(("go-github-com-stretchr-testify" ,go-github-com-stretchr-testify)))
+    (propagated-inputs
+      `(("go-github-com-mgutz-ansi" ,go-github-com-mgutz-ansi)
+        ("go-github-com-mattn-go-isatty" ,go-github-com-mattn-go-isatty)
+        ("go-github-com-mattn-go-colorable"
+         ,go-github-com-mattn-go-colorable)))
+    (home-page "https://github.com/tomlazar/table")
+    (synopsis "Colorful tables in go with less effort")
+    (description "Print tabular data on the command line using ansi color
+esacape codes.  Support for writing the ouput based on the fields in a str=
uct
+and for defining and creating the table manully using the underlying objec=
t.")
+    (license license:expat)))
+
+(define-public go-github-com-looplab-tarjan
+  (package
+    (name "go-github-com-looplab-tarjan")
+    (version "0.1.0")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/looplab/tarjan")
+               (commit (string-append "v" version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "0km1qzkw1c0gl0fjkmcaibirxzk35z9f44nspk46vbjfgcm78z58"))=
))
+    (build-system go-build-system)
+    (arguments '(#:import-path "github.com/looplab/tarjan"))
+    (home-page "https://github.com/looplab/tarjan")
+    (synopsis "Graph loop detection using Tarjan's algorithm")
+    (description "Tarjan is a graph loop detection function using Tarjan's
+algorithm.  The algorithm takes a input graph and produces a slice where e=
ach
+item is a slice of strongly connected vertices.  The input graph is in for=
m of
+a map where the key is a graph vertex and the value is the edges in for of=
 a
+slice of vertices.")
+    (license license:asl2.0)))
+
+(define-public go-github-com-md14454-gosensors
+  (package
+    (name "go-github-com-md14454-gosensors")
+    (version "0.0.0-20180726083412-bded752ab001")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url "https://github.com/md14454/gosensors")
+               (commit (go-version->git-ref version))))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "08ffz56h93rcsyp84zxirdj3azb9rkql7sc0dj22pp17w11qlwhx"))=
))
+    (build-system go-build-system)
+    (propagated-inputs
+     `(("lm-sensors:lib" ,lm-sensors "lib")))
+    (arguments '(#:import-path "github.com/md14454/gosensors"
+                 #:tests? #f))
+    (home-page "https://github.com/md14454/gosensors")
+    (synopsis "Go bindings for libsensors")
+    (description "Go bindings for libsensors.so from the lm-sensors project
+via external C bindings.")
+    (license license:expat)))
+
 (define-public go-github-com-dreamacro-go-shadowsocks2
   (package
     (name "go-github-com-dreamacro-go-shadowsocks2")
diff --git a/gnu/packages/hardware.scm b/gnu/packages/hardware.scm
index 8727f0f20a..3acad0cfc5 100644
--- a/gnu/packages/hardware.scm
+++ b/gnu/packages/hardware.scm
@@ -9,6 +9,7 @@
 ;;; Copyright =C2=A9 2021 Raghav Gururajan <rg@HIDDEN>
 ;;; Copyright =C2=A9 2021 Vinicius Monego <monego@HIDDEN>
 ;;; Copyright =C2=A9 2021, 2022 John Kehayias <john.kehayias@HIDDEN=
m>
+;;; Copyright =C2=A9 2022 Elais Player <elais@HIDDEN>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -40,6 +41,7 @@ (define-module (gnu packages hardware)
   #:use-module (gnu packages gcc)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
+  #:use-module (gnu packages golang)
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages libusb)
   #:use-module (gnu packages linux)
@@ -54,6 +56,7 @@ (define-module (gnu packages hardware)
   #:use-module (gnu packages python-web)
   #:use-module (gnu packages python-xyz)
   #:use-module (gnu packages qt)
+  #:use-module (gnu packages syncthing)
   #:use-module (gnu packages tls)
   #:use-module (gnu packages virtualization)
   #:use-module (gnu packages web)
@@ -62,6 +65,7 @@ (define-module (gnu packages hardware)
   #:use-module (gnu packages xorg)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system go)
   #:use-module (guix build-system meson)
   #:use-module (guix build-system python)
   #:use-module (guix download)
@@ -928,6 +932,50 @@ (define-public screentest
     (home-page "https://github.com/TobiX/screentest")
     (license license:gpl2)))
=20
+(define-public fan2go
+  (package
+    (name "fan2go")
+    (version "0.4.0")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/markusressel/fan2go")
+                    (commit version)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "12mrkpzi3bwxwbhzm7l4hlpb8jna89p9ywaf3nf82739jallf2fy"))))
+    (build-system go-build-system)
+    (arguments
+     '(#:import-path "github.com/markusressel/fan2go"
+       #:tests? #f))
+    (inputs
+     `(("lm-sensors:lib" ,lm-sensors "lib")))
+    (propagated-inputs
+     `(("go-etcd-io-bbolt" ,go-etcd-io-bbolt)
+       ("go-github-com-tomlazar-table" ,go-github-com-tomlazar-table)
+       ("go-github-com-spf13-viper" ,go-github-com-spf13-viper)
+       ("go-github-com-spf13-cobra" ,go-github-com-spf13-cobra)
+       ("go-github-com-pterm-pterm" ,go-github-com-pterm-pterm)
+       ("go-github-com-prometheus-client-golang"
+        ,go-github-com-prometheus-client-golang)
+       ("go-github-com-oklog-run" ,go-github-com-oklog-run)
+       ("go-github-com-mitchellh-go-homedir" ,go-github-com-mitchellh-go-h=
omedir)
+       ("go-github-com-mgutz-ansi" ,go-github-com-mgutz-ansi)
+       ("go-github-com-md14454-gosensors" ,go-github-com-md14454-gosensors)
+       ("go-github-com-mattn-go-colorable" ,go-github-com-mattn-go-colorab=
le)
+       ("go-github-com-looplab-tarjan" ,go-github-com-looplab-tarjan)
+       ("go-github-com-guptarohit-asciigraph"
+        ,go-github-com-guptarohit-asciigraph)
+       ("go-github-com-asecurityteam-rolling"
+        ,go-github-com-asecurityteam-rolling)))
+    (home-page "https://github.com/markusressel/fan2go")
+    (synopsis "Fan speed control daemon based on reading temperature senso=
rs")
+    (description "fan2go is a simple fan control daemon that relies on
+lm-sensors to get both temperature and RPM sensor readings, as well as PWM
+controls, so you will have to set it up first.")
+    (license license:agpl3)))
+
 (define-public tpm2-tss
   (package
     (name "tpm2-tss")

base-commit: a27e47f9d1e22dc32bb250cfeef88cfacb930e23
--=20
2.34.0


--=-=-=--

Message sent:

Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Mailer: MIME-tools 5.505 (Entity 5.505)
Content-Type: text/plain; charset=utf-8
X-Loop: help-debbugs@HIDDEN
From: help-debbugs@HIDDEN (GNU bug Tracking System)
To: Elais Player <elais@HIDDEN>
Subject: bug#53376: Acknowledgement ([PATCH] gnu: Add fan2go)
Message-ID: <handler.53376.B.164263430525410.ack <at> debbugs.gnu.org>
References: <877davuziz.fsf@HIDDEN>
X-Gnu-PR-Message: ack 53376
X-Gnu-PR-Package: guix-patches
X-Gnu-PR-Keywords: patch
Reply-To: 53376 <at> debbugs.gnu.org
Date: Wed, 19 Jan 2022 23:19:02 +0000

Thank you for filing a new bug report with debbugs.gnu.org.

This is an automatically generated reply to let you know your message
has been received.

Your message is being forwarded to the package maintainers and other
interested parties for their attention; they will reply in due course.

Your message has been sent to the package maintainer(s):
 guix-patches@HIDDEN

If you wish to submit further information on this problem, please
send it to 53376 <at> debbugs.gnu.org.

Please do not send mail to help-debbugs@HIDDEN unless you wish
to report a problem with the Bug-tracking system.

--=20
53376: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3D53376
GNU Bug Tracking System
Contact help-debbugs@HIDDEN with problems

Message sent to guix-patches@HIDDEN:

Subject: [bug#53376] [PATCH] gnu: Add fan2go
Resent-From: Maxime Devos <maximedevos@HIDDEN>
Original-Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
Resent-CC: guix-patches@HIDDEN
Resent-Date: Thu, 20 Jan 2022 10:04:02 +0000
Resent-Message-ID: <handler.53376.B53376.164267299120620 <at> debbugs.gnu.org>
Resent-Sender: help-debbugs@HIDDEN
To: Elais Player <elais@HIDDEN>, 53376 <at> debbugs.gnu.org
Message-ID: <e20baf5585f95ecfdfe6c391757bc4c4fd126206.camel@HIDDEN>
From: Maxime Devos <maximedevos@HIDDEN>
Date: Thu, 20 Jan 2022 11:03:01 +0100
In-Reply-To: <877davuziz.fsf@HIDDEN>
References: <877davuziz.fsf@HIDDEN>
Content-Type: multipart/signed; micalg="pgp-sha512";
 protocol="application/pgp-signature"; boundary="=-bQ7XFUmtNN/dZs9HWSEA"
User-Agent: Evolution 3.38.3-1 
MIME-Version: 1.0
Precedence: list
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>


--=-bQ7XFUmtNN/dZs9HWSEA
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi,

I took a quick look at the synopsi and descriptions:

Elais Player schreef op wo 19-01-2022 om 15:16 [-0700]:
> + (define-public go-github-com-atomicgo-cursor
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (synopsis "Description")

A synopsis is missing.

> +=C2=A0=C2=A0=C2=A0 (description "Package cursor contains cross-platform =
methods to
> move the
> +terminal cursor in different directions.=C2=A0

This doesn't seem grammatical, how about =E2=80=98This package contains [..=
.]=E2=80=99?

> =C2=A0This package can be used to create
> +interactive CLI tools and games, live charts, algorithm
> visualizations and
> +other updatable output of any kind.")
> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +
> +(define-public go-github-com-gookit-color
> +=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (home-page "https://github.com/gookit/color")
> +=C2=A0=C2=A0=C2=A0 (synopsis "Command line color library")

Is it a command line tool, or a library?  It cannot be both
(with some exceptions, e.g. glibc can be run).=20

How about: =E2=80=98Library for setting colors in the terminal=E2=80=99.=
=20

> +=C2=A0=C2=A0=C2=A0 (description "Package color is Command line color lib=
rary.

Likewise (grammar), and why is =E2=80=98Command=E2=80=99 capitalised?
How about: =E2=80=98This package allows setting the colours used in the
terminal ...=E2=80=99

> =C2=A0 Support rich
> +color rendering output, universal API method, compatible with
> Windows system

Likewise (grammar), the imperative doesn't make much sense here

> +")
> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +
> +(define-public go-github-com-pterm-pterm
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 ]...]
> +=C2=A0=C2=A0=C2=A0 (synopsis "Console output beautifier")
> +=C2=A0=C2=A0=C2=A0 (description "Pterm is a modern go module to beautify=
 console
> output.=C2=A0 It
> +can be used without configuration, but if desired, everything can be
> +customized down to the smallest detail.")

=E2=80=98Modern, smallest detail=E2=80=99: this is getting a bit close to m=
arketing
phrases (see (guix)Synopses and Descriptions), although it is rather
mild here.

> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +
> +(define-public go-github-com-marvinjwendt-testza
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (synopsis "Testing framework for go")
> +=C2=A0=C2=A0=C2=A0 (description "Package testza is a full-featured testi=
ng
> framework for Go.

In the synopsis 'go' is not capitalised, while in the description, it
is.  There is some inconsistency here.

> +It integrates with the default test runner, so you can use it with
> the
> +standard `go test` tool.

Guix doesn't use markdown, it uses Texinfo, so IIUC this needs to be
@command{go test}.  Correct markup helps with

> =C2=A0 Testza contains easy to use methods, like
> +assertions, output capturing, mocking, and much more.")
> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +
> +(define-public go-github-com-xo-terminfo
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (synopsis "Terminfo file reader in pure go")
> +=C2=A0=C2=A0=C2=A0 (description "Package terminfo provides a pure-Go imp=
lementation
> of
> +reading information from the terminfo database.")

Go is capitalised inconsistently.  Also, IIUC the proposition needs to
be 'for' and not 'of'.

> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +
> +(define-public go-github-com-klauspost-cpuid-v2
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (synopsis "CPU feature identification for Go")
> +=C2=A0=C2=A0=C2=A0 (description "Cpuid provides information about the CP=
U running
> the current
> +program.=C2=A0 CPU features are detected on startup, and kept for fast

I think =E2=80=98kept=E2=80=99 -> =E2=80=98remembered=E2=80=99 would be cle=
arer here (YMMV).

> access
> +through the life of the application.=C2=A0 Currently x86 / x64
> (AMD64/i386) and ARM
> +(ARM64) is supported, and no external C (cgo) code is used, which
> should make
> +the library very easy to use.")

AFAIK external C code works just fine=C2=A0in Guix without any complication=
s
so this remark doesn't seem necessary in Guix.  But then I never
actually write any Go code, so take it with some grain of salt.


> +=C2=A0=C2=A0=C2=A0 (license license:expat)))

Since not all architectures are supported, I recommend setting
'supported-systems'.

> +[...]
> +
> +(define-public go-github-com-guptarohit-asciigraph
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 (name "go-github-com-guptarohit-asciigraph")
> +=C2=A0=C2=A0=C2=A0 (version "0.5.2")
> +=C2=A0=C2=A0=C2=A0 (source
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (origin
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (method git-fetch)
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (uri (git-reference
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (url "https://github.com/guptarohit/asciigraph")
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (commit (string-append "v" version))))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (file-name (git-file-name nam=
e version))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (sha256
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (base32
> "1hkxaqs30hbybbxapr8vng09r6wr8afpkqkn0g6aj1vl3fvhjn49"))))
> +=C2=A0=C2=A0=C2=A0 (build-system go-build-system)
> +=C2=A0=C2=A0=C2=A0 (arguments '(#:import-path "github.com/guptarohit/asc=
iigraph"))
> +=C2=A0=C2=A0=C2=A0 (home-page "https://github.com/guptarohit/asciigraph"=
)
> +=C2=A0=C2=A0=C2=A0 (synopsis "Package to make a lightweight ASCII line g=
raph")
> +=C2=A0=C2=A0=C2=A0 (description "Go package to make a lightweight ASCII =
line graph
> for CLI
> +apps.")
> +=C2=A0=C2=A0=C2=A0 (license license:bsd-3)))
> +
> +(define-public go-github-com-oklog-run
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 (name "go-github-com-oklog-run")
> +=C2=A0=C2=A0=C2=A0 (version "1.1.0")
> +=C2=A0=C2=A0=C2=A0 (source
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (origin
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (method git-fetch)
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (uri (git-reference
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (url "https://github.com/oklog/run")
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (commit (string-append "v" version))))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (file-name (git-file-name nam=
e version))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (sha256
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (base32
> "0r55p3kgdkgw55i33lqvvvl60mjp92mhd1170m980sw98z9150jk"))))
> +=C2=A0=C2=A0=C2=A0 (build-system go-build-system)
> +=C2=A0=C2=A0=C2=A0 (arguments '(#:import-path "github.com/oklog/run"))
> +=C2=A0=C2=A0=C2=A0 (home-page "https://github.com/oklog/run")
> +=C2=A0=C2=A0=C2=A0 (synopsis "Universal mechanism to manage goroutine li=
fecycles")
> +=C2=A0=C2=A0=C2=A0 (description "Package run implements an actor-runner =
with

=E2=80=98Package run=E2=80=99 -> =E2=80=98This package=E2=80=99?
> deterministic
> +teardown.=C2=A0 It is somewhat similar to package errgroup, except it

=E2=80=98package errgroup=E2=80=99 -> @code{errgroup}?

> does not
> +require actor goroutines to understand context semantics.=C2=A0 This
> makes it
> +suitable for use in more circumstances; for example, goroutines
> which are
> +handling connections from net.Listeners, or scanning input from a
> closable
> +io.Reader.")

Texinfo markup (@code{...}) for io.Reader and net.Listeners.

> +=C2=A0=C2=A0=C2=A0 (license license:asl2.0)))
> +
> +(define-public go-github-com-tomlazar-table
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 (name "go-github-com-tomlazar-table")
> +=C2=A0=C2=A0=C2=A0 (version "0.1.0")
> +=C2=A0=C2=A0=C2=A0 (source
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (origin
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (method git-fetch)
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (uri (git-reference
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (url "https://github.com/tomlazar/table")
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (commit (string-append "v" version))))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (file-name (git-file-name nam=
e version))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (sha256
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (base32
> "017ps5n44xvdl2b656v89qi0gib95fz60vqvnpq7kjzjn9sr3awy"))))
> +=C2=A0=C2=A0=C2=A0 (build-system go-build-system)
> +=C2=A0=C2=A0=C2=A0 (arguments '(#:import-path "github.com/tomlazar/table=
"))
> +=C2=A0=C2=A0=C2=A0 (inputs
> +=C2=A0=C2=A0=C2=A0=C2=A0 `(("go-github-com-stretchr-testify" ,go-github-=
com-stretchr-
> testify)))
> +=C2=A0=C2=A0=C2=A0 (propagated-inputs
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 `(("go-github-com-mgutz-ansi" ,go-github-=
com-mgutz-ansi)
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ("go-github-com-mattn-go-isat=
ty" ,go-github-com-mattn-go-
> isatty)
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ("go-github-com-mattn-go-colo=
rable"
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ,go-github-com-mattn-go=
-colorable)))
> +=C2=A0=C2=A0=C2=A0 (home-page "https://github.com/tomlazar/table")
> +=C2=A0=C2=A0=C2=A0 (synopsis "Colorful tables in go with less effort")
> +=C2=A0=C2=A0=C2=A0 (description "Print tabular data on the command line =
using ansi
> color
> +esacape codes.

The imperative doesn't make much sense here.  Also, a typo: =E2=80=98esacap=
a=E2=80=99.
And 'ansi' probably needs to be capitalised to =E2=80=98ANSI=E2=80=99

> =C2=A0 Support for writing the ouput based on the fields in a struct

Likewise (imperative).

> +and for defining and creating the table manully using the underlying
> object.")
> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +
> +(define-public go-github-com-looplab-tarjan
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 [..]
> +=C2=A0=C2=A0=C2=A0 (synopsis "Graph loop detection using Tarjan's algori=
thm")
> +=C2=A0=C2=A0=C2=A0 (description "Tarjan is a graph loop detection functi=
on using
> Tarjan's
> +algorithm.=C2=A0 The algorithm takes a input graph and produces a slice

IIUC, the standard graph theory terminology would be =E2=80=98... and produ=
ces
a list of the strongly connected components=E2=80=99.  AFAIK =E2=80=98slice=
=E2=80=99 is not
used in gaph theory.

> where each
> +item is a slice of strongly connected vertices.=C2=A0 The input graph is
> in form of
> +a map where the key is a graph vertex and the value is the edges in
> for of a
> +slice of vertices.")

Aside from that,, it's a nice description.

> +=C2=A0=C2=A0=C2=A0 (license license:asl2.0)))
> +
> +(define-public go-github-com-md14454-gosensors
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 (name "go-github-com-md14454-gosensors")
> +=C2=A0=C2=A0=C2=A0 (version "0.0.0-20180726083412-bded752ab001")
> +=C2=A0=C2=A0=C2=A0 (source
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (origin
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (method git-fetch)
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (uri (git-reference
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (url "https://github.com/md14454/gosensors")
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 (commit (go-version->git-ref version))))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (file-name (git-file-name nam=
e version))
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (sha256
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (base32
> "08ffz56h93rcsyp84zxirdj3azb9rkql7sc0dj22pp17w11qlwhx"))))
> +=C2=A0=C2=A0=C2=A0 (build-system go-build-system)
> +=C2=A0=C2=A0=C2=A0 (propagated-inputs
> +=C2=A0=C2=A0=C2=A0=C2=A0 `(("lm-sensors:lib" ,lm-sensors "lib")))

Is propagation necessary, or would 'inputs' suffice?
Propagation can cause profile collisions and in some cases make profile
building slow, and hence is to be avoided.

> +=C2=A0=C2=A0=C2=A0 (arguments '(#:import-path "github.com/md14454/gosens=
ors"
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0 #:tests? #f))

Why are tests disabled?  If tests fail, that can mean the tests
function as intended because they might be detecting a real issue.

> +=C2=A0=C2=A0=C2=A0 (home-page "https://github.com/md14454/gosensors")
> +=C2=A0=C2=A0=C2=A0 (synopsis "Go bindings for libsensors")
> +=C2=A0=C2=A0=C2=A0 (description "Go bindings for libsensors.so from the =
lm-sensors

The .so seems to be an implementation detail,
=E2=80=98Go bindings for @code{lm-sensors}=E2=80=99?

> project
> +via external C bindings.")
> +=C2=A0=C2=A0=C2=A0 (license license:expat)))
> +[...]
> +(define-public fan2go
> +=C2=A0 (package
> +=C2=A0=C2=A0=C2=A0 (name "fan2go")
> +=C2=A0=C2=A0=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (arguments
> +=C2=A0=C2=A0=C2=A0=C2=A0 '(#:import-path "github.com/markusressel/fan2go=
"
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 #:tests? #f))

Why are tests disabled here?
This is a command line utility, not a Go library, so propagation
is probably unnecessary (though some 'wrap-program' might be required).

> +=C2=A0=C2=A0=C2=A0 [...]
> +=C2=A0=C2=A0=C2=A0 (home-page "https://github.com/markusressel/fan2go")
> +=C2=A0=C2=A0=C2=A0 (synopsis "Fan speed control daemon based on reading =
temperature
> sensors")

Something like
=E2=80=98Automated fan speed control daemon preventing overheating=E2=80=99
(assuming that's correct) seems more descriptive.  While reading
temperature sensors seems unavoidable in an implementation, it still
seems an implementation detail to me.

> +=C2=A0=C2=A0=C2=A0 (description "fan2go is a simple fan control daemon t=
hat relies

@command{fan2go}

> on
> +lm-sensors to get both temperature and RPM sensor readings, as well

@code{fan2go}

> as PWM

What's RPM and PWM?=C2=A0 From (guix)Synopses and descriptions:

=E2=80=98[...] avoid using acronyms without first introducing them.=E2=80=
=99

> +controls, so you will have to set it up first.")

What setup is needed here?  Can this requirement be eliminated in Guix?

Greetings,
Maxime.

--=-bQ7XFUmtNN/dZs9HWSEA
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYekzVRccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7h9HAP4qVCtnkjkeCMx/e7Ss+pR+YdXo
YOiOpvUlkVxFgfOC7gEAubj+/X0/uloIlMU1CeHA96pdHeSUgaEQtea10WAl2w4=
=W6zX
-----END PGP SIGNATURE-----

--=-bQ7XFUmtNN/dZs9HWSEA--

Last modified: Thu, 20 Jan 2022 10:15:01 UTC

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