unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#46541] [PATCH] gnu: Add ttyload.
@ 2021-02-15 19:28 Stefan Reichör
  2021-02-16  8:32 ` 宋文武
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Reichör @ 2021-02-15 19:28 UTC (permalink / raw)
  To: 46541; +Cc: Stefan Reichör

* gnu/packages/admin.scm (ttyload): New variable.
---
 gnu/packages/admin.scm | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 77386d6033..f1b398c39a 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -36,6 +36,7 @@
 ;;; Copyright © 2020 Vincent Legoll <vincent.legoll@gmail.com>
 ;;; Copyright © 2020 Morgan Smith <Morgan.J.Smith@outlook.com>
 ;;; Copyright © 2021 Zheng Junjie <873216071@qq.com>
+;;; Copyright © 2021 Stefan Reichör <stefan@xsteve.at>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -527,6 +528,40 @@ or via the @code{facter} Ruby library.")
     (home-page "https://github.com/puppetlabs/facter-ng")
     (license license:expat)))
 
+(define-public ttyload
+  (let ((revision "1")
+        (commit "f9495372801ce4b4dad98ad854203e694c31c1eb"))
+    (package
+      (name "ttyload")
+      (version (git-version "0.5.3" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/lindes/ttyload")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "0ldb7a13b9v876c6cbrs78pkizj64drnqx95z5shfbwgpwfhr4im"))))
+      (build-system gnu-build-system)
+      (arguments
+       `(#:tests? #f      ; no tests
+         #:make-flags
+         (list (string-append "CC=" ,(cc-for-target)))
+         #:phases
+         (modify-phases %standard-phases
+           (delete 'configure)
+           (replace 'install
+             (lambda* (#:key outputs #:allow-other-keys)
+               (let* ((out (assoc-ref outputs "out"))
+                      (bin (string-append out "/bin")))
+                 (install-file "ttyload" bin)))))))
+      (home-page "https://www.daveltd.com/src/util/ttyload/")
+      (synopsis "Console based color-coded graphs of CPU load average")
+      (description
+       "Show graphs for 1 minute, 5 minute, 15 minute load averages on the console.")
+      (license license:expat))))
+
 (define-public htop
   (package
     (name "htop")
-- 
2.25.1





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#46541] [PATCH] gnu: Add ttyload.
  2021-02-15 19:28 [bug#46541] [PATCH] gnu: Add ttyload Stefan Reichör
@ 2021-02-16  8:32 ` 宋文武
  2021-02-16 19:52   ` Stefan Reichör
  0 siblings, 1 reply; 4+ messages in thread
From: 宋文武 @ 2021-02-16  8:32 UTC (permalink / raw)
  To: Stefan Reichör; +Cc: 46541

Hello!

Stefan Reichör <stefan@xsteve.at> writes:
> [...]
> * gnu/packages/admin.scm (ttyload): New variable.
> +      (home-page "https://www.daveltd.com/src/util/ttyload/")
> +      (synopsis "Console based color-coded graphs of CPU load average")
> +      (description
> +       "Show graphs for 1 minute, 5 minute, 15 minute load averages on the console.")
Better limit lines under 80 characters.

> +      (license license:expat))))
It's a modified "ISC License", I think it should be (fsf-free "file://LICENSE").




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#46541] [PATCH] gnu: Add ttyload.
  2021-02-16  8:32 ` 宋文武
@ 2021-02-16 19:52   ` Stefan Reichör
  2021-02-17  2:04     ` bug#46541: " 宋文武
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Reichör @ 2021-02-16 19:52 UTC (permalink / raw)
  To: 宋文武; +Cc: 46541

[-- Attachment #1: Type: text/plain, Size: 717 bytes --]

Hi 宋文武,

thanks for reviewing my patch.

I hope that I do the correct thing in replying to this message (I have
never done this before ;-))

Stefan.

> Hello!
>
> Stefan Reichör <stefan@xsteve.at> writes:
>> [...]
>> * gnu/packages/admin.scm (ttyload): New variable.
>> +      (home-page "https://www.daveltd.com/src/util/ttyload/")
>> +      (synopsis "Console based color-coded graphs of CPU load average")
>> +      (description
>> +       "Show graphs for 1 minute, 5 minute, 15 minute load averages on the console.")
> Better limit lines under 80 characters.
>
>> +      (license license:expat))))
> It's a modified "ISC License", I think it should be (fsf-free "file://LICENSE").


[-- Attachment #2: gnu-add-ttyload.patch --]
[-- Type: text/x-diff, Size: 2303 bytes --]

gnu: Add ttyload.

From: Stefan Reichör <stefan@xsteve.at>

* gnu/packages/admin.scm (ttyload): New variable.
---
 gnu/packages/admin.scm |   36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 77386d6033..f2cf28367f 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -36,6 +36,7 @@
 ;;; Copyright © 2020 Vincent Legoll <vincent.legoll@gmail.com>
 ;;; Copyright © 2020 Morgan Smith <Morgan.J.Smith@outlook.com>
 ;;; Copyright © 2021 Zheng Junjie <873216071@qq.com>
+;;; Copyright © 2021 Stefan Reichör <stefan@xsteve.at>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -527,6 +528,41 @@ or via the @code{facter} Ruby library.")
     (home-page "https://github.com/puppetlabs/facter-ng")
     (license license:expat)))
 
+(define-public ttyload
+  (let ((revision "1")
+        (commit "f9495372801ce4b4dad98ad854203e694c31c1eb"))
+    (package
+      (name "ttyload")
+      (version (git-version "0.5.3" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/lindes/ttyload")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "0ldb7a13b9v876c6cbrs78pkizj64drnqx95z5shfbwgpwfhr4im"))))
+      (build-system gnu-build-system)
+      (arguments
+       `(#:tests? #f      ; no tests
+         #:make-flags
+         (list (string-append "CC=" ,(cc-for-target)))
+         #:phases
+         (modify-phases %standard-phases
+           (delete 'configure)
+           (replace 'install
+             (lambda* (#:key outputs #:allow-other-keys)
+               (let* ((out (assoc-ref outputs "out"))
+                      (bin (string-append out "/bin")))
+                 (install-file "ttyload" bin)))))))
+      (home-page "https://www.daveltd.com/src/util/ttyload/")
+      (synopsis "Console based color-coded graphs of CPU load average")
+      (description
+       "Show graphs for 1 minute, 5 minute, 15 minute load averages on the
+console.")
+      (license (license:fsf-free "file://LICENSE")))))
+
 (define-public htop
   (package
     (name "htop")

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* bug#46541: [PATCH] gnu: Add ttyload.
  2021-02-16 19:52   ` Stefan Reichör
@ 2021-02-17  2:04     ` 宋文武
  0 siblings, 0 replies; 4+ messages in thread
From: 宋文武 @ 2021-02-17  2:04 UTC (permalink / raw)
  To: Stefan Reichör; +Cc: 46541-done

Stefan Reichör <stefan@xsteve.at> writes:

> Hi 宋文武,
>
> thanks for reviewing my patch.
>
> I hope that I do the correct thing in replying to this message (I have
> never done this before ;-))

LGTM, pushed, thank you!  (I changed the license to non-copyleft, as
suggested by dftxbs3e in #guix).




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-02-17  2:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-15 19:28 [bug#46541] [PATCH] gnu: Add ttyload Stefan Reichör
2021-02-16  8:32 ` 宋文武
2021-02-16 19:52   ` Stefan Reichör
2021-02-17  2:04     ` bug#46541: " 宋文武

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).