[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gentoo-dev
Subject:    [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in sci-chemistry/gromacs:
From:       Donnie Berkholz <dberkholz () gentoo ! org>
Date:       2009-01-22 4:53:59
Message-ID: 20090122045359.GC9101 () hermes
[Download RAW message or body]


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 #3 (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]