On 04:14 Thu 22 Jan     , Jeffrey Gardner (je_fro) wrote:
> Index: gromacs-4.0.3.ebuild

>       if ( use double-precision ) ; then

What is this pointless subshell doing? There's heavy use of subshells 
throughout this ebuild for no reason. I quoted them below when I saw 
them.

>                       if ( use sse || use sse2 ) ; then

>                       if ( ! use sse && ! use sse2 && ! use 3dnow ) ; then

>       # if we need external blas

I really hope the other option is not internal blas. If so, it shouldn't 
be an option at all. Same goes for lapack.

>       if ( use double-precision && use single-precision ); then

-- 
Thanks,
Donnie

Donnie Berkholz
Developer, Gentoo Linux
Blog: http://dberkholz.wordpress.com

Attachment: pgpGkq7Fcn4aO.pgp
Description: PGP signature

Reply via email to