Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] New pygraphviz recipe #765

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] New pygraphviz recipe #765

wants to merge 1 commit into from

Conversation

nividic
Copy link
Contributor

@nividic nividic commented Apr 21, 2017

Try to build pygraphviz

@Lnaden
Copy link
Contributor

Lnaden commented Apr 21, 2017

I would hold off on merging this for just a bit (this morning, 7/21/17), I am about to merge the #761 which will change the docker build image. We will require conda-forge as a channel after that PR along with omnia, so the install instructions will change.

We also encourage migrating packages over to conda-forge after that as well, so it might be worth trying to build the package there instead.

@Lnaden
Copy link
Contributor

Lnaden commented May 1, 2017

This PR is having issues building, missing things like coverage and toolchain (which I think is conda-forge only). We won't be able to merge the PR which makes this conda-forge ready (#761) until OpenMM patch 7.1.1 is ready. I don't have an exact timeline, but it should not be too long. We have merged the dev version of the conda-forge PR and so far things look good.

If you can fix this PR without conda-forge requirements before we get the 7.1.1 patch for OpenMM in, feel free to merge this.

The toolchain from conda forge is basically just a bunch of sym links and environment variables, so it should be possible to build without it, but then the build.sh script will be different.

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