From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eric Bavier <ericbavier@gmail.com>
Subject: Re: [PATCH 1/2] Add offlineimap
Date: Tue, 20 May 2014 09:07:02 -0500
Message-ID: <87mwecr1bt.fsf@gmail.com>
References: <87ppj9qwi1.fsf@member.fsf.org> <87r43oq4b8.fsf@gnu.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:60959)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <ericbavier@gmail.com>) id 1WmkeA-0000Zp-Ph
	for Guix-devel@gnu.org; Tue, 20 May 2014 10:04:08 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <ericbavier@gmail.com>) id 1Wmke5-0008BV-3z
	for Guix-devel@gnu.org; Tue, 20 May 2014 10:04:02 -0400
In-reply-to: <87r43oq4b8.fsf@gnu.org>
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
To: =?utf-8?Q?Ludovic_Court=C3=A8s?= <ludo@gnu.org>
Cc: Guix-devel@gnu.org


Ludovic Courtès writes:

> Eric Bavier <ericbavier@gmail.com> skribis:
>
>> From a6dccce1e179f96e39cff2ea67fb436aeaf06cd4 Mon Sep 17 00:00:00 2001
>> From: Eric Bavier <bavier@member.fsf.org>
>> Date: Mon, 19 May 2014 16:33:08 -0500
>> Subject: [PATCH 1/2] gnu: Add offlineimap
>>
>> * gnu/packages/mail.scm (offlineimap): New variable.
>
> Looks good to me!
>
>> +    (synopsis "Synch emails between two repositories")
>
> “Sync” or “Synchronize”?

I changed it to "Synchronize"

>
>> +    (license gpl2)))
>
> From a quick glance at the code, it appears to be ‘gpl2+’ (with “or
> later” clause.)  Could you double-check and adjust accordingly?

I double-checked, and yes, it's gpl2+.

Pushed with those changes.  Thanks.

-- 
Eric Bavier