Bug 257321

Summary: math/eigen3: Remove unnecessary OpenBLAS dependency
Product: Ports & Packages Reporter: alt2600
Component: Individual Port(s)Assignee: freebsd-kde (group) <kde>
Status: Closed FIXED    
Severity: Affects Some People CC: adridg
Priority: --- Keywords: needs-patch, needs-qa
Version: LatestFlags: bugzilla: maintainer-feedback? (kde)
koobs: merge-quarterly?
Hardware: Any   
OS: Any   
URL: https://eigen.tuxfamily.org/index.php?title=Main_Page#Requirements
Bug Depends on:    
Bug Blocks: 251842    
Attachments:
Description Flags
patch-eigen3-Makefile-Disable-Full none

Description alt2600 2021-07-22 00:30:55 UTC 
https://eigen.tuxfamily.org/index.php?title=Main_Page#Requirements

Requirements

Eigen doesn't have any dependencies other than the C++ standard library.

We use the CMake build system, but only to build the documentation and unit-tests, and to automate installation. If you just want to use Eigen, you can use the header files right away. There is no binary library to link to, and no configured header file. Eigen is a pure template library defined in the headers. 




there is no reason why this has OpenBLAS as a required library. This should also close PR 251842, for the same basic reason. Nor would it seemingly need any of the other listed library dependencies

Library dependencies:

        libcholmod.so : math/suitesparse-cholmod
        libumfpack.so : math/suitesparse-umfpack
        libsuperlu.so : math/superlu
        libadolc.so : math/adol-c
        libfftw3.so : math/fftw3
        libmpfr.so : math/mpfr
        libgmp.so : math/gmp
        libboost_thread.so : devel/boost-libs
        libopenblas.so : math/openblas
Comment 1 alt2600 2021-07-22 17:00:22 UTC 
Created attachment 226619 [details]
patch-eigen3-Makefile-Disable-Full

Patch attached. This simply turns off the FULL option as a Default

check-orphans confirms there is no change with "building" the port.

/usr/ports/math/eigen3|$} sudo make check-orphans
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
===> No pkg-plist issues found (check-plist)
Comment 2 commit-hook freebsd_committer freebsd_triage 2021-09-07 21:31:08 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=b41385d1aad52bf5e2e6f2bb65777e201d9e5cd7

commit b41385d1aad52bf5e2e6f2bb65777e201d9e5cd7
Author:     Adriaan de Groot <adridg@FreeBSD.org>
AuthorDate: 2021-09-07 08:58:27 +0000
Commit:     Adriaan de Groot <adridg@FreeBSD.org>
CommitDate: 2021-09-07 21:28:08 +0000

    math/eigen3: massage OPTIONs

    Existing OPTIONs pull in too many dependencies -- Eigen is a template
    library and doesn't **need** any of the libs -- and are overly-specific,
    like requiring a particular BLAS implementation.

    PR:             257321 251842
    Reported by:    alt2600@icloud.com

 UPDATING             | 18 ++++++++++++++++++
 math/eigen3/Makefile | 16 ++++++++++------
 2 files changed, 28 insertions(+), 6 deletions(-)