Page MenuHomeFreeBSD

Upgrading math/atlas to 3.10.2
AbandonedPublic

Authored by thierry on Nov 11 2014, 9:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 16, 9:33 PM
Unknown Object (File)
Thu, Jul 11, 6:34 AM
Unknown Object (File)
Wed, Jul 10, 5:36 AM
Unknown Object (File)
Tue, Jul 9, 6:12 PM
Unknown Object (File)
Jun 12 2024, 10:50 PM
Unknown Object (File)
Jun 8 2024, 3:46 AM
Unknown Object (File)
May 13 2024, 11:59 AM
Unknown Object (File)
May 11 2024, 1:50 AM
Subscribers
None
Tokens
"Like" token, awarded by pfg.

Details

Reviewers
pfg
Summary

math/atlas is an important port, but a complex one, and I'd like a pair review before committing it.

The attached patch is almost ready, but it still contains unused variables from the previous version and it needs a @mode in plist, but I'd like to be sure that it's OK before going further.

Test Plan
  • All tests passed, but some symbols are missing and it needs to be linked with "-llapack -lblas". It seems that it was already so previously (LIB_DEPENDS+= libblas.so if MAINTAINER_MODE), but anyway that seems strange...
  • Tested with some other ports depending on libatlas.

(Notice sent to maho@ )

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

thierry retitled this revision from to Upgrading math/atlas to 3.10.2.
thierry updated this object.
thierry edited the test plan for this revision. (Show Details)
thierry added a reviewer: pfg.
thierry set the repository for this revision to rP FreeBSD ports repository.

I think this is OK.

On the long run we should probably deprecate Atlas and just use OpenBlas for everything.

Thanks Pedro!

I've just proposed a new revision
https://reviews.freebsd.org/D1354
to handle the different Blas/lapack options, and set OpenBlas by default.

For an unknown reason, it does not build anymore with the attached patch.
It needs some time to investigate and hopefully to fix.