[PATCH] gnu: Add collectl.

  • Done
  • quality assurance status badge
Details
5 participants
  • phodina
  • Roel Janssen
  • Sharlatan Hellseher
  • Steve George
  • zimoun
Owner
Somebody
Submitted by
Roel Janssen
Severity
normal

Debbugs page

Roel Janssen wrote 4 years ago
(address . guix-patches@gnu.org)
4270400a-fcf1-aec6-3dd6-9dbccca03cc5@gnu.org
Hi Guix,

Here's a patch to add collectl.

Cheers,
Roel
From b03d5b84bb8c68b08d25a2e243b405b58ac4c047 Mon Sep 17 00:00:00 2001
From: Roel Janssen <roel@gnu.org>
Date: Thu, 15 Apr 2021 15:38:37 +0200
Subject: [PATCH] gnu: Add collectl.

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

Toggle diff (62 lines)
diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 512ac320e6..90b0bd5059 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -40,6 +40,7 @@
;;; Copyright © 2021 qblade <qblade@protonmail.com>
;;; Copyright © 2021 Hyunseok Kim <lasnesne@lagunposprasihopre.org>
;;; Copyright © 2021 David Larsson <david.larsson@selfhosted.xyz>
+;;; Copyright © 2021 Roel Janssen <roel@gnu.org>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -423,6 +424,47 @@ environments:
@end itemize")
(license license:gpl3)))
+(define-public collectl
+ (package
+ (name "collectl")
+ (version "4.3.1")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append
+ "mirror://sourceforge/collectl/collectl/collectl-" version
+ "/collectl-" version ".src.tar.gz"))
+ (sha256
+ (base32
+ "1wc9k3rmhqzh6cx5dcpqhlc3xcpadsn2ic54r19scdjbjx6jd1r1"))))
+ (build-system gnu-build-system)
+ (arguments
+ `(#:tests? #f ; There are no tests.
+ #:phases
+ (modify-phases %standard-phases
+ (delete 'build) ; There's nothing to build.
+ (replace 'configure
+ (lambda* (#:key outputs #:allow-other-keys)
+ (substitute* "INSTALL"
+ (("DESTDIR:=\"/\"") (format #f "DESTDIR:=~s"
+ (assoc-ref outputs "out")))
+ (("DESTDIR/usr") "DESTDIR"))))
+ (replace 'install
+ (lambda* (#:key outputs #:allow-other-keys)
+ (substitute* "collectl"
+ (("\\$configFile='';")
+ (string-append "$configFile='"
+ (assoc-ref outputs "out")
+ "/etc';")))
+ (invoke "./INSTALL"))))))
+ (inputs
+ `(("perl" ,perl)))
+ (home-page "http://collectl.sourceforge.net")
+ (synopsis "Performance data collector")
+ (description "This package provides a program that collects various
+performance measurement data like CPU, memory, disk and network performance
+numbers.")
+ (license license:artistic2.0)))
+
(define-public daemontools
(package
(name "daemontools")
--
2.30.2
zimoun wrote 3 years ago
(name . Roel Janssen)(address . roel@gnu.org)(address . 47798@debbugs.gnu.org)
871r3xtugq.fsf@gmail.com
Hi Roel,

On Thu, 15 Apr 2021 at 15:40, Roel Janssen <roel@gnu.org> wrote:

Toggle quote (10 lines)
>>From b03d5b84bb8c68b08d25a2e243b405b58ac4c047 Mon Sep 17 00:00:00 2001
> From: Roel Janssen <roel@gnu.org>
> Date: Thu, 15 Apr 2021 15:38:37 +0200
> Subject: [PATCH] gnu: Add collectl.
>
> * gnu/packages/admin.scm (collectl): New variable.
> ---
> gnu/packages/admin.scm | 42 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)

I have not built this patch myself but it LGTM. If it builds fine for
you and “guix lint” does not report, feel free to push. :-)

Cheers,
simon
phodina wrote 3 years ago
[PATCH v2] gnu: Add collectl.
(name . 47798@debbugs.gnu.org)(address . 47798@debbugs.gnu.org)
i6nQ6QXtW1o_8fnyF6uN-ASENGrGvqEmS7cSn_U_FI4GX6RREsq2IGnZ35yF4IpffNDPDtu-cP8Zgh21I7ZGADVVRw74ribkAEoZkP6yXyE=@protonmail.com
Hi zimoun and Roel,

I've build and used the package. The only issue was with the copyright line, therefore I'm sending this patch.

Petr

----8>------------cut here------------------->8-----------

* gnu/packages/admin.scm (collectl): New variable.

Toggle diff (60 lines)
diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 86b6b7ce50..33aebfecfe 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -44,6 +44,7 @@
;;; Copyright © 2021 Brice Waegeneire <brice@waegenei.re>
;;; Copyright © 2021 Maxim Cournoyer <maxim.cournoyer@gmail.com>
;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
+;;; Copyright © 2021 Roel Janssen <roel@gnu.org>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -430,6 +431,47 @@ (define-public cloud-utils
@end itemize")
(license license:gpl3)))

+(define-public collectl
+ (package
+ (name "collectl")
+ (version "4.3.1")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append
+ "mirror://sourceforge/collectl/collectl/collectl-" version
+ "/collectl-" version ".src.tar.gz"))
+ (sha256
+ (base32
+ "1wc9k3rmhqzh6cx5dcpqhlc3xcpadsn2ic54r19scdjbjx6jd1r1"))))
+ (build-system gnu-build-system)
+ (arguments
+ `(#:tests? #f ; There are no tests.
+ #:phases
+ (modify-phases %standard-phases
+ (delete 'build) ; There's nothing to build.
+ (replace 'configure
+ (lambda* (#:key outputs #:allow-other-keys)
+ (substitute* "INSTALL"
+ (("DESTDIR:=\"/\"") (format #f "DESTDIR:=~s"
+ (assoc-ref outputs "out")))
+ (("DESTDIR/usr") "DESTDIR"))))
+ (replace 'install
+ (lambda* (#:key outputs #:allow-other-keys)
+ (substitute* "collectl"
+ (("\\$configFile='';")
+ (string-append "$configFile='"
+ (assoc-ref outputs "out")
+ "/etc';")))
+ (invoke "./INSTALL"))))))
+ (inputs
+ `(("perl" ,perl)))
+ (home-page "http://collectl.sourceforge.net")
+ (synopsis "Performance data collector")
+ (description "This package provides a program that collects various
+performance measurement data like CPU, memory, disk and network performance
+numbers.")
+ (license license:artistic2.0)))
+
(define-public daemontools
(package
(name "daemontools")
--
2.33.1
Steve George wrote 9 months ago
owner 47798
(address . control@debbugs.gnu.org)
1717677672-1419-bts-steve@futurile.net
owner 47798 !
thanks
Sharlatan Hellseher wrote 4 weeks ago
[PATCH] gnu: Add collectl.
(address . 47798-done@debbugs.gnu.org)
87v7tguqey.fsf@gmail.com
Hi,

I've converted to G-expression, tested locally and pushed to master as
f5aa303cc09b266e33db37beb6e6b7b6af79b870.

--
Thanks,
Oleg
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEmEeB3micIcJkGAhndtcnv/Ys0rUFAmerv8UACgkQdtcnv/Ys
0rUT4g/+JxsoD3220BUqnkh6w+EVxLMkd2IWuepyTXyYixi0gzNo3mFs1VRG7G/f
U2GZx8JpVxTq0CehlbZsSkXv5mDj7v0n3hYzMZNkp46JKFEbR9RolXlJtmVBWn7t
BLFbzR8WJozh4M8XGMGKZx/1XH0md4sMpm1cgTHVTODzPfwSERhoTMZov0828EHZ
x9T3haTntunjxmyeSpBWJRvLkR/JMVd1+r85VKxQOWj028KGBcij1ERZwZVk8x8B
9+pHTdrjbKC+fQbhnCBWKd4iI+PY5sg9N+kN4fwbymQ0+KREhbYScv68D5EpmG09
Rnecoiu0pnz9jJeucD9styrlBqxro8gRWGJhHc2kiLEFape+r+Kj68+XLFKeEESs
uQT9HxqAjfSCSdP/OxEoknD2uhPNYmdNXliJidNnPIhjh/Gpvv6tIVEniCOq3IwL
MvwZxPFjupGnX4D3h3oY7dtWUJjCgxAgQbdRW8iZN/2kxHqp/oEuFpoB3aYgP95Z
LkQmM/loYM/6IX1zL3DNr/qUbr64IqRqlmVDyyTgqk8+CSMAbaXRrDxAb+oNKwZ2
Z5v4mU+trnc+4cnWb1JciDnsurEaX8sshYWDoXRBjvTZ5ZlXx3VfUp4t7OawldAX
Q095PpgducQbe9dBNUZmZfE27h1ZSAZp3ZRGaCY7wiMCJPu0ToU=
=EfFt
-----END PGP SIGNATURE-----

Closed
?
Your comment

This issue is archived.

To comment on this conversation send an email to 47798@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 47798
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help