Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
Alex Kost skribis: >> From e93d21889bdcbed02a517f30d8401a3bb144cce4 Mon Sep 17 00:00:00 2001 >> From: Claes Wallin >> Date: Fri, 19 Jun 2015 20:12:09 +0200 >> Subject: [PATCH] gnu: Add pv. >> >> * gnu/package/pv.scm (pv): New file. >> * gnu-system.am (GNU_SYSTEM_MODULES): Add it. > > Thank you! It looks good to me, so I'll push it in a couple of days if > no one will send additional comments/notes. If it looks good to you, go ahead. Thanks for reviewing it. Ludo’.
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
On 26-Jun-2015 3:33 pm, "Alex Kost" wrote: > Claes Wallin (韋嘉誠) (2015-06-25 00:35 +0300) wrote: > > From e93d21889bdcbed02a517f30d8401a3bb144cce4 Mon Sep 17 00:00:00 2001 > > From: Claes Wallin > > Date: Fri, 19 Jun 2015 20:12:09 +0200 > > Subject: [PATCH] gnu: Add pv. > > > > * gnu/package/pv.scm (pv): New file. > > * gnu-system.am (GNU_SYSTEM_MODULES): Add it. > > Thank you! It looks good to me, so I'll push it in a couple of days if > no one will send additional comments/notes. Sweet!
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
Claes Wallin (韋嘉誠) (2015-06-25 00:35 +0300) wrote: > On Tue, Jun 23, 2015 at 1:43 PM, Alex Kost wrote: > >> --8<---cut here---start->8--- >> gnu: Add pv. >> >> * gnu/package/pv.scm (pv): New file. >> * gnu-system.am (GNU_SYSTEM_MODULES): Add it. >> --8<---cut here---end--->8--- >> >> And you also need to add "gnu/package/pv.scm" to "gnu-system.am". See >> commit 741115b for example. > >> IIUC you are the only author of this file, right? Then there is no >> place for Ludovic there. > >> I think we prefer 'define-public' over exporting the package variables, >> but it is probably not a strong convention. > >> I would put "Pipe Viewer" in parentheses: >> >> "pv (Pipe Viewer) is a terminal-based tool for monitoring the >> progress > >> I realize that you took this description from the home page, but our >> convention is to use two spaces between sentences. > > > Updated accordingly! > > > From e93d21889bdcbed02a517f30d8401a3bb144cce4 Mon Sep 17 00:00:00 2001 > From: Claes Wallin > Date: Fri, 19 Jun 2015 20:12:09 +0200 > Subject: [PATCH] gnu: Add pv. > > * gnu/package/pv.scm (pv): New file. > * gnu-system.am (GNU_SYSTEM_MODULES): Add it. Thank you! It looks good to me, so I'll push it in a couple of days if no one will send additional comments/notes. -- Alex
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
On Tue, Jun 23, 2015 at 1:43 PM, Alex Kost wrote: > --8<---cut here---start->8--- > gnu: Add pv. > > * gnu/package/pv.scm (pv): New file. > * gnu-system.am (GNU_SYSTEM_MODULES): Add it. > --8<---cut here---end--->8--- > > And you also need to add "gnu/package/pv.scm" to "gnu-system.am". See > commit 741115b for example. > IIUC you are the only author of this file, right? Then there is no > place for Ludovic there. > I think we prefer 'define-public' over exporting the package variables, > but it is probably not a strong convention. > I would put "Pipe Viewer" in parentheses: > > "pv (Pipe Viewer) is a terminal-based tool for monitoring the progress > I realize that you took this description from the home page, but our > convention is to use two spaces between sentences. Updated accordingly! -- /c 0001-gnu-Add-pv.patch Description: Binary data
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
On Tue, Jun 23, 2015 at 1:43 PM, Alex Kost wrote: > Claes Wallin (韋嘉誠) (2015-06-19 21:51 +0300) wrote: > I think there is no need in this ↑ line. The commit message should be: > > --8<---cut here---start->8--- > gnu: Add pv. > > * gnu/package/pv.scm (pv): New file. > * gnu-system.am (GNU_SYSTEM_MODULES): Add it. > --8<---cut here---end--->8--- Ok! > And you also need to add "gnu/package/pv.scm" to "gnu-system.am". See > commit 741115b for example. I will take a look. >> +;;; Copyright 2012, 2013, 2015 Ludovic Courts >> +;;; Copyright 2015 Claes Wallin > > IIUC you are the only author of this file, right? Then there is no > place for Ludovic there. Ok, I copied it from another file and modified it, but if the convention is that the boilerplate is too thin to be credited, I'll remove him. >> + #:export (pv)) >> + >> +(define pv > > I think we prefer 'define-public' over exporting the package variables, > but it is probably not a strong convention. define-public is less redundant. I like it. > I would put "Pipe Viewer" in parentheses: > > "pv (Pipe Viewer) is a terminal-based tool for monitoring the progress > > but I think you may ignore this comment. No, it's a fair point. >> +of data through a pipeline. It can be inserted into any normal pipeline >of data through a pipeline. It can be inserted into any normal pipeline > > I realize that you took this description from the home page, but our > convention is to use two spaces between sentences. If there's a convention I'll follow it. I'll just note here once that I disagree with it. Double space is a type-writer convention that "nobody" follows any more. ;-) Regarding scraping the text I tried quickly to see what license the web site text is under, but didn't find anything conclusive. I figured if Debian is copying it, I should be fine. Maybe I'll get back to checking it later. Thanks for the feedback. I have several more packages in the pipeline once I get this one right! -- /c
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
Claes Wallin (韋嘉誠) (2015-06-19 21:51 +0300) wrote: > From 9213e7207f5e72bf66460d2cdf4e27a86609e322 Mon Sep 17 00:00:00 2001 > From: Claes Wallin > Date: Fri, 19 Jun 2015 20:12:09 +0200 > Subject: [PATCH] gnu/package/pv.scm (pv): New file, new variable > > Signed-off-by: Claes Wallin I think there is no need in this ↑ line. The commit message should be: --8<---cut here---start->8--- gnu: Add pv. * gnu/package/pv.scm (pv): New file. * gnu-system.am (GNU_SYSTEM_MODULES): Add it. --8<---cut here---end--->8--- And you also need to add "gnu/package/pv.scm" to "gnu-system.am". See commit 741115b for example. > --- > gnu/packages/pv.scm | 48 > 1 file changed, 48 insertions(+) > create mode 100644 gnu/packages/pv.scm > > diff --git a/gnu/packages/pv.scm b/gnu/packages/pv.scm > new file mode 100644 > index 000..f0eac85 > --- /dev/null > +++ b/gnu/packages/pv.scm > @@ -0,0 +1,48 @@ > +;;; GNU Guix --- Functional package management for GNU > +;;; Copyright 2012, 2013, 2015 Ludovic Courts > +;;; Copyright 2015 Claes Wallin IIUC you are the only author of this file, right? Then there is no place for Ludovic there. > +;;; > +;;; 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 pv) > + #:use-module (guix licenses) > + #:use-module (guix packages) > + #:use-module (guix download) > + #:use-module (guix build-system gnu) > + #:export (pv)) > + > +(define pv I think we prefer 'define-public' over exporting the package variables, but it is probably not a strong convention. > + (package > +(name "pv") > +(version "1.6.0") > +(source > + (origin > + (method url-fetch) > + (uri (string-append "http://www.ivarch.com/programs/sources/pv-"; > + version ".tar.bz2")) > + (sha256 > + (base32 > +"13gg6r84pkvznpd1l11qw1jw9yna40gkgpni256khyx21m785khf" > +(build-system gnu-build-system) > +(home-page "http://www.ivarch.com/programs/pv.shtml";) > +(synopsis "Pipeline progress indicator") > +(description > + "pv - Pipe Viewer - is a terminal-based tool for monitoring the progress I would put "Pipe Viewer" in parentheses: "pv (Pipe Viewer) is a terminal-based tool for monitoring the progress but I think you may ignore this comment. > +of data through a pipeline. It can be inserted into any normal pipeline of data through a pipeline. It can be inserted into any normal pipeline I realize that you took this description from the home page, but our convention is to use two spaces between sentences. > +between two processes to give a visual indication of how quickly data is > +passing through, how long it has taken, how near to completion it is, and an > +estimate of how long it will be until completion.") > +(license artistic2.0))) -- Alex
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
On Fri, Jun 19, 2015 at 8:51 PM, Claes Wallin (韋嘉誠) wrote: > Argh. Why is there always some small detail you don't notice until you > posted, even in a small file like this? > > Removed commented-out lines. Bump? 0001-gnu-package-pv.scm-pv-New-file-new-variable.patch Description: Binary data
Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
Argh. Why is there always some small detail you don't notice until you posted, even in a small file like this? Removed commented-out lines. 0001-gnu-package-pv.scm-pv-New-file-new-variable.patch Description: Binary data
[PATCH] gnu/package/pv.scm (pv): New file, new variable
>From 7df5b2d6c71d7de70e4af59f1944c87c0b0ce807 Mon Sep 17 00:00:00 2001 From: Claes Wallin Date: Fri, 19 Jun 2015 20:12:09 +0200 Subject: [PATCH] gnu/package/pv.scm (pv): New file, new variable Signed-off-by: Claes Wallin --- gnu/packages/pv.scm | 51 +++ 1 file changed, 51 insertions(+) create mode 100644 gnu/packages/pv.scm diff --git a/gnu/packages/pv.scm b/gnu/packages/pv.scm new file mode 100644 index 000..bd159c9 --- /dev/null +++ b/gnu/packages/pv.scm @@ -0,0 +1,51 @@ +;;; GNU Guix --- Functional package management for GNU +;;; Copyright © 2012, 2013, 2015 Ludovic Courtès +;;; Copyright © 2015 Claes Wallin +;;; +;;; 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 pv) + #:use-module (guix licenses) + #:use-module (guix packages) + #:use-module (guix download) + #:use-module (guix build-system gnu) + #:export (pv)) + +(define pv + (package +(name "pv") +(version "1.6.0") +(source + (origin + (method url-fetch) + (uri (string-append "http://www.ivarch.com/programs/sources/pv-"; + version ".tar.bz2")) + (sha256 + (base32 +"13gg6r84pkvznpd1l11qw1jw9yna40gkgpni256khyx21m785khf" +(build-system gnu-build-system) +;(native-inputs `(("perl" ,perl))) +;(inputs `(("flex" ,flex))) +;(propagated-inputs `(("m4" ,m4))) +(home-page "http://www.ivarch.com/programs/pv.shtml";) +(synopsis "Pipeline progress indicator") +(description + "pv - Pipe Viewer - is a terminal-based tool for monitoring the progress +of data through a pipeline. It can be inserted into any normal pipeline +between two processes to give a visual indication of how quickly data is +passing through, how long it has taken, how near to completion it is, and an +estimate of how long it will be until completion.") +(license artistic2.0))) -- 2.2.1