Opened 9 years ago
Closed 9 years ago
#49511 closed enhancement (fixed)
scalapack: add test phase, define BLAS/LAPACK dependencies, don't install cmake files
Reported by: | dstrubbe (David Strubbe) | Owned by: | seanfarley (Sean Farley) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.4 |
Keywords: | haspatch | Cc: | |
Port: | scalapack |
Description
Enable test phase for scalapack. Doing so required setting MPIEXEC, allowing use of 'ps', and turning on BUILD_TESTING.
Also, I found that about half of the tests failed, some after timing out (at 1500 s). To fix this, inspired by a Homebrew ticket I found, I need to avoid using the raw Accelerate framework. vecLibFort can be used instead. I also added variants to allow use of OpenBLAS or ATLAS instead. They all pass all the tests (promptly) now. Before, ScaLAPACK's own search for linear algebra libraries was being used, and could find a mix of Accelerate and ATLAS libraries depending on whether ATLAS was installed at the time or not, which is against the typical approach of MacPorts anyway. These issues did not seem to cause trouble in actual application usage for me, though maybe that is just a question of which routines were being used (since about half passed the tests).
Removed some cmake files from installation which do not seem useful.
Attachments (1)
Change History (2)
Changed 9 years ago by dstrubbe (David Strubbe)
Attachment: | Portfile-scalapack.diff added |
---|
comment:1 Changed 9 years ago by seanfarley (Sean Farley)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed in r144288. Thanks for the patch! The cmake files need to stay for other packages that use cmake to find scalapack and its flags.