From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Wurmus Subject: Re: [PATCH] Add Vc Date: Mon, 03 Oct 2016 10:45:15 +0200 Message-ID: <87int93jok.fsf@elephly.net> References: <20161001222412.GA6848@jasmine> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:57774) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bqysL-0001cm-B5 for guix-devel@gnu.org; Mon, 03 Oct 2016 04:45:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bqysH-0005xH-55 for guix-devel@gnu.org; Mon, 03 Oct 2016 04:45:28 -0400 Received: from sender163-mail.zoho.com ([74.201.84.163]:21399) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bqysG-0005x1-Rg for guix-devel@gnu.org; Mon, 03 Oct 2016 04:45:25 -0400 In-reply-to: <20161001222412.GA6848@jasmine> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Leo Famulari Cc: guix-devel@gnu.org Leo Famulari writes: > This is a depend of Krita. > From 808b47b3b7a9d769f3a6a873d19a0051f64720f7 Mon Sep 17 00:00:00 2001 > From: Leo Famulari > Date: Fri, 30 Sep 2016 13:38:20 -0400 > Subject: [PATCH] gnu: Add Vc. > * gnu/packages/maths.scm (vc): New variable. > --- > gnu/packages/maths.scm | 34 ++++++++++++++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm > index 0401cd3..c033123 100644 > --- a/gnu/packages/maths.scm > +++ b/gnu/packages/maths.scm > @@ -2629,3 +2629,37 @@ the same amount of space as the original point representation. This is useful > when using the Gilbert curve as a space filling curve through a > high-dimensional space where not all demensions have the same cardinality.") > (license license:lgpl2.1+))) > + > +(define-public vc > + (package > + (name "vc") > + (version "1.2.0") > + (source > + (origin (method url-fetch) > + (uri (string-append "https://github.com/VcDevel/Vc/releases/" > + "download/" version "/Vc-" version ".tar.gz")) > + (sha256 > + (base32 > + "1rh6dhqar3y07n4xqyml0sa0v48qv3ch9dc3yc2in855hlh4vnqi")))) > + (build-system cmake-build-system) > + (arguments > + '(#:configure-flags > + '("-DBUILD_TESTING=ON"))) ; Is this the test suite we want? > + (synopsis "SIMD Vector Classes for C++ ") There’s a trailing space here. Also, “Vector Classes” probably should be lowercase. > + (description "Vc provides portable, zero-overhead C++ types for explicitly > +data-parallel programming. It is a library designed to ease explicit > +vectorization of C++ code. Its types enable explicitly stating data-parallel > +operations on multiple values. The parallelism is therefore added via the type > +system. Vc has an intuitive API and provides portability between different > +compilers and compiler versions as well as portability between different vector > +instruction sets. Thus, an application written with Vc can be > compiled for: Before “Thus” you need an additional space. > +@enumerate > +@item AVX and AVX2 > +@item SSE2 upto SSE4.2 or SSE4a > +@item Scalar > +@item MIC > +@item NEON (in development) > +@item NVIDIA GPUs / CUDA (in development) > +@end enumerate\n") > + (home-page "https://github.com/VcDevel") Shouldn’t this be https://github.com/VcDevel/Vc ? > + (license license:bsd-3))) The rest looks good to me! Thanks! ~~ Ricardo