Skip to content

[WIP] Workaround (or permanent fix?) for depends_on(py-numpy)#722

Closed
citibeth wants to merge 2 commits intospack:developfrom
citibeth:efischer/160401-PythonRpathProblem
Closed

[WIP] Workaround (or permanent fix?) for depends_on(py-numpy)#722
citibeth wants to merge 2 commits intospack:developfrom
citibeth:efischer/160401-PythonRpathProblem

Conversation

@citibeth
Copy link
Copy Markdown
Member

@citibeth citibeth commented Apr 2, 2016

I merged today from develop, and it broke anything that depends_on('py-numpy'). I believe these used to work. But now I have to set LD_LIBRARY_PATH to point to Numpy (or else python3 -c numpy won't find libopenblas.so).

The most likely reason for this problem is that OpenBLAS builds a shared object (previously I was using ATLAS, I believe with static library). That has uncovered a latent problem, that Python extensions (eg, those built in py-numpy) do not use RPATH (and thus cannot find libopenblas.so). Or... maybe this problem with Python libs was introduced in the last two weeks. Or maybe with
refactoring of build_environment on March 21?

Seeking advice...

See: #719

@citibeth
Copy link
Copy Markdown
Member Author

citibeth commented Apr 4, 2016

I have some ideas of how we can get RPATH into Python packages. I don't think this PR should be merged in the long run.

@adamjstewart
Copy link
Copy Markdown
Member

@citibeth Can this be closed now that #1188 is merged? It doesn't solve the RPATH problem globally, but it does seem to fix numpy.

@citibeth
Copy link
Copy Markdown
Member Author

Yes let's close it, I never thought the code submitted there was what we
really wanted.

Better would be to figure out how to call setuptools, use compiler
wrappers, etc. to get RPATHs in Python extensions.

On Thu, Aug 11, 2016 at 5:10 PM, Adam J. Stewart notifications@github.com
wrote:

@citibeth https://github.com/citibeth Can this be closed now that #1188
#1188 is merged? It doesn't solve the
RPATH problem globally, but it does seem to fix numpy.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#722 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB1cdwYk6Vp_q26tfesKsBcomR8E8covks5qe4_DgaJpZM4H-MTl
.

@citibeth citibeth closed this Aug 11, 2016
matz-e pushed a commit to matz-e/spack that referenced this pull request Apr 27, 2020
* remove neurom@mut_morphio dep on morph-tool

* re-add morphio dep

* add neuroc

* add morph-repair

* update license

* Add bbp-morphology workflow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants