From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
To: Vicente Vera <vicentemvp@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 04/19] gnu: Add r-bh.
Date: Mon, 14 Sep 2015 12:50:00 +0200 [thread overview]
Message-ID: <87fv2h8ecn.fsf@mdc-berlin.de> (raw)
In-Reply-To: <CAMfbzvBVwx-cs4hE=6o+rOgPWJx28arOEq2vnPaH2g8_V7s3hw@mail.gmail.com>
> * 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 “boost” package, which provides the headers for this
version of Boost. Would it be possible to use the existing “boost”
package in place of “r-bh”?
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
next prev parent reply other threads:[~2015-09-14 10:57 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-13 16:46 [PATCH 04/19] gnu: Add r-bh Vicente Vera
2015-09-14 10:50 ` Ricardo Wurmus [this message]
2015-09-14 19:43 ` Vicente Vera
2015-09-15 17:34 ` Vicente Vera
2015-09-17 12:02 ` Ricardo Wurmus
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=87fv2h8ecn.fsf@mdc-berlin.de \
--to=ricardo.wurmus@mdc-berlin.de \
--cc=guix-devel@gnu.org \
--cc=vicentemvp@gmail.com \
/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).