Summary: | math/suitesparse: drop USES=fortran | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Jan Beich <jbeich> | ||||
Component: | Individual Port(s) | Assignee: | freebsd-fortran (Nobody) <fortran> | ||||
Status: | In Progress --- | ||||||
Severity: | Affects Only Me | CC: | rsmith, thierry | ||||
Priority: | --- | Keywords: | patch | ||||
Version: | Latest | Flags: | thierry:
maintainer-feedback+
|
||||
Hardware: | Any | ||||||
OS: | Any | ||||||
See Also: | https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=267742 | ||||||
Attachments: |
|
Description
Jan Beich
2019-09-28 13:53:18 UTC
$ grep -i fortran ./SuiteSparse_config/SuiteSparse_config.mk BLAS = -lrefblas -lgfortran -lstdc++ # use the Intel ifort compiler for Fortran codes # standard Fortran LAPACK along with OpenBLAS to obtain very good # make BLAS='-lacml -lgfortran' # Fortran compiler (not required for 'make' or 'make library') # A Fortran compiler is optional. Only required for the optional Fortran F77 ?= gfortran ## BLAS = -lrefblas -lgfortran @echo 'Fortran compiler: F77= ' '$(F77)' @echo 'Fortran flags: F77FLAGS= ' '$(F77FLAGS)' So it seems that Fortran is optional: # A Fortran compiler is optional. Only required for the optional Fortran # interfaces to AMD and UMFPACK. Not needed by 'make' or 'make install' Does anyone need these interfaces? Note: if we adopt the proposed patch, maintainership should be dropped from fortran mailing list. I was just building suitesparse 5.8.0. If I remove "fortran" from USES, I get this error: ===> suitesparse-5.8.0 is marked as broken: USES=gmake is incompatible with cmake's ninja-generator. If I also remove "gmake" from USES, I get another error: ===> Building for suitesparse-5.8.0 ninja: error: loading 'build.ninja': No such file or directory So it seems to me that we do need USES=fortran. Please note that after the split of math/suitesparse into several ports, only math/suitesparse-amd uses fortran. I'm not sure how it is used, and we could probably make it optional. Sorry, but starting with 6.0.0 SuiteSparse checks for a Fortran compiler from the beginning, i.e. math/suitesparse-config . See PR 267742. Since v6.0.2, FORTRAN is no more used, excepted for AMD, like in releases 5.*. It could even be disabled in AMD, but in this case amd.f and amdbar.f would be skipped: there is no interest for that, since FORTRAN is already brought by OpenBLAS. |