From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ricardo Wurmus <rekado@elephly.net>
Subject: Re: [PATCH 2/4] gnu: Add r-bigmemory-sri.
Date: Fri, 28 Oct 2016 22:44:54 +0200
Message-ID: <87lgx8jja2.fsf@elephly.net>
References: <87a8dohd7q.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]:32915)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <rekado@elephly.net>) id 1c0E1X-0006Ga-1B
	for guix-devel@gnu.org; Fri, 28 Oct 2016 16:45:12 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <rekado@elephly.net>) id 1c0E1W-0005Nv-8S
	for guix-devel@gnu.org; Fri, 28 Oct 2016 16:45:11 -0400
In-reply-to: <87a8dohd7q.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" <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
To: Roel Janssen <roel@gnu.org>
Cc: guix-devel@gnu.org


Roel Janssen <roel@gnu.org> writes:

> * gnu/packages/statistics.scm (r-bigmemory-sri): New variable.
> ---
>  gnu/packages/statistics.scm | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
>
> diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm
> index 9e2d4a9..a50e792 100644
> --- a/gnu/packages/statistics.scm
> +++ b/gnu/packages/statistics.scm
> @@ -2810,6 +2810,25 @@ via mutexes and may eventually support interprocess communication and
>  message passing.")
>      (license (list license:lgpl2.1 license:asl2.0))))
>  
> +(define-public r-bigmemory-sri
> +  (package
> +    (name "r-bigmemory-sri")
> +    (version "0.1.3")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (cran-uri "bigmemory.sri" version))

I think this should have a property to record the upstream name to help
the updater.

> +       (sha256
> +        (base32
> +         "0mg14ilwdkd64q2ri9jdwnk7mp55dqim7xfifrs65sdsv1934h2m"))))
> +    (build-system r-build-system)
> +    (home-page
> +     "http://cran.r-project.org/web/packages/bigmemory.sri")

Please pull them onto one line.

> +    (synopsis "A shared resource interface for the bigmemory
> package")

Don’t start with “A”.

> +    (description "This package provides a shared resource interface for the
> +bigmemory and synchronicity packages.")
> +    (license (list license:lgpl3 license:asl2.0))))

What does this list mean?
Also: is this LGPL3+ or LGPL3 only?

~~ Ricardo