* [PATCH 0/2] Add tools for sysadmins: htop and dfc.
@ 2013-08-30 20:08 Cyril Roelandt
2013-08-30 20:08 ` [PATCH 1/2] gnu: Add htop Cyril Roelandt
` (6 more replies)
0 siblings, 7 replies; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 20:08 UTC (permalink / raw)
To: guix-devel
Hey!
This patch series adds htop and dfc, which are 'modern' versions of top and df.
Note that dfc is the first Guix package to use the CMake build system!
Cyril.
---
Cyril Roelandt (2):
gnu: Add htop.
gnu: Add dfc.
gnu-system.am | 1 +
gnu/packages/sysutils.scm | 70 +++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 71 insertions(+)
create mode 100644 gnu/packages/sysutils.scm
--
1.7.10.4
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH 1/2] gnu: Add htop.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
@ 2013-08-30 20:08 ` Cyril Roelandt
2013-08-30 20:50 ` Nikita Karetnikov
2013-08-30 20:08 ` [PATCH 2/2] gnu: Add dfc Cyril Roelandt
` (5 subsequent siblings)
6 siblings, 1 reply; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 20:08 UTC (permalink / raw)
To: guix-devel
* gnu/packages/sysutils.scm: New file.
* gnu-system.am: Add it.
---
gnu-system.am | 1 +
gnu/packages/sysutils.scm | 47 +++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 48 insertions(+)
create mode 100644 gnu/packages/sysutils.scm
diff --git a/gnu-system.am b/gnu-system.am
index 2600858..fc1872e 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -155,6 +155,7 @@ GNU_SYSTEM_MODULES = \
gnu/packages/sqlite.scm \
gnu/packages/ssh.scm \
gnu/packages/system.scm \
+ gnu/packages/sysutils.scm \
gnu/packages/tcl.scm \
gnu/packages/tcsh.scm \
gnu/packages/texinfo.scm \
diff --git a/gnu/packages/sysutils.scm b/gnu/packages/sysutils.scm
new file mode 100644
index 0000000..c79ac90
--- /dev/null
+++ b/gnu/packages/sysutils.scm
@@ -0,0 +1,47 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2013 Cyril Roelandt <tipecaml@gmail.com>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix. If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages sysutils)
+ #:use-module (guix licenses)
+ #:use-module (guix packages)
+ #:use-module (guix download)
+ #:use-module (guix build-system gnu)
+ #:use-module (gnu packages)
+ #:use-module (gnu packages ncurses))
+
+(define-public htop
+ (package
+ (name "htop")
+ (version "1.0.2")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append
+ "mirror://sourceforge/htop/"
+ version "/htop-" version ".tar.gz"))
+ (sha256
+ (base32
+ "18fqrhvnm7h4c3939av8lpiwrwxbyw6hcly0jvq0vkjf0ixnaq7f"))))
+ (build-system gnu-build-system)
+ (inputs
+ `(("ncurses" ,ncurses)))
+ (home-page "http://htop.sourceforge.net/")
+ (synopsis "An interactive process viewer for Linux")
+ (description
+ "This is htop, an interactive process viewer for Linux. It is a text-mode
+application (for console or X terminals) and requires ncurses.")
+ (license gpl2)))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread
* [PATCH 2/2] gnu: Add dfc.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
2013-08-30 20:08 ` [PATCH 1/2] gnu: Add htop Cyril Roelandt
@ 2013-08-30 20:08 ` Cyril Roelandt
2013-08-30 21:28 ` Ludovic Courtès
2013-08-30 20:58 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Nikita Karetnikov
` (4 subsequent siblings)
6 siblings, 1 reply; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 20:08 UTC (permalink / raw)
To: guix-devel
* gnu/packages/sysutils.scm (dfc): New variable.
---
gnu/packages/sysutils.scm | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/gnu/packages/sysutils.scm b/gnu/packages/sysutils.scm
index c79ac90..953d723 100644
--- a/gnu/packages/sysutils.scm
+++ b/gnu/packages/sysutils.scm
@@ -20,10 +20,33 @@
#:use-module (guix licenses)
#:use-module (guix packages)
#:use-module (guix download)
+ #:use-module (guix build-system cmake)
#:use-module (guix build-system gnu)
#:use-module (gnu packages)
#:use-module (gnu packages ncurses))
+(define-public dfc
+ (package
+ (name "dfc")
+ (version "3.0.3")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (string-append
+ "http://projects.gw-computing.net/attachments/download/78/dfc-"
+ version ".tar.gz"))
+ (sha256
+ (base32
+ "1b4hfqv23l87cb37fxwzfk2sgspkyxpr3ig2hsd23hr6mm982j7z"))))
+ (build-system cmake-build-system)
+ (arguments '(#:tests? #f))
+ (home-page "http://projects.gw-computing.net/projects/dfc")
+ (synopsis "Display file system space usage using graphs and colors.")
+ (description
+ "dfc (df color) is a modern version of df. It uses colors, draws pretty
+graphs and can export its output to different formats.")
+ (license bsd-3)))
+
(define-public htop
(package
(name "htop")
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 20:58 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Nikita Karetnikov
@ 2013-08-30 20:37 ` Cyril Roelandt
2013-08-30 21:50 ` Ludovic Courtès
2013-08-30 21:52 ` Nikita Karetnikov
0 siblings, 2 replies; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 20:37 UTC (permalink / raw)
To: Nikita Karetnikov; +Cc: guix-devel
On 08/30/2013 10:58 PM, Nikita Karetnikov wrote:
> More nitpicking…
>
>> + (uri (string-append
>> + "mirror://sourceforge/htop/"
>> + version "/htop-" version ".tar.gz"))
>
> Could you place ‘string-append’ and "mirror://sourceforge/htop/" on
> the same line?
>
Yes. I never know what to do in such cases. Is there a document that
contains these coding-style rules, such as the PEP8 for Python ?
>> + (arguments '(#:tests? #f))
>
> Why did you disable tests? Could you add a comment about that?
>
Sure. There are no tests :)
Cyril.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] gnu: Add htop.
2013-08-30 20:50 ` Nikita Karetnikov
@ 2013-08-30 20:38 ` Cyril Roelandt
2013-08-30 21:26 ` Ludovic Courtès
0 siblings, 1 reply; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 20:38 UTC (permalink / raw)
To: Nikita Karetnikov; +Cc: guix-devel
On 08/30/2013 10:50 PM, Nikita Karetnikov wrote:
>> + (synopsis "An interactive process viewer for Linux")
>> + (description
>> + "This is htop, an interactive process viewer for Linux. It is a text-mode
>> +application (for console or X terminals) and requires ncurses.")
>
> I’d suggest to change “Linux” to “the Linux kernel” or “GNU/Linux” to
> avoid ambiguity. It might be even better not to use this word at all.
>
It seems to be also available in the FreeBSD ports, and in NetBSD
through pkgsrc. I'll s/ for Linux//.
Cyril.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] gnu: Add htop.
2013-08-30 20:08 ` [PATCH 1/2] gnu: Add htop Cyril Roelandt
@ 2013-08-30 20:50 ` Nikita Karetnikov
2013-08-30 20:38 ` Cyril Roelandt
0 siblings, 1 reply; 23+ messages in thread
From: Nikita Karetnikov @ 2013-08-30 20:50 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
[-- Attachment #1: Type: text/plain, Size: 391 bytes --]
> + (synopsis "An interactive process viewer for Linux")
> + (description
> + "This is htop, an interactive process viewer for Linux. It is a text-mode
> +application (for console or X terminals) and requires ncurses.")
I’d suggest to change “Linux” to “the Linux kernel” or “GNU/Linux” to
avoid ambiguity. It might be even better not to use this word at all.
[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
2013-08-30 20:08 ` [PATCH 1/2] gnu: Add htop Cyril Roelandt
2013-08-30 20:08 ` [PATCH 2/2] gnu: Add dfc Cyril Roelandt
@ 2013-08-30 20:58 ` Nikita Karetnikov
2013-08-30 20:37 ` Cyril Roelandt
2013-08-30 21:07 ` [PATCH 1/2 v2] gnu: Add htop Cyril Roelandt
` (3 subsequent siblings)
6 siblings, 1 reply; 23+ messages in thread
From: Nikita Karetnikov @ 2013-08-30 20:58 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
[-- Attachment #1: Type: text/plain, Size: 362 bytes --]
More nitpicking…
> + (uri (string-append
> + "mirror://sourceforge/htop/"
> + version "/htop-" version ".tar.gz"))
Could you place ‘string-append’ and "mirror://sourceforge/htop/" on
the same line?
> + (arguments '(#:tests? #f))
Why did you disable tests? Could you add a comment about that?
[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH 1/2 v2] gnu: Add htop.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
` (2 preceding siblings ...)
2013-08-30 20:58 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Nikita Karetnikov
@ 2013-08-30 21:07 ` Cyril Roelandt
2013-08-30 21:07 ` [PATCH 2/2 v2] gnu: Add dfc Cyril Roelandt
2013-08-31 9:23 ` [PATCH 1/2 v2] gnu: Add htop Ludovic Courtès
2013-08-30 21:30 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Andreas Enge
` (2 subsequent siblings)
6 siblings, 2 replies; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 21:07 UTC (permalink / raw)
To: guix-devel
* gnu/packages/sysutils.scm: New file.
* gnu-system.am: Add it.
---
gnu-system.am | 1 +
gnu/packages/sysutils.scm | 46 +++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 47 insertions(+)
create mode 100644 gnu/packages/sysutils.scm
diff --git a/gnu-system.am b/gnu-system.am
index 2600858..fc1872e 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -155,6 +155,7 @@ GNU_SYSTEM_MODULES = \
gnu/packages/sqlite.scm \
gnu/packages/ssh.scm \
gnu/packages/system.scm \
+ gnu/packages/sysutils.scm \
gnu/packages/tcl.scm \
gnu/packages/tcsh.scm \
gnu/packages/texinfo.scm \
diff --git a/gnu/packages/sysutils.scm b/gnu/packages/sysutils.scm
new file mode 100644
index 0000000..e7ee51f
--- /dev/null
+++ b/gnu/packages/sysutils.scm
@@ -0,0 +1,46 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2013 Cyril Roelandt <tipecaml@gmail.com>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix. If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages sysutils)
+ #:use-module (guix licenses)
+ #:use-module (guix packages)
+ #:use-module (guix download)
+ #:use-module (guix build-system gnu)
+ #:use-module (gnu packages)
+ #:use-module (gnu packages ncurses))
+
+(define-public htop
+ (package
+ (name "htop")
+ (version "1.0.2")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append "mirror://sourceforge/htop/"
+ version "/htop-" version ".tar.gz"))
+ (sha256
+ (base32
+ "18fqrhvnm7h4c3939av8lpiwrwxbyw6hcly0jvq0vkjf0ixnaq7f"))))
+ (build-system gnu-build-system)
+ (inputs
+ `(("ncurses" ,ncurses)))
+ (home-page "http://htop.sourceforge.net/")
+ (synopsis "An interactive process viewer")
+ (description
+ "This is htop, an interactive process viewer. It is a text-mode
+application (for console or X terminals) and requires ncurses.")
+ (license gpl2)))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread
* [PATCH 2/2 v2] gnu: Add dfc.
2013-08-30 21:07 ` [PATCH 1/2 v2] gnu: Add htop Cyril Roelandt
@ 2013-08-30 21:07 ` Cyril Roelandt
2013-08-31 9:24 ` Ludovic Courtès
2013-08-31 9:23 ` [PATCH 1/2 v2] gnu: Add htop Ludovic Courtès
1 sibling, 1 reply; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 21:07 UTC (permalink / raw)
To: guix-devel
* gnu/packages/sysutils.scm (dfc): New variable.
---
gnu/packages/sysutils.scm | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/gnu/packages/sysutils.scm b/gnu/packages/sysutils.scm
index e7ee51f..d91719c 100644
--- a/gnu/packages/sysutils.scm
+++ b/gnu/packages/sysutils.scm
@@ -20,10 +20,33 @@
#:use-module (guix licenses)
#:use-module (guix packages)
#:use-module (guix download)
+ #:use-module (guix build-system cmake)
#:use-module (guix build-system gnu)
#:use-module (gnu packages)
#:use-module (gnu packages ncurses))
+(define-public dfc
+ (package
+ (name "dfc")
+ (version "3.0.3")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (string-append
+ "http://projects.gw-computing.net/attachments/download/78/dfc-"
+ version ".tar.gz"))
+ (sha256
+ (base32
+ "1b4hfqv23l87cb37fxwzfk2sgspkyxpr3ig2hsd23hr6mm982j7z"))))
+ (build-system cmake-build-system)
+ (arguments '(#:tests? #f)) ; There are no tests.
+ (home-page "http://projects.gw-computing.net/projects/dfc")
+ (synopsis "Display file system space usage using graphs and colors.")
+ (description
+ "dfc (df color) is a modern version of df. It uses colors, draws pretty
+graphs and can export its output to different formats.")
+ (license bsd-3)))
+
(define-public htop
(package
(name "htop")
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 21:30 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Andreas Enge
@ 2013-08-30 21:17 ` Cyril Roelandt
0 siblings, 0 replies; 23+ messages in thread
From: Cyril Roelandt @ 2013-08-30 21:17 UTC (permalink / raw)
To: Andreas Enge; +Cc: guix-devel
On 08/30/2013 11:30 PM, Andreas Enge wrote:
> On Fri, Aug 30, 2013 at 10:08:34PM +0200, Cyril Roelandt wrote:
>> This patch series adds htop and dfc, which are 'modern' versions of top and df.
>> Note that dfc is the first Guix package to use the CMake build system!
>
> Since cmake currently does not pass its tests on x86_64, I suppose the
> package will so far be only available on i686?
>
Yes. I'll have to look into this cmake issue. I think I'll start by
upgrading to 2.8.11.2, see if it fixes the issue.
Cyril.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] gnu: Add htop.
2013-08-30 20:38 ` Cyril Roelandt
@ 2013-08-30 21:26 ` Ludovic Courtès
0 siblings, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-30 21:26 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> On 08/30/2013 10:50 PM, Nikita Karetnikov wrote:
>>> + (synopsis "An interactive process viewer for Linux")
>>> + (description
>>> + "This is htop, an interactive process viewer for Linux. It is a text-mode
>>> +application (for console or X terminals) and requires ncurses.")
>>
>> I’d suggest to change “Linux” to “the Linux kernel” or “GNU/Linux” to
>> avoid ambiguity. It might be even better not to use this word at all.
>>
>
> It seems to be also available in the FreeBSD ports, and in NetBSD
> through pkgsrc. I'll s/ for Linux//.
Agreed, just remove “for Linux”.
Also, remove “An” from the synopsis.
Then OK to push.
Thanks!
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 2/2] gnu: Add dfc.
2013-08-30 20:08 ` [PATCH 2/2] gnu: Add dfc Cyril Roelandt
@ 2013-08-30 21:28 ` Ludovic Courtès
0 siblings, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-30 21:28 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> + (build-system cmake-build-system)
Woow. :-)
> + (arguments '(#:tests? #f))
Please add a comment saying why tests are disabled.
> + (home-page "http://projects.gw-computing.net/projects/dfc")
> + (synopsis "Display file system space usage using graphs and colors.")
No period at the end of the synopsis.
OK to push with these changes!
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
` (3 preceding siblings ...)
2013-08-30 21:07 ` [PATCH 1/2 v2] gnu: Add htop Cyril Roelandt
@ 2013-08-30 21:30 ` Andreas Enge
2013-08-30 21:17 ` Cyril Roelandt
2013-08-31 15:51 ` Ludovic Courtès
2013-09-01 18:03 ` [PATCH 1/2 v3] gnu: Add htop Cyril Roelandt
6 siblings, 1 reply; 23+ messages in thread
From: Andreas Enge @ 2013-08-30 21:30 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
On Fri, Aug 30, 2013 at 10:08:34PM +0200, Cyril Roelandt wrote:
> This patch series adds htop and dfc, which are 'modern' versions of top and df.
> Note that dfc is the first Guix package to use the CMake build system!
Since cmake currently does not pass its tests on x86_64, I suppose the
package will so far be only available on i686?
Andreas
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 20:37 ` Cyril Roelandt
@ 2013-08-30 21:50 ` Ludovic Courtès
2013-08-30 21:52 ` Nikita Karetnikov
1 sibling, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-30 21:50 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> On 08/30/2013 10:58 PM, Nikita Karetnikov wrote:
>> More nitpicking…
>>
>>> + (uri (string-append
>>> + "mirror://sourceforge/htop/"
>>> + version "/htop-" version ".tar.gz"))
>>
>> Could you place ‘string-append’ and "mirror://sourceforge/htop/" on
>> the same line?
>>
>
> Yes. I never know what to do in such cases. Is there a document that
> contains these coding-style rules, such as the PEP8 for Python ?
Mainly this: http://mumble.net/~campbell/scheme/style.txt .
And the knowledge that’s in Emacs’s Scheme mode.
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 20:37 ` Cyril Roelandt
2013-08-30 21:50 ` Ludovic Courtès
@ 2013-08-30 21:52 ` Nikita Karetnikov
2013-08-31 9:32 ` Ludovic Courtès
1 sibling, 1 reply; 23+ messages in thread
From: Nikita Karetnikov @ 2013-08-30 21:52 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
[-- Attachment #1: Type: text/plain, Size: 1535 bytes --]
> Yes. I never know what to do in such cases. Is there a document that
> contains these coding-style rules, such as the PEP8 for Python ?
The pretty-printing rule is described in SICP. Search for “There is no
limit” [1]. However, that paragraph doesn’t say what to do when the
line is too long.
I’m trying to keep lines under 79 characters (as suggested in PEP8). It
allows “to have several files open side-by-side, and works well when
using code review tools that present the two versions in adjacent
columns” [2]. I also find that short lines are easier to read.
I believe it’s OK to write
(proc very-very-very-very-very-very-long-argument
a b c)
instead of
(proc very-very-very-very-very-very-long-argument
a
b
c)
because this allows to display more lines on the screen.
Also, it may be a good idea to write
(very-long-procedure1
argument1
(very-long-procedure2
argument2
(very-long-procedure3
argument3
(...))))
instead of
(very-long-procedure1 argument1
(very-long-procedure2 argument2
(very-long-procedure3 argument3
(...))))
That said, you have to break the rules sometimes.
Should we mention this in the “packaging rules” section?
[1] https://mitpress.mit.edu/sicp/full-text/book/book-Z-H-10.html#%_sec_1.1.1
[2] http://www.python.org/dev/peps/pep-0008/#maximum-line-length
[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2 v2] gnu: Add htop.
2013-08-30 21:07 ` [PATCH 1/2 v2] gnu: Add htop Cyril Roelandt
2013-08-30 21:07 ` [PATCH 2/2 v2] gnu: Add dfc Cyril Roelandt
@ 2013-08-31 9:23 ` Ludovic Courtès
1 sibling, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-31 9:23 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> + (synopsis "An interactive process viewer")
-An, but OK otherwise.
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 2/2 v2] gnu: Add dfc.
2013-08-30 21:07 ` [PATCH 2/2 v2] gnu: Add dfc Cyril Roelandt
@ 2013-08-31 9:24 ` Ludovic Courtès
0 siblings, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-31 9:24 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> + (synopsis "Display file system space usage using graphs and colors.")
Minus period, but OK otherwise.
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 21:52 ` Nikita Karetnikov
@ 2013-08-31 9:32 ` Ludovic Courtès
2013-08-31 17:38 ` Nikita Karetnikov
0 siblings, 1 reply; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-31 9:32 UTC (permalink / raw)
To: Nikita Karetnikov; +Cc: guix-devel
Nikita Karetnikov <nikita@karetnikov.org> skribis:
> Should we mention this in the “packaging rules” section?
I think our messages crossed. What do you think of the “Coding Style”
section in ‘HACKING’?
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
` (4 preceding siblings ...)
2013-08-30 21:30 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Andreas Enge
@ 2013-08-31 15:51 ` Ludovic Courtès
2013-09-01 18:03 ` [PATCH 1/2 v3] gnu: Add htop Cyril Roelandt
6 siblings, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-08-31 15:51 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> This patch series adds htop and dfc, which are 'modern' versions of top and df.
[...]
> gnu/packages/sysutils.scm | 70 +++++++++++++++++++++++++++++++++++++++++++++
It just occurred to me that there’s already (gnu packages system), which
contains sysadmin tools.
Could you put the contents of sysutils.scm in there?
Thanks,
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 0/2] Add tools for sysadmins: htop and dfc.
2013-08-31 9:32 ` Ludovic Courtès
@ 2013-08-31 17:38 ` Nikita Karetnikov
0 siblings, 0 replies; 23+ messages in thread
From: Nikita Karetnikov @ 2013-08-31 17:38 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: guix-devel
[-- Attachment #1: Type: text/plain, Size: 190 bytes --]
> I think our messages crossed. What do you think of the “Coding Style”
> section in ‘HACKING’?
It’s OK. Let’s keep it.
We could always augment it if there’s a need.
[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH 1/2 v3] gnu: Add htop.
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
` (5 preceding siblings ...)
2013-08-31 15:51 ` Ludovic Courtès
@ 2013-09-01 18:03 ` Cyril Roelandt
2013-09-01 18:03 ` [PATCH 2/2 v3] gnu: Add dfc Cyril Roelandt
2013-09-01 19:50 ` [PATCH 1/2 v3] gnu: Add htop Ludovic Courtès
6 siblings, 2 replies; 23+ messages in thread
From: Cyril Roelandt @ 2013-09-01 18:03 UTC (permalink / raw)
To: guix-devel
* gnu/packages/sysutils.scm (htop): New variable.
---
gnu/packages/system.scm | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/gnu/packages/system.scm b/gnu/packages/system.scm
index e326e49..fb364f1 100644
--- a/gnu/packages/system.scm
+++ b/gnu/packages/system.scm
@@ -1,5 +1,6 @@
;;; GNU Guix --- Functional package management for GNU
;;; Copyright © 2012, 2013 Ludovic Courtès <ludo@gnu.org>
+;;; Copyright © 2013 Cyril Roelandt <tipecaml@gmail.com>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -25,6 +26,27 @@
#:use-module (gnu packages ncurses)
#:use-module (gnu packages linux))
+(define-public htop
+ (package
+ (name "htop")
+ (version "1.0.2")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append "mirror://sourceforge/htop/"
+ version "/htop-" version ".tar.gz"))
+ (sha256
+ (base32
+ "18fqrhvnm7h4c3939av8lpiwrwxbyw6hcly0jvq0vkjf0ixnaq7f"))))
+ (build-system gnu-build-system)
+ (inputs
+ `(("ncurses" ,ncurses)))
+ (home-page "http://htop.sourceforge.net/")
+ (synopsis "Interactive process viewer")
+ (description
+ "This is htop, an interactive process viewer. It is a text-mode
+application (for console or X terminals) and requires ncurses.")
+ (license gpl2)))
+
(define-public pies
(package
(name "pies")
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread
* [PATCH 2/2 v3] gnu: Add dfc.
2013-09-01 18:03 ` [PATCH 1/2 v3] gnu: Add htop Cyril Roelandt
@ 2013-09-01 18:03 ` Cyril Roelandt
2013-09-01 19:50 ` [PATCH 1/2 v3] gnu: Add htop Ludovic Courtès
1 sibling, 0 replies; 23+ messages in thread
From: Cyril Roelandt @ 2013-09-01 18:03 UTC (permalink / raw)
To: guix-devel
* gnu/packages/system.scm (dfc): New variable.
---
gnu/packages/system.scm | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/gnu/packages/system.scm b/gnu/packages/system.scm
index fb364f1..1f8a7d5 100644
--- a/gnu/packages/system.scm
+++ b/gnu/packages/system.scm
@@ -21,11 +21,34 @@
#:use-module (guix licenses)
#:use-module (guix packages)
#:use-module (guix download)
+ #:use-module (guix build-system cmake)
#:use-module (guix build-system gnu)
#:use-module (gnu packages)
#:use-module (gnu packages ncurses)
#:use-module (gnu packages linux))
+(define-public dfc
+ (package
+ (name "dfc")
+ (version "3.0.3")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (string-append
+ "http://projects.gw-computing.net/attachments/download/78/dfc-"
+ version ".tar.gz"))
+ (sha256
+ (base32
+ "1b4hfqv23l87cb37fxwzfk2sgspkyxpr3ig2hsd23hr6mm982j7z"))))
+ (build-system cmake-build-system)
+ (arguments '(#:tests? #f)) ; There are no tests.
+ (home-page "http://projects.gw-computing.net/projects/dfc")
+ (synopsis "Display file system space usage using graphs and colors.")
+ (description
+ "dfc (df color) is a modern version of df. It uses colors, draws pretty
+graphs and can export its output to different formats.")
+ (license bsd-3)))
+
(define-public htop
(package
(name "htop")
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2 v3] gnu: Add htop.
2013-09-01 18:03 ` [PATCH 1/2 v3] gnu: Add htop Cyril Roelandt
2013-09-01 18:03 ` [PATCH 2/2 v3] gnu: Add dfc Cyril Roelandt
@ 2013-09-01 19:50 ` Ludovic Courtès
1 sibling, 0 replies; 23+ messages in thread
From: Ludovic Courtès @ 2013-09-01 19:50 UTC (permalink / raw)
To: Cyril Roelandt; +Cc: guix-devel
Cyril Roelandt <tipecaml@gmail.com> skribis:
> * gnu/packages/sysutils.scm (htop): New variable.
Should be ‘system.scm’. :-)
[...]
> + (synopsis "Display file system space usage using graphs and colors.")
No period at the end.
Feel free to push, then!
Ludo’.
^ permalink raw reply [flat|nested] 23+ messages in thread
end of thread, other threads:[~2013-09-01 19:55 UTC | newest]
Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-30 20:08 [PATCH 0/2] Add tools for sysadmins: htop and dfc Cyril Roelandt
2013-08-30 20:08 ` [PATCH 1/2] gnu: Add htop Cyril Roelandt
2013-08-30 20:50 ` Nikita Karetnikov
2013-08-30 20:38 ` Cyril Roelandt
2013-08-30 21:26 ` Ludovic Courtès
2013-08-30 20:08 ` [PATCH 2/2] gnu: Add dfc Cyril Roelandt
2013-08-30 21:28 ` Ludovic Courtès
2013-08-30 20:58 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Nikita Karetnikov
2013-08-30 20:37 ` Cyril Roelandt
2013-08-30 21:50 ` Ludovic Courtès
2013-08-30 21:52 ` Nikita Karetnikov
2013-08-31 9:32 ` Ludovic Courtès
2013-08-31 17:38 ` Nikita Karetnikov
2013-08-30 21:07 ` [PATCH 1/2 v2] gnu: Add htop Cyril Roelandt
2013-08-30 21:07 ` [PATCH 2/2 v2] gnu: Add dfc Cyril Roelandt
2013-08-31 9:24 ` Ludovic Courtès
2013-08-31 9:23 ` [PATCH 1/2 v2] gnu: Add htop Ludovic Courtès
2013-08-30 21:30 ` [PATCH 0/2] Add tools for sysadmins: htop and dfc Andreas Enge
2013-08-30 21:17 ` Cyril Roelandt
2013-08-31 15:51 ` Ludovic Courtès
2013-09-01 18:03 ` [PATCH 1/2 v3] gnu: Add htop Cyril Roelandt
2013-09-01 18:03 ` [PATCH 2/2 v3] gnu: Add dfc Cyril Roelandt
2013-09-01 19:50 ` [PATCH 1/2 v3] gnu: Add htop Ludovic Courtès
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).