Opened 4 years ago

Closed 4 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)

Portfile-scalapack.diff (2.2 KB) - added by dstrubbe (David Strubbe) 4 years ago.

Download all attachments as: .zip

Change History (2)

Changed 4 years ago by dstrubbe (David Strubbe)

Attachment: Portfile-scalapack.diff added

comment:1 Changed 4 years ago by seanfarley (Sean Farley)

Resolution: fixed
Status: newclosed

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.

Note: See TracTickets for help on using tickets.