From: Alex Kost <alezost@gmail.com>
To: "Claes Wallin (韋嘉誠)" <gnu@clacke.user.lysator.liu.se>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] gnu/package/pv.scm (pv): New file, new variable
Date: Fri, 26 Jun 2015 16:32:59 +0300 [thread overview]
Message-ID: <87r3oylhmc.fsf@gmail.com> (raw)
In-Reply-To: <CAGv_=BoV_FmOprMoSO5At8DOVsPtP5Qk6SxMr-UR3=Awngw3vQ@mail.gmail.com> ("Claes Wallin \=\?utf-8\?B\?KOmfi+WYieiqoCkiJ3M\=\?\= message of "Wed, 24 Jun 2015 23:35:00 +0200")
Claes Wallin (韋嘉誠) (2015-06-25 00:35 +0300) wrote:
> On Tue, Jun 23, 2015 at 1:43 PM, Alex Kost <alezost@gmail.com> 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 <claes.wallin@greatsinodevelopment.com>
> 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
next prev parent reply other threads:[~2015-06-26 13:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-19 18:36 [PATCH] gnu/package/pv.scm (pv): New file, new variable Claes Wallin (韋嘉誠)
2015-06-19 18:51 ` Claes Wallin (韋嘉誠)
2015-06-22 13:57 ` Claes Wallin (韋嘉誠)
2015-06-23 11:43 ` Alex Kost
2015-06-23 19:52 ` Claes Wallin (韋嘉誠)
2015-06-24 21:35 ` Claes Wallin (韋嘉誠)
2015-06-26 13:32 ` Alex Kost [this message]
2015-06-26 16:56 ` Claes Wallin (韋嘉誠)
2015-06-27 10:15 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87r3oylhmc.fsf@gmail.com \
--to=alezost@gmail.com \
--cc=gnu@clacke.user.lysator.liu.se \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).