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

Build OpenMM 7.3.1 release candidate #7

Merged
merged 2 commits into from
Mar 1, 2019
Merged

Conversation

peastman
Copy link
Contributor

@jchodera can you check that all of this looks correct?

@jchodera
Copy link
Member

Looks good! We've been having issues with the conda-build compiler toolchain again (sigh) so we may need to do some debugging to get this built.

Have been buried in grantwriting this week...apologies for the slow response.

@peastman
Copy link
Contributor Author

peastman commented Mar 1, 2019

A bunch of the builds are failing. For the Mac builds, it looks like it's because the script invokes brew -y, and there's no such option as -y. For the Windows builds it has the error message

Error: bad character '"' in build/string: py37_cuda"100"_rc_1

@jchodera
Copy link
Member

jchodera commented Mar 1, 2019

For the Mac builds, it looks like it's because the script invokes brew -y, and there's no such option as -y

They keep changing the syntax. We've fixed that elsewhere, but not here. Can you implement this fix and get rid of the -y globally?

For the Windows builds it has the error message

We can't build conda packages for win using this build machinery yet. I started to experiment with this in this PR, but never got very far:

#2

@jchodera
Copy link
Member

jchodera commented Mar 1, 2019

I did get pretty far in the AppVeyor overhaul in #2, and installation of the CUDA 10.0 toolkit actually worked---but I think we're still building for CUDA 9.2, right?

@peastman
Copy link
Contributor Author

peastman commented Mar 1, 2019

Right.

@jchodera
Copy link
Member

jchodera commented Mar 1, 2019

OK, this should push all the packages after a few tries.

@jchodera jchodera merged commit c5cb0e7 into omnia-md:master Mar 1, 2019
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