Skip to content
This repository was archived by the owner on Mar 21, 2018. It is now read-only.

Pin boost#6

Closed
jschueller wants to merge 2 commits intoconda-forge:masterfrom
jschueller:patch-1
Closed

Pin boost#6
jschueller wants to merge 2 commits intoconda-forge:masterfrom
jschueller:patch-1

Conversation

@jschueller
Copy link
Copy Markdown
Contributor

No description provided.

@conda-forge-linter
Copy link
Copy Markdown

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jakirkham
Copy link
Copy Markdown
Member

Seems we are having some troubles here, but it may be caused by changes we have made in terms of handling BLAS dependent packages. We should modernize your stack @183amir. Could you please remind which things are BLAS dependent?

@jakirkham
Copy link
Copy Markdown
Member

jakirkham commented May 21, 2016

Moral of the story, @183amir, just follow along with what we have done with NumPy as can be seen here. Anything that says blas you need. Also, anything that says variant you need. Put your best attempt at this in a PR and ping me. We will go from there.

@183amir
Copy link
Copy Markdown
Contributor

183amir commented May 21, 2016

@jakirkham only bob.math links against blas.

@jakirkham
Copy link
Copy Markdown
Member

Right, which is why we need to update this. We just provided BLAS support in conda-forge. It happened at the same time as this Boost update was proposed. So, you are in a special situation as both affect bob.math. In any event, I think this will be easy to fix. It is pretty much a copy-and-paste job. Just follow the NumPy recipe linked above.

@183amir 183amir closed this May 21, 2016
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.

4 participants