Skip to content
This repository has been archived by the owner on Sep 29, 2022. It is now read-only.

Fix lapack for conda_builder_linux from ContinuumIO/docker-images #658

Merged
merged 1 commit into from
Apr 2, 2016

Conversation

bjodah
Copy link
Contributor

@bjodah bjodah commented Apr 2, 2016

See anaconda/docker-images#23
ping @msarahan

Actually I would need to remove both gcc and libgcc from meta.yaml for this to work out of the
box with conda_builder_linux.

What is the policy for adding compiler requirements in conda-recipes? My take is that
they can be kept in the recipe but commented out.

@bjodah bjodah mentioned this pull request Apr 2, 2016
@msarahan
Copy link
Collaborator

msarahan commented Apr 2, 2016

Perfect, I think this is exactly what's needed.

gcc should be removed from meta.yaml, but libgcc might need to stay in. The reason is that the docker build image contains runtimes that are potentially (probably) newer than system ones. These are backwards compatible, but any code you compile with the docker image may not find the symbols it needs in older runtimes.

This libgcc is a bit of a work in progress. If you need it now, grab it from my anaconda.org channel:

conda install -c msarahan libgcc

The one on the defaults channel should soon be updated to be the same. The tricky detail about doing things with GCC the way that we have is that our runtimes need to always be the same or newer version than the system libraries.

Ideally, compiler requirements could be made explicit, or at the very least, encoded into readily accessible metadata. We sort of do that on Windows, since each version of Python uses a different version of Visual Studio. This is another WIP, and the most current issue on the topic is at conda/conda-build#848 - we'd welcome your thoughts on the topic.

@msarahan msarahan merged commit df44076 into conda-archive:master Apr 2, 2016
@bjodah
Copy link
Contributor Author

bjodah commented Apr 2, 2016

@msarahan great! Thanks for the explanation and the hint about libgcc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants