Eric Bavier writes:

> On 2015-03-18 05:09, Ricardo Wurmus wrote:
>> +       (alist-cons-before
>> +        'install 'prepare-out
>> +        (lambda _
>> +          (mkdir-p (string-append (assoc-ref %outputs "out") "/lib"))
>> +          (mkdir-p (string-append (assoc-ref %outputs "out") 
>> "/include")))
>
> Could you note that README.txt states that "These directories must 
> already exist"?

I will add a note.

>> +    (inputs
>> +     `(("tbb" ,tbb)
>> +       ("lapack" ,lapack)
>> +       ("atlas" ,atlas)))
>
> It appears from the #:make-flags that only the blas library is used.  
> Are both "lapack" and "atlas" required?  Or would just one suffice?

You are right: it builds fine with just "lapack".  I'll remove "atlas"
from the inputs.  Thanks for the review!

Attached is a new version of the patch.

~~ Ricardo