conda-forge / openblas-feedstock

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

Missing dylib files in 0.3.6 #75

Closed jiahuanglin closed 4 years ago

jiahuanglin commented 5 years ago

Issue: With the new openblas, you can't import numpy.multiarray

  Referenced from: /Users/jiahuanglin/anaconda3/envs/test/lib/python2.7/site-packages/numpy/core/_multiarray_umath.so
  Reason: image not found

In 0.3.5:

~/anaconda3/pkgs/openblas-0.3.5-h436c29b_1001/lib                                                                                      
λ ll
total 248600
drwxr-xr-x  3 jiahuanglin  staff    96B 20 Mar 18:08 cmake
lrwxr-xr-x  1 jiahuanglin  staff    21B 20 Mar 18:08 libblas.a -> libopenblasp-r0.3.5.a
lrwxr-xr-x  1 jiahuanglin  staff    25B 20 Mar 18:08 libblas.dylib -> libopenblasp-r0.3.5.dylib
lrwxr-xr-x  1 jiahuanglin  staff    21B 20 Mar 18:08 libcblas.a -> libopenblasp-r0.3.5.a
lrwxr-xr-x  1 jiahuanglin  staff    25B 20 Mar 18:08 libcblas.dylib -> libopenblasp-r0.3.5.dylib
lrwxr-xr-x  1 jiahuanglin  staff    21B 20 Mar 18:08 liblapack.a -> libopenblasp-r0.3.5.a
lrwxr-xr-x  1 jiahuanglin  staff    25B 20 Mar 18:08 liblapack.dylib -> libopenblasp-r0.3.5.dylib
lrwxr-xr-x  1 jiahuanglin  staff    21B 20 Mar 18:08 libopenblas.a -> libopenblasp-r0.3.5.a
lrwxr-xr-x  1 jiahuanglin  staff    25B 20 Mar 18:08 libopenblas.dylib -> libopenblasp-r0.3.5.dylib
-rw-rw-r--  3 jiahuanglin  staff    62M 12 Mar 18:57 libopenblasp-r0.3.5.a
-rwxrwxr-x  1 jiahuanglin  staff    60M 12 Mar 18:58 libopenblasp-r0.3.5.dylib
drwxr-xr-x  6 jiahuanglin  staff   192B 20 Mar 18:08 pkgconfig

In 0.3.6:

~/anaconda3/pkgs/openblas-0.3.6-hd44dcd8_4/lib                                                                                        
λ ll
total 127952
drwxr-xr-x  3 jiahuanglin  staff    96B 16 Jul 12:52 cmake
lrwxrwxr-x  1 jiahuanglin  staff    21B  5 Jul 12:46 libopenblas.a -> libopenblasp-r0.3.6.a
-rw-rw-r--  4 jiahuanglin  staff    62M  5 Jul 12:45 libopenblasp-r0.3.6.a
drwxr-xr-x  3 jiahuanglin  staff    96B 16 Jul 12:52 pkgconfig
isuruf commented 5 years ago

Please provide details in the issue template.

s22chan commented 5 years ago

I don't see how the conda environment is relevant: it's a packaging problem. There's plenty of other issues that don't follow the template.

jschueller commented 5 years ago

Seems openblas holds only the static libs now, and libopenblas the shared ones: see https://github.com/conda-forge/openblas-feedstock/pull/70

s22chan commented 5 years ago

Thanks @jschueller, so it looks like the symlink done at https://github.com/conda-forge/openblas-feedstock/blob/ca646ba227a1a2910ed1c12ddbb27a3f23ce03f4/recipe/build.sh#L56 needs to replicated in the libopenblas feedstock

isuruf commented 5 years ago

needs to replicated in the libopenblas feedstock

It's the same feedstock. @jiahuanglin needs to give the full error message and the conda env to debug this problem.

s22chan commented 5 years ago

@isuruf I'll try to submit a PR later when I have the time. It's pretty simple though, the libopenblas conda package is missing the dylib

isuruf commented 5 years ago

I'm sorry, but I don't understand what you are saying. Which file is missing? Why do you need the missing file? What's the error you get?

s22chan commented 5 years ago

@jiahuanglin can you post the full error that shows which dylib is missing?

isuruf commented 5 years ago

And also the output of conda list -n test