From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Subject: Re: [PATCH 04/19] gnu: Add r-bh.
Date: Mon, 14 Sep 2015 12:50:00 +0200
Message-ID: <87fv2h8ecn.fsf@mdc-berlin.de>
References: <CAMfbzvBVwx-cs4hE=6o+rOgPWJx28arOEq2vnPaH2g8_V7s3hw@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:53806)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Ricardo.Wurmus@mdc-berlin.de>) id 1ZbRRp-0005FI-Vk
	for guix-devel@gnu.org; Mon, 14 Sep 2015 06:57:22 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Ricardo.Wurmus@mdc-berlin.de>) id 1ZbRRm-0004S6-QC
	for guix-devel@gnu.org; Mon, 14 Sep 2015 06:57:21 -0400
Received: from sinope.bbbm.mdc-berlin.de ([141.80.25.23]:59661)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Ricardo.Wurmus@mdc-berlin.de>) id 1ZbRRm-0004Rp-Hj
	for guix-devel@gnu.org; Mon, 14 Sep 2015 06:57:18 -0400
In-Reply-To: <CAMfbzvBVwx-cs4hE=6o+rOgPWJx28arOEq2vnPaH2g8_V7s3hw@mail.gmail.com>
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: Vicente Vera <vicentemvp@gmail.com>
Cc: guix-devel@gnu.org


> * gnu/packages/statistics.scm (r-bh): New variable.

[...]

> +    (synopsis "Boost C++ Header Files")
> +    (description
> +     "Boost provides free peer-reviewed portable C++ source libraries.
> +A large part of Boost is provided as C++ template code which is
> +resolved entirely at compile-time without linking. This package aims
> +to provide the most useful subset of Boost libraries for template use
> +among CRAN package. By placing these libraries in this package, we
> +offer a more efficient distribution system for CRAN as replication of
> +this code in the sources of other packages is avoided.")
> +    (license license:boost1.0)))

It seems ugly to duplicate the Boost headers in a separate package.  We
already have a =E2=80=9Cboost=E2=80=9D package, which provides the header=
s for this
version of Boost.  Would it be possible to use the existing =E2=80=9Cboos=
t=E2=80=9D
package in place of =E2=80=9Cr-bh=E2=80=9D?

If not: please use two spaces in the description.  I think the first two
sentences can be removed as they relate to the Boost library.  Also,
please use lower case for the synopsis.

~~ Ricardo