[Bug 257400] science/py-scipy Add OPTION for Atlas blas

From: <bugzilla-noreply_at_freebsd.org>
Date: Thu, 29 Jul 2021 22:20:53 +0000
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=257400

alt2600_at_icloud.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #226716|0                           |1
        is obsolete|                            |

--- Comment #9 from alt2600_at_icloud.com ---
Created attachment 226789
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=226789&action=edit
git-diff-scipy-add-atlas-rev1

confirmed under openblas based bhyve to configure and build correctly. Noticed
under PR 251718 and from the configure this wants both openblas and blas so I
added back a LIB_DEPENDS for the OPENBLAS option to include it, but otherwise
the uses blaslapack:openblas caused no issue. not sure if in the end this port
wants cblas or how applicable that is with openblas but I just wanted to
confirm no breakage, not extend features. This patch will allow ATLAS or
OpenBLAS use, maintaining OpenBLAS as the default. I consider this patch to now
be tested and complete.

it should work for Netlib case, but if Atlas is installed it will bring in an
Atlas dependency, despite possibly not linking to it in reality. If someone
wants to get netlib I'm sure they can explore the syntax further. I can confirm
it appears to configure for it, build, and install, but still shows an Atlas
pkg dependency once installed so it remains excluded. see math/py-numpy if
someone really wanted to take the time for netlib. 


### make configure tail excerpt
creating /tmp/tmpqmtazvj7/tmp
creating /tmp/tmpqmtazvj7/tmp/tmpqmtazvj7
compile options: '-MMD -MF /tmp/tmpqmtazvj7/file.c.d -c'
cc: /tmp/tmpqmtazvj7/file.c
  FOUND:
    libraries = ['openblas', 'blas']
    library_dirs = ['/usr/local/lib']
    include_dirs = ['/usr/include', '/usr/local/include']
    language = c
    define_macros = [('HAVE_CBLAS', None)]

  FOUND:
    define_macros = [('NO_ATLAS_INFO', 1), ('HAVE_CBLAS', None)]
    libraries = ['openblas', 'blas']
    library_dirs = ['/usr/local/lib']
    include_dirs = ['/usr/include', '/usr/local/include']
    language = c

non-existing path in 'scipy/signal/windows': 'tests'
non-existing path in 'scipy/sparse': 'sparsetools/sparsetools_impl.h'
non-existing path in 'scipy/sparse': 'sparsetools/bsr_impl.h'
non-existing path in 'scipy/sparse': 'sparsetools/csc_impl.h'
non-existing path in 'scipy/sparse': 'sparsetools/csr_impl.h'
non-existing path in 'scipy/sparse': 'sparsetools/other_impl.h'
[makenpz] scipy/special/tests/data/boost.npz not rebuilt
[makenpz] scipy/special/tests/data/gsl.npz not rebuilt
[makenpz] scipy/special/tests/data/local.npz not rebuilt
running config



/usr/ports/science/py-scipy $ sudo make install clean
Password:
===>  Installing for py38-scipy-1.6.0
===>  Checking if py38-scipy is already installed
===>   Registering installation for py38-scipy-1.6.0
Installing py38-scipy-1.6.0...


/usr/ports/science/py-scipy $ pkg info -dr py38-scipy
py38-scipy-1.6.0
Depends on     :
        py38-numpy-1.16.6_2,1
        openblas-0.3.16,1
        blas-3.10.0
        python38-3.8.11
        gcc10-10.3.0
        py38-setuptools-57.0.0

-- 
You are receiving this mail because:
You are the assignee for the bug.
Received on Thu Jul 29 2021 - 22:20:53 UTC

Original text of this message