From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] gnu: fftw: Build threads library. Date: Wed, 16 Nov 2016 22:23:22 +0100 Message-ID: <878tsjf7at.fsf@gnu.org> References: <20161114224342.11644-1-rekado@elephly.net> <20161114224342.11644-2-rekado@elephly.net> <87lgwlf3ug.fsf@gnu.org> <87eg2c5vm1.fsf@elephly.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:48423) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c77g1-0001ek-KO for guix-devel@gnu.org; Wed, 16 Nov 2016 16:23:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c77fx-0006BX-33 for guix-devel@gnu.org; Wed, 16 Nov 2016 16:23:29 -0500 In-Reply-To: <87eg2c5vm1.fsf@elephly.net> (Ricardo Wurmus's message of "Tue, 15 Nov 2016 21:36:06 +0100") 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: Ricardo Wurmus Cc: guix-devel@gnu.org Ricardo Wurmus skribis: > Ludovic Court=C3=A8s writes: > >> Ricardo Wurmus skribis: >> >>> * gnu/packages/algebra.scm (fftw)[arguments]: Add "--enable-threads" to >>> configure flags. >>> --- >>> gnu/packages/algebra.scm | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/gnu/packages/algebra.scm b/gnu/packages/algebra.scm >>> index 8e9695d..e212775 100644 >>> --- a/gnu/packages/algebra.scm >>> +++ b/gnu/packages/algebra.scm >>> @@ -544,7 +544,8 @@ a C program.") >>> "1kwbx92ps0r7s2mqy7lxbxanslxdzj7dp7r7gmdkzv1j8yqf3kwf")= ))) >>> (build-system gnu-build-system) >>> (arguments >>> - '(#:configure-flags '("--enable-shared" "--enable-openmp") >>> + '(#:configure-flags >>> + '("--enable-shared" "--enable-openmp" "--enable-threads") >>> #:phases (alist-cons-before >>> 'build 'no-native >>> (lambda _ >> >> LGTM! >> >> Per >> , >> this is a perfect candidate for =E2=80=98staging=E2=80=99. > > Thanks for the reference. We don=E2=80=99t have a staging branch yet, as= far as > I can tell. May I just push this commit to a new =E2=80=9Cstaging=E2=80= =9D branch then? Yep! So the idea would be to freeze it in ~2 weeks (Nov. 30) and to merge it in 3 weeks (Dec. 7). Let=E2=80=99s see if we can be disciplined. = :-) Ludo=E2=80=99.