search

conda-forge / openblas-feedstock

A conda-smithy repository for openblas.
BSD 3-Clause "New" or "Revised" License
9 stars 38 forks source link

Provide libblas.so and liblapack.so #23

Closed isuruf closed 7 years ago

isuruf commented 7 years ago

Any reason why https://github.com/conda-forge/openblas-feedstock/blob/master/recipe/build.sh#L46-L49 is commented?

jschueller commented 7 years ago

maybe it conflicts with lapack-feedstock ?

isuruf commented 7 years ago

Thanks. Shouldn't that be handled by features, just like blas is? I mean have lapack package track a lapack feature and let netlib and openblas be two providers.

jschueller commented 7 years ago

yes it should

jakirkham commented 7 years ago

FYI OpenBLAS has its own LAPACK.

As far as those lines, I commented them from the outset as there was a thought that people might be able to mix BLAS implementations. However, we have since seen this cause crashes (e.g. mixing linked libraries with OpenBLAS and MKL). So it might be ok to uncomment them. For the most part, if I need those lines, I add them to the recipe that builds with a BLAS and then remove the symlinks at the end of the build.

isuruf commented 7 years ago

It's weird that conda-forge has only one blas implementation (openblas and no netlib) but has two lapack (openblas and netlib) with the netlib lapack bundling netlib's blas.

isuruf commented 7 years ago

Created https://github.com/conda-forge/blas-feedstock/issues/6