Re: [PATCH] gnu: fftw: Build threads library.

[Ricardo Wurmus <rekado@elephly.net>]

Ludovic Courtès <ludo@gnu.org> writes:

> Ricardo Wurmus <rekado@elephly.net> 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
> <https://lists.gnu.org/archive/html/guix-devel/2016-10/msg00933.html>,
> this is a perfect candidate for ‘staging’.

Thanks for the reference.  We don’t have a staging branch yet, as far as
I can tell.  May I just push this commit to a new “staging” branch then?

> (In the meantime, if that helps, you could additionally create an FFTW
> variant for use by Ardour, in master.)

Yes, I’ll do that.

~~ Ricardo